Support screen.lockOrientation() / screen.unlockOrientation() when running layout tests

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/memory/shared_memory.h"
@@ skipping 127 matching lines @@
 namespace content {
 
 static bool g_sandbox_enabled = true;
 static blink::WebGamepadListener* web_gamepad_listener = NULL;
 base::LazyInstance<WebGamepads>::Leaky g_test_gamepads =
     LAZY_INSTANCE_INITIALIZER;
 base::LazyInstance<blink::WebDeviceMotionData>::Leaky
     g_test_device_motion_data = LAZY_INSTANCE_INITIALIZER;
 base::LazyInstance<blink::WebDeviceOrientationData>::Leaky
     g_test_device_orientation_data = LAZY_INSTANCE_INITIALIZER;
+static blink::WebScreenOrientationLockType g_test_current_screen_lock =

[jochen (gone - plz use gerrit), 2014/05/07 06:48:41]

i'd rather not have so many variables here. Is it possible to put the logic and
variables into a class?

+    blink::WebScreenOrientationLockDefault;
+static blink::WebScreenOrientationType g_test_device_screen_orientation =
+    blink::WebScreenOrientationPortraitPrimary;
+static blink::WebScreenOrientationType g_test_current_screen_orientation =
+    blink::WebScreenOrientationPortraitPrimary;
 static blink::WebScreenOrientationListener*
     g_test_screen_orientation_listener = NULL;
 
 //------------------------------------------------------------------------------
 
 class RendererWebKitPlatformSupportImpl::MimeRegistry
     : public SimpleWebMimeRegistryImpl {
  public:
   virtual blink::WebMimeRegistry::SupportsType supportsMediaMIMEType(
       const blink::WebString& mime_type,
@@ skipping 973 matching lines @@
   }
 
   if (!screen_orientation_dispatcher_) {
     screen_orientation_dispatcher_.reset(
         new ScreenOrientationDispatcher(RenderThread::Get()));
   }
 
   screen_orientation_dispatcher_->setListener(listener);
 }
 
+namespace {
+
+bool isScreenOrientationAllowedByTestingLock(

[jochen (gone - plz use gerrit), 2014/05/07 06:48:41]

methods should start with capital letters

[Inactive, 2014/05/07 18:41:05]

Done.

+    blink::WebScreenOrientationType orientation) {
+  if (g_test_current_screen_lock == blink::WebScreenOrientationLockDefault ||
+      g_test_current_screen_lock == blink::WebScreenOrientationLockAny)
+    return true;

[jochen (gone - plz use gerrit), 2014/05/07 06:48:41]

nit { } since the if part is multiline

[Inactive, 2014/05/07 18:41:05]

Done.

+
+  switch (orientation) {
+    case blink::WebScreenOrientationPortraitPrimary:
+      return g_test_current_screen_lock ==
+                 blink::WebScreenOrientationLockPortraitPrimary ||
+             g_test_current_screen_lock ==
+                 blink::WebScreenOrientationLockPortrait;
+    case blink::WebScreenOrientationPortraitSecondary:
+      return g_test_current_screen_lock ==
+                 blink::WebScreenOrientationLockPortraitSecondary ||
+             g_test_current_screen_lock ==
+                 blink::WebScreenOrientationLockPortrait;
+    case blink::WebScreenOrientationLandscapePrimary:
+      return g_test_current_screen_lock ==
+                 blink::WebScreenOrientationLockLandscapePrimary ||
+             g_test_current_screen_lock ==
+                 blink::WebScreenOrientationLockLandscape;
+    case blink::WebScreenOrientationLandscapeSecondary:
+      return g_test_current_screen_lock ==
+                 blink::WebScreenOrientationLockLandscapeSecondary ||
+             g_test_current_screen_lock ==
+                 blink::WebScreenOrientationLockLandscape;
+    default:
+      return false;
+  }
+}
+
+blink::WebScreenOrientationType suitableScreenOrientationForTestingLock() {
+  switch (g_test_current_screen_lock) {
+    case blink::WebScreenOrientationLockPortraitSecondary:
+      return blink::WebScreenOrientationPortraitSecondary;
+    case blink::WebScreenOrientationLockLandscapePrimary:
+    case blink::WebScreenOrientationLockLandscape:
+      return blink::WebScreenOrientationLandscapePrimary;
+    case blink::WebScreenOrientationLockLandscapeSecondary:
+      return blink::WebScreenOrientationLandscapePrimary;
+    default:
+      return blink::WebScreenOrientationPortraitPrimary;
+  }
+}
+
+void updateScreenOrientationForTesting(
+    blink::WebScreenOrientationType orientation) {
+  if (orientation == g_test_current_screen_orientation)
+    return;
+  g_test_current_screen_orientation = orientation;
+  if (g_test_screen_orientation_listener)
+    g_test_screen_orientation_listener->didChangeScreenOrientation(orientation);
+}
+
+} // namespace
+
 void RendererWebKitPlatformSupportImpl::lockOrientation(
     blink::WebScreenOrientationLockType orientation) {
   if (RenderThreadImpl::current() &&
       RenderThreadImpl::current()->layout_test_mode()) {
+    g_test_current_screen_lock = orientation;
+    if (!isScreenOrientationAllowedByTestingLock(
+            g_test_current_screen_orientation)) {
+      g_test_current_screen_orientation =
+          suitableScreenOrientationForTestingLock();
+      if (g_test_screen_orientation_listener) {
+        g_test_screen_orientation_listener->didChangeScreenOrientation(
+            g_test_current_screen_orientation);
+      }
+    }
     return;
   }
   RenderThread::Get()->Send(new ScreenOrientationHostMsg_Lock(orientation));
 }
 
 void RendererWebKitPlatformSupportImpl::unlockOrientation() {
   if (RenderThreadImpl::current() &&
       RenderThreadImpl::current()->layout_test_mode()) {
+    bool should_update_current_screen_orientation =
+        !isScreenOrientationAllowedByTestingLock(
+            g_test_device_screen_orientation);
+    g_test_current_screen_lock = blink::WebScreenOrientationLockDefault;
+    if (should_update_current_screen_orientation)
+      updateScreenOrientationForTesting(g_test_device_screen_orientation);
     return;
   }
   RenderThread::Get()->Send(new ScreenOrientationHostMsg_Unlock);
 }
 
 // static
 void RendererWebKitPlatformSupportImpl::SetMockScreenOrientationForTesting(
     blink::WebScreenOrientationType orientation) {
-  if (!g_test_screen_orientation_listener)
+  g_test_device_screen_orientation = orientation;
+  if (!isScreenOrientationAllowedByTestingLock(orientation))

[jochen (gone - plz use gerrit), 2014/05/07 06:48:41]

what happens if a test fails before it unlocks?

[Inactive, 2014/05/07 18:41:05]

Flakes :) I added a ResetMockScreenOrientationForTesting() method to address the
problem. I have verified that it works but I am not 100% sure the way I hooked
it up is the best way.

     return;
-  g_test_screen_orientation_listener->didChangeScreenOrientation(orientation);
+  updateScreenOrientationForTesting(g_test_device_screen_orientation);
 }
 
 //------------------------------------------------------------------------------
 
 void RendererWebKitPlatformSupportImpl::queryStorageUsageAndQuota(
     const blink::WebURL& storage_partition,
     blink::WebStorageQuotaType type,
     blink::WebStorageQuotaCallbacks callbacks) {
   if (!thread_safe_sender_.get() || !quota_message_filter_.get())
     return;
   QuotaDispatcher::ThreadSpecificInstance(
       thread_safe_sender_.get(),
       quota_message_filter_.get())->QueryStorageUsageAndQuota(
           storage_partition,
           static_cast<quota::StorageType>(type),
           QuotaDispatcher::CreateWebStorageQuotaCallbacksWrapper(callbacks));
 }
 
 }  // namespace content