Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(473)

Unified Diff: content/renderer/renderer_blink_platform_impl.cc

Issue 676953005: Revert of Replace Chrome IPC with Mojo IPC for querying BatteryStatus service (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: 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 a043d1bb5ac6af73421e039844b2ee0ea8df81a7..242a5950a3c6cca3b1b844a75efd806d3955dfcf 100644
--- a/content/renderer/renderer_blink_platform_impl.cc
+++ b/content/renderer/renderer_blink_platform_impl.cc
@@ -1045,34 +1045,39 @@
// hardware changes. In order to make that happen, they will receive a null
// thread.
if (thread && RenderThreadImpl::current()->layout_test_mode())
- thread = NULL;
+ thread = 0;
switch (type) {
- case blink::WebPlatformEventDeviceMotion:
- return new DeviceMotionEventPump(thread);
- case blink::WebPlatformEventDeviceOrientation:
- return new DeviceOrientationEventPump(thread);
- case blink::WebPlatformEventDeviceLight:
- return new DeviceLightEventPump(thread);
- case blink::WebPlatformEventGamepad:
- return new GamepadSharedMemoryReader(thread);
- case blink::WebPlatformEventScreenOrientation:
- return new ScreenOrientationObserver();
- default:
- // A default statement is required to prevent compilation errors when
- // Blink adds a new type.
- VLOG(1) << "RendererBlinkPlatformImpl::startListening() with "
- "unknown type.";
- }
-
- return NULL;
+ case blink::WebPlatformEventDeviceMotion: {
+ return new DeviceMotionEventPump(thread);
+ }
+ case blink::WebPlatformEventDeviceOrientation: {
+ return new DeviceOrientationEventPump(thread);
+ }
+ case blink::WebPlatformEventDeviceLight: {
+ return new DeviceLightEventPump(thread);
+ }
+ case blink::WebPlatformEventBattery: {
+ return new BatteryStatusDispatcher(thread);
+ }
+ case blink::WebPlatformEventGamepad:
+ return new GamepadSharedMemoryReader(thread);
+ break;
+ case blink::WebPlatformEventScreenOrientation:
+ return new ScreenOrientationObserver();
+ default:
+ // A default statement is required to prevent compilation errors when Blink
+ // adds a new type.
+ VLOG(1) << "RendererBlinkPlatformImpl::startListening() with "
+ "unknown type.";
+ }
+
+ return 0;
}
void RendererBlinkPlatformImpl::SetPlatformEventObserverForTesting(
blink::WebPlatformEventType type,
scoped_ptr<PlatformEventObserverBase> observer) {
- DCHECK(type != blink::WebPlatformEventBattery);
-
if (platform_event_observers_.Lookup(type))
platform_event_observers_.Remove(type);
platform_event_observers_.AddWithID(observer.release(), type);
@@ -1081,12 +1086,6 @@
void RendererBlinkPlatformImpl::startListening(
blink::WebPlatformEventType type,
blink::WebPlatformEventListener* listener) {
- if (type == blink::WebPlatformEventBattery) {
- battery_status_dispatcher_.reset(new BatteryStatusDispatcher(
- static_cast<blink::WebBatteryStatusListener*>(listener)));
- return;
- }
-
PlatformEventObserverBase* observer = platform_event_observers_.Lookup(type);
if (!observer) {
observer = CreatePlatformEventObserverFromType(type);
@@ -1145,11 +1144,6 @@
void RendererBlinkPlatformImpl::stopListening(
blink::WebPlatformEventType type) {
- if (type == blink::WebPlatformEventBattery) {
- battery_status_dispatcher_.reset();
- return;
- }
-
PlatformEventObserverBase* observer = platform_event_observers_.Lookup(type);
if (!observer)
return;
« no previous file with comments | « content/renderer/battery_status/battery_status_dispatcher.cc ('k') | content/renderer/shared_memory_seqlock_reader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698