commit c47e277ca2dbf13b4bb5e402a7330c29805c04ac
author Oksana Zhuravlova <oksamyt@chromium.org> Thu Jul 19 05:44:47 2018
committer Commit Bot <commit-bot@chromium.org> Thu Jul 19 05:44:47 2018
tree eb47975c2e34204d3f3457ce3bdaa4c3ea4428cf
parent f90c1bcee4808cb99d8371d097c51b210d7eb79e

Remove platform_event_observers_ from RendererBlinkPlatformImpl

Since platform_event_observers_ is unused, this change removes it and
related code.

Bug: 855308
Change-Id: I717d8d3676dd4788e2dc27cc2f8d757fac591648
Reviewed-on: https://chromium-review.googlesource.com/1137151
Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576384}

content/renderer/renderer_blink_platform_impl.cc

diff --git a/content/renderer/renderer_blink_platform_impl.cc b/content/renderer/renderer_blink_platform_impl.cc
index 4030cd3b..b77c77c 100644
--- a/content/renderer/renderer_blink_platform_impl.cc
+++ b/content/renderer/renderer_blink_platform_impl.cc
@@ -41,7 +41,6 @@
 #include "content/public/common/webplugininfo.h"
 #include "content/public/renderer/content_renderer_client.h"
 #include "content/public/renderer/media_stream_utils.h"
-#include "content/public/renderer/platform_event_observer.h"
 #include "content/public/renderer/render_frame.h"
 #include "content/renderer/blob_storage/webblobregistry_impl.h"
 #include "content/renderer/dom_storage/local_storage_cached_areas.h"
@@ -1051,14 +1050,6 @@
   GetContentClient()->renderer()->RecordRapporURL(metric, url);
 }
 
-void RendererBlinkPlatformImpl::SetPlatformEventObserverForTesting(
-    blink::WebPlatformEventType type,
-    std::unique_ptr<PlatformEventObserverBase> observer) {
-  if (platform_event_observers_.Lookup(type))
-    platform_event_observers_.Remove(type);
-  platform_event_observers_.AddWithID(std::move(observer), type);
-}
-
 service_manager::Connector* RendererBlinkPlatformImpl::GetConnector() {
   return connector_.get();
 }

content/renderer/renderer_blink_platform_impl.h

diff --git a/content/renderer/renderer_blink_platform_impl.h b/content/renderer/renderer_blink_platform_impl.h
index 4c09b20..e3626b2 100644
--- a/content/renderer/renderer_blink_platform_impl.h
+++ b/content/renderer/renderer_blink_platform_impl.h
@@ -59,7 +59,6 @@
 class BlinkInterfaceProviderImpl;
 class ChildURLLoaderFactoryBundle;
 class LocalStorageCachedAreas;
-class PlatformEventObserverBase;
 class ThreadSafeSender;
 class WebDatabaseObserverImpl;
 
@@ -199,14 +198,6 @@
   void WillStopWorkerThread() override;
   void WorkerContextCreated(const v8::Local<v8::Context>& worker) override;
 
-  // Set the PlatformEventObserverBase in |platform_event_observers_| associated
-  // with |type| to |observer|. If there was already an observer associated to
-  // the given |type|, it will be replaced.
-  // Note that |observer| will be owned by this object after the call.
-  void SetPlatformEventObserverForTesting(
-      blink::WebPlatformEventType type,
-      std::unique_ptr<PlatformEventObserverBase> observer);
-
   // Disables the WebSandboxSupport implementation for testing.
   // Tests that do not set up a full sandbox environment should call
   // SetSandboxEnabledForTesting(false) _before_ creating any instances
@@ -257,11 +248,6 @@
 
   // TODO(crbug.com/850997): Remove when Device*EventPump classes are
   // moved to blink
-  void InitDeviceSensorEventPump(blink::WebPlatformEventType type,
-                                 blink::WebPlatformEventListener* listener);
-
-  // TODO(crbug.com/850997): Remove when Device*EventPump classes are
-  // moved to blink
   void StopDeviceSensorEventPump(blink::WebPlatformEventType type);
 
   // Ensure that the WebDatabaseHost has been initialized.
@@ -299,9 +285,6 @@
 
   std::unique_ptr<WebDatabaseObserverImpl> web_database_observer_impl_;
 
-  base::IDMap<std::unique_ptr<PlatformEventObserverBase>>
-      platform_event_observers_;
-
   // NOT OWNED
   blink::scheduler::WebThreadScheduler* main_thread_scheduler_;