Poll for Display rotation on Android 4.0 and 4.1 when needed.

content/renderer/renderer_webkitplatformsupport_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/renderer_webkitplatformsupport_impl.h"
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
@@ skipping 15 matching lines @@
 #include "content/child/web_database_observer_impl.h"
 #include "content/child/webblobregistry_impl.h"
 #include "content/child/webfileutilities_impl.h"
 #include "content/child/webmessageportchannel_impl.h"
 #include "content/common/file_utilities_messages.h"
 #include "content/common/gpu/client/context_provider_command_buffer.h"
 #include "content/common/gpu/client/gpu_channel_host.h"
 #include "content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.h"
 #include "content/common/gpu/gpu_process_launch_causes.h"
 #include "content/common/mime_registry_messages.h"
+#include "content/common/screen_orientation_messages.h"
 #include "content/common/view_messages.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/webplugininfo.h"
 #include "content/public/renderer/content_renderer_client.h"
 #include "content/renderer/battery_status/battery_status_dispatcher.h"
 #include "content/renderer/battery_status/fake_battery_status_dispatcher.h"
 #include "content/renderer/device_sensors/device_light_event_pump.h"
 #include "content/renderer/device_sensors/device_motion_event_pump.h"
 #include "content/renderer/device_sensors/device_orientation_event_pump.h"
 #include "content/renderer/dom_storage/webstoragenamespace_impl.h"
@@ skipping 1098 matching lines @@
   }
   battery_status_dispatcher_->SetListener(listener);
 }
 
 // static
 void RendererWebKitPlatformSupportImpl::MockBatteryStatusChangedForTesting(
     const blink::WebBatteryStatus& status) {
   g_test_battery_status_dispatcher.Get().PostBatteryStatusChange(status);
 }
 
+//------------------------------------------------------------------------------

[jam, 2014/07/16 21:29:14]

why?

[mlamouri (slow - plz ping), 2014/07/17 09:44:16]

This seems to be a convention in this file where there are sections separated
like that. Just following the local style.

[jam, 2014/07/17 17:19:51]

this file's convention is wrong. some files have "/-----..." between overridden
methods from different interfaces. but in this case, there are separaters
between overridden methods from the same interface which doesn't make sense.
please don't add a new one. if you're so inclined, you can remove the existing
ones.

also, order these new files to match the header, i.e. move them above the
previous method

+
+void RendererWebKitPlatformSupportImpl::startScreenOrientationListening() {
+  RenderThread::Get()->Send(new ScreenOrientationHostMsg_StartListening());
+}
+
+void RendererWebKitPlatformSupportImpl::stopScreenOrientationListening() {
+  RenderThread::Get()->Send(new ScreenOrientationHostMsg_StopListening());
+}
+
 }  // namespace content