Revert "Migrate SensorProviderProxy to use GC mojo wrappers."

This reverts commit a42ef137.

Reason for revert: Surprising impact to rendering.mobile/queueing_durations

https://bugs.chromium.org/p/chromium/issues/detail?id=1067937

Original change's description:
> Migrate SensorProviderProxy to use GC mojo wrappers.
> 
> No behavior change. This CL reduces potential risks of use-after-free bugs.
> 
> Bug: 1049056
> Change-Id: Ie9d5865ff5be6203eadb71687a37b501cea51540
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109576
> Reviewed-by: Kentaro Hara <haraken@chromium.org>
> Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
> Commit-Queue: Minoru Chikamune <chikamune@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#755272}

TBR=haraken@chromium.org,kouhei@chromium.org,johannkoenig@google.com,talp@chromium.org,chikamune@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1049056
Change-Id: I2cf441cde80a0e1246423879d519ccc14373e20a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2143011Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Minoru Chikamune <chikamune@chromium.org>
Cr-Commit-Position: refs/heads/master@{#757682}

third_party/blink/renderer/modules/sensor/sensor_provider_proxy.cc

@@ -13,17 +13,14 @@ namespace blink {
 
 // SensorProviderProxy
 SensorProviderProxy::SensorProviderProxy(Document& document)
-    : Supplement<Document>(document),
-      sensor_provider_(document.ToExecutionContext()),
-      inspector_mode_(false) {}
+    : Supplement<Document>(document), inspector_mode_(false) {}
 
 void SensorProviderProxy::InitializeIfNeeded() {
   if (IsInitialized())
     return;
 
   GetSupplementable()->GetBrowserInterfaceBroker().GetInterface(
-      sensor_provider_.BindNewPipeAndPassReceiver(
-          GetSupplementable()->GetTaskRunner(TaskType::kSensor)));
+      sensor_provider_.BindNewPipeAndPassReceiver());
   sensor_provider_.set_disconnect_handler(
       WTF::Bind(&SensorProviderProxy::OnSensorProviderConnectionError,
                 WrapWeakPersistent(this)));
@@ -49,7 +46,6 @@ SensorProviderProxy::~SensorProviderProxy() = default;
 
 void SensorProviderProxy::Trace(Visitor* visitor) {
   visitor->Trace(sensor_proxies_);
-  visitor->Trace(sensor_provider_);
   Supplement<Document>::Trace(visitor);
 }
 

third_party/blink/renderer/modules/sensor/sensor_provider_proxy.h

@@ -6,13 +6,12 @@
 #define THIRD_PARTY_BLINK_RENDERER_MODULES_SENSOR_SENSOR_PROVIDER_PROXY_H_
 
 #include "base/macros.h"
+#include "mojo/public/cpp/bindings/remote.h"
 #include "services/device/public/mojom/sensor.mojom-blink-forward.h"
 #include "services/device/public/mojom/sensor_provider.mojom-blink.h"
 #include "third_party/blink/renderer/core/dom/document.h"
 #include "third_party/blink/renderer/modules/modules_export.h"
 #include "third_party/blink/renderer/platform/heap/handle.h"
-#include "third_party/blink/renderer/platform/mojo/heap_mojo_remote.h"
-#include "third_party/blink/renderer/platform/mojo/heap_mojo_wrapper_mode.h"
 #include "third_party/blink/renderer/platform/supplementable.h"
 
 namespace blink {
@@ -59,9 +58,7 @@ class MODULES_EXPORT SensorProviderProxy final
   void OnSensorProviderConnectionError();
   SensorsSet sensor_proxies_;
 
-  HeapMojoRemote<device::mojom::blink::SensorProvider,
-                 HeapMojoWrapperMode::kWithoutContextObserver>
-      sensor_provider_;
+  mojo::Remote<device::mojom::blink::SensorProvider> sensor_provider_;
   bool inspector_mode_;
 
   DISALLOW_COPY_AND_ASSIGN(SensorProviderProxy);