Call RenderViewImpl::SetScreenOrientationForTesting to make sure that events are not sent when orie…

content/renderer/renderer_webkitplatformsupport_impl.cc

Index: content/renderer/renderer_webkitplatformsupport_impl.cc
diff --git a/content/renderer/renderer_webkitplatformsupport_impl.cc b/content/renderer/renderer_webkitplatformsupport_impl.cc
index 433cca128460c80fe3c84eeac8fa56c3714c8fa0..eba4a861dfd9f29c012a1cb7a3bfaf2fe333c01b 100644
--- a/content/renderer/renderer_webkitplatformsupport_impl.cc
+++ b/content/renderer/renderer_webkitplatformsupport_impl.cc
@@ -1080,6 +1080,7 @@ void RendererWebKitPlatformSupportImpl::SetMockDeviceMotionDataForTesting(
 // static
 void RendererWebKitPlatformSupportImpl::ResetMockScreenOrientationForTesting()
 {
+  assert(!(g_test_screen_orientation_controller == 0));

[jochen (gone - plz use gerrit), 2014/06/02 07:21:25]

DCHECK(g_test_screen_orientation_controller)

[ostap, 2014/06/03 00:11:54]

LazyInstance implements only == operator, so it has to be 

DCHECK(!(g_test_screen_orientation_controller == 0));

   g_test_screen_orientation_controller.Get().ResetData();

[jochen (gone - plz use gerrit), 2014/06/02 07:21:25]

.Get() will create an controller btw

[ostap, 2014/06/03 00:11:54]

Yes. ResetMockScreenOrientationForTesting() shouldn't be called without
SetMockDeviceMotionDataForTesting().
That's why there is DCHECK before.

 }
 
@@ -1164,9 +1165,10 @@ void RendererWebKitPlatformSupportImpl::unlockOrientation() {
 
 // static
 void RendererWebKitPlatformSupportImpl::SetMockScreenOrientationForTesting(
+    RenderView* render_view,
     blink::WebScreenOrientationType orientation) {
   g_test_screen_orientation_controller.Get()
-      .UpdateDeviceOrientation(orientation);
+      .UpdateDeviceOrientation(render_view, orientation);
 }
 
 //------------------------------------------------------------------------------