Address review comments on PR 12969

[chrisdecenzo]

Problem

• Not all comments in PR were addressed

Proposed Solution

• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - LoadContentApp in examples/tv-app/linux/AppImpl.h and various other places should perhaps take CatalogVendorApp by reference, not by value.
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - bool Matches(CatalogVendorApp app) should perhaps take the thing to match by reference.
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - The CreateBindingWithCallback API should not exist and certainly should not be using the old CHIPClusters command API we are trying to remove.
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - Same thing about CreateBindingWithCallback.
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment)
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - kCatalogVendorId and various vendor id use in examples/tv-app should probably use chip::VendorId.
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - Same thing about VendorId.
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - ContentLauncherManager::mSupportedStreamingProtocols should be declared to be of type std::underlying_type_t<relevant-spec-enum> instead of hardcoding uint32_t.
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - Can we build ContentApp.cpp and ContentAppPlatform.cpp from somewhere (e.g. src/app/BUILD.gn) where they will be compiled with -Wconversion? Though it's possible this is not relevant and they are being built that way, at least for the Linux tv-app. Should check!
• Link attribute reads to Content Apps, refactor content app platform to leverage latest cluster lib conventions #12969 (comment) - Use of ApplicationLauncherApplication (from af-structs.h) instead of the cluster-objects struct for that type, which necessitates back-and-forth conversion code.

[woody-apple]

These all look addressed, please re-open if not!