Delete most usage of deprecated mojom syntax

Outside of Chrome OS mojoms there are still a few places using
deprecated interface type syntax. This fixes those.

Bug: 955171
Change-Id: I1cd2eb4ac99e8f47f0eaf76d0595cb050372c926
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2248163Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Jonathan Ross <jonross@chromium.org>
Commit-Queue: Ken Rockot <rockot@google.com>
Cr-Commit-Position: refs/heads/master@{#778974}

components/viz/service/frame_sinks/root_compositor_frame_sink_impl.cc

@@ -64,7 +64,7 @@ RootCompositorFrameSinkImpl::Create(
   bool hw_support_for_multiple_refresh_rates = false;
   bool wants_vsync_updates = false;
 
-  if (params->external_begin_frame_controller.is_pending()) {
+  if (params->external_begin_frame_controller) {
     auto owned_external_begin_frame_source_mojo =
         std::make_unique<ExternalBeginFrameSourceMojo>(
             frame_sink_manager,

content/test/data/lite_js_test.mojom

@@ -53,7 +53,8 @@ interface TestMessageTarget {
   Flatten(array<TestStruct> values) => (array<int32> values);
   FlattenUnions(array<TestUnion> unions) => (array<int32> x, array<int32> s);
 
-  RequestSubinterface(Subinterface& request, SubinterfaceClient client);
+  RequestSubinterface(pending_receiver<Subinterface> receiver,
+                      pending_remote<SubinterfaceClient> client);
 };
 
 interface Subinterface {

services/device/public/mojom/nfc_provider.mojom

@@ -10,7 +10,7 @@ interface NFCProvider {
   // Gets an NFC that is associated with |host_id|. |host_id|
   // is used to obtain the Activity that this NFC instance should use on
   // Android (see NFCDelegate.java).
-  GetNFCForHost(int32 host_id, NFC& nfc);
+  GetNFCForHost(int32 host_id, pending_receiver<NFC> receiver);
 
   // Suspends all pending NFC operations. Could be used when web page
   // visibility is lost.

services/network/public/mojom/network_context.mojom

@@ -669,10 +669,10 @@ struct URLLoaderFactoryParams {
   ClientSecurityState? client_security_state;
 
   // Used to report CORP violations caused by COEP.
-  CrossOriginEmbedderPolicyReporter? coep_reporter;
+  pending_remote<CrossOriginEmbedderPolicyReporter>? coep_reporter;
 
   // Used to notify clients about cookie reads or writes.
-  CookieAccessObserver? cookie_observer;
+  pending_remote<CookieAccessObserver>? cookie_observer;
 
   // If this equals kForbid, the context to which this loader is bound does not
   // allow any Trust Tokens (https://github.com/wicg/trust-token-api)

services/network/public/mojom/url_loader.mojom

@@ -90,7 +90,7 @@ struct TrustedUrlRequestParams {
   // Observer which should be notified when this URLRequest reads or writes
   // a cookie. If this is set to non-null, the observer passed to
   // URLLoaderFactory will be ignored.
-  CookieAccessObserver? cookie_observer;
+  pending_remote<CookieAccessObserver>? cookie_observer;
 };
 
 // Typemapped to network::ResourceRequest.

services/viz/privileged/mojom/compositing/frame_sink_manager.mojom

@@ -41,7 +41,8 @@ struct RootCompositorFrameSinkParams {
   pending_associated_receiver<DisplayPrivate> display_private;
   pending_remote<DisplayClient> display_client;
 
-  associated ExternalBeginFrameController&? external_begin_frame_controller;
+  pending_associated_receiver<ExternalBeginFrameController>?
+      external_begin_frame_controller;
 };
 
 // The FrameSinkManager interface is a privileged interface that allows the

third_party/blink/public/mojom/loader/url_loader_factory_bundle.mojom

@@ -20,18 +20,19 @@ struct URLLoaderFactoryBundle {
   //
   // TODO(jam): https://crbug.com/887109: Remove |default_factory| and put it
   // inside |scheme_specific_factories| instead.
-  network.mojom.URLLoaderFactory? default_factory;
+  pending_remote<network.mojom.URLLoaderFactory>? default_factory;
 
   // A mapping from URL scheme to factory interface.
-  map<string, network.mojom.URLLoaderFactory> scheme_specific_factories;
+  map<string, pending_remote<network.mojom.URLLoaderFactory>>
+      scheme_specific_factories;
 
   // A mapping from isolated world's origin to factory interface.
-  map<url.mojom.Origin, network.mojom.URLLoaderFactory>
+  map<url.mojom.Origin, pending_remote<network.mojom.URLLoaderFactory>>
       isolated_world_factories;
 
   // A special factory that is used for AppCache.
   // TODO(https://crbug.com/582750): Drop this when AppCache is deprecated.
-  network.mojom.URLLoaderFactory? appcache_factory;
+  pending_remote<network.mojom.URLLoaderFactory>? appcache_factory;
 
   // Whether redirect checks should be bypassed, since they are happening in the
   // browser.