Implement ScreenOrientationDelegate on ChromeOS

ash/system/chromeos/screen_orientation/screen_orientation_delegate_ash.cc

+// Copyright 2014 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 "ash/system/chromeos/screen_orientation/screen_orientation_delegate_ash.h"

[flackr, 2014/10/14 20:08:17]

This should maybe be in ash/display/screen_orientation_delegate_chromeos.{cc,h}.

ash/system is for things on the system tray I believe.

[jonross, 2014/10/22 18:45:30]

Done.

+
+#include "ash/display/display_info.h"
+#include "ash/display/display_manager.h"
+#include "ash/shell.h"
+#include "ash/wm/maximize_mode/maximize_mode_controller.h"
+#include "content/public/browser/screen_orientation_provider.h"
+#include "ui/gfx/display.h"
+#include "ui/gfx/geometry/size.h"
+
+namespace ash {
+
+ScreenOrientationDelegateAsh::ScreenOrientationDelegateAsh()
+    : natural_orientation_(blink::WebScreenOrientationLockLandscape) {
+  content::ScreenOrientationProvider::SetDelegate(this);
+
+  DisplayManager* display_manager = Shell::GetInstance()->display_manager();
+  if (display_manager->HasInternalDisplay()) {
+    DisplayInfo info =
+        display_manager->GetDisplayInfo(gfx::Display::InternalDisplayId());
+    gfx::Display::Rotation rotation = info.rotation();
+    gfx::Size size = info.size_in_pixel();
+    if (((rotation == gfx::Display::ROTATE_0 ||
+         rotation == gfx::Display::ROTATE_180) &&

[mlamouri (slow - plz ping), 2014/10/15 15:39:33]

nit: one space is missing to align this line

[jonross, 2014/10/22 18:45:30]

Done.

+        size.height() >= size.width()) ||
+        ((rotation == gfx::Display::ROTATE_90 ||
+          rotation == gfx::Display::ROTATE_270) &&
+         size.height() < size.width())) {
+      natural_orientation_ = blink::WebScreenOrientationLockPortrait;
+    }
+  }

[flackr, 2014/10/14 20:08:18]

Move this to a helper function (GetDisplayNaturalOrientation?) and call from
initializer list.

[mlamouri (slow - plz ping), 2014/10/15 15:39:33]

+1 with that, it would make this algorithm a bit more readable. 

[jonross, 2014/10/22 18:45:30]

Done.

+}
+
+ScreenOrientationDelegateAsh::~ScreenOrientationDelegateAsh() {
+  content::ScreenOrientationProvider::SetDelegate(NULL);
+}
+
+bool ScreenOrientationDelegateAsh::FullScreenRequired(
+    content::WebContents* web_contents) {
+  return true;
+}
+
+void ScreenOrientationDelegateAsh::Lock(
+    blink::WebScreenOrientationLockType lock_orientation) {
+  MaximizeModeController* controller =
+      Shell::GetInstance()->maximize_mode_controller();
+  switch (lock_orientation) {
+    case blink::WebScreenOrientationLockAny:
+    case blink::WebScreenOrientationLockDefault:
+      DCHECK(!controller->rotation_locked());

[flackr, 2014/10/14 20:08:17]

Won't this be true if the user manually locked a rotation?

[mlamouri (slow - plz ping), 2014/10/15 15:39:33]

LockDefault is not expected to be called. You should DCHECK() if you end up with
that lock_orientation value.

Regarding 'any', I think you should explicitely unlock if that's ChromeOS
behaviour.

[jonross, 2014/10/22 18:45:30]

Since it should not be called I'll use NOTREACHED, as a user can manually lock
the rotation.

Chaning Any to unlock, which is the default behaviour on ChromeOS.

+      break;
+    case blink::WebScreenOrientationLockPortraitPrimary:
+      LockRotationToPrimaryOrientation(blink::WebScreenOrientationLockPortrait);
+      break;
+    case blink::WebScreenOrientationLockPortrait:
+      LockToRotationMatchingOrientation(lock_orientation);
+      break;
+    case blink::WebScreenOrientationLockPortraitSecondary:
+      LockRotationToSecondaryOrientation(
+          blink::WebScreenOrientationLockPortrait);
+      break;
+    case blink::WebScreenOrientationLockLandscapeSecondary:
+      LockRotationToSecondaryOrientation(
+          blink::WebScreenOrientationLockLandscape);
+      break;
+    case blink::WebScreenOrientationLockLandscapePrimary:
+      LockRotationToPrimaryOrientation(
+          blink::WebScreenOrientationLockLandscape);
+      break;
+    case blink::WebScreenOrientationLockLandscape:
+      LockToRotationMatchingOrientation(lock_orientation);
+      break;
+    case blink::WebScreenOrientationLockNatural:
+    default:

[flackr, 2014/10/14 20:08:18]

Should probably handle all expected WebScreenOrientationLockType's and make
default a NOTREACHED() and/or not have default (if the compile time check
works).

[mlamouri (slow - plz ping), 2014/10/15 15:39:33]

+1

[jonross, 2014/10/22 18:45:30]

Done.

+      controller->LockRotation(gfx::Display::ROTATE_0);
+      break;
+  }
+}
+
+bool ScreenOrientationDelegateAsh::ScreenOrientationProviderSupported() {
+  return Shell::GetInstance()->maximize_mode_controller()->
+      IsMaximizeModeWindowManagerEnabled();
+}
+
+void ScreenOrientationDelegateAsh::Unlock() {
+  Shell::GetInstance()->maximize_mode_controller()->SetRotationLocked(false);

[flackr, 2014/10/14 20:08:17]

How does this interact with a user-locked orientation?

+}
+
+void ScreenOrientationDelegateAsh::LockRotationToPrimaryOrientation(
+          blink::WebScreenOrientationLockType lock_orientation) {
+  Shell::GetInstance()->maximize_mode_controller()->
+      LockRotation(natural_orientation_ == lock_orientation ?
+          gfx::Display::ROTATE_0 :
+          gfx::Display::ROTATE_90);
+}
+
+void ScreenOrientationDelegateAsh::LockRotationToSecondaryOrientation(
+          blink::WebScreenOrientationLockType lock_orientation) {
+  Shell::GetInstance()->maximize_mode_controller()->
+      LockRotation(natural_orientation_ == lock_orientation ?
+          gfx::Display::ROTATE_180 :
+          gfx::Display::ROTATE_270);
+}
+
+void ScreenOrientationDelegateAsh::LockToRotationMatchingOrientation(
+    blink::WebScreenOrientationLockType lock_orientation) {
+  //TODO(jonross): Update MaximizeModeController to allow rotation between
+  // two angles of an orientation.

[flackr, 2014/10/14 20:08:17]

I'm a little unclear on what you mean here? Can you add an example? Is this e.g.
from ROTATE_0 to ROTATE_180?

[jonross, 2014/10/22 18:45:30]

Correct, it would be to allow the web page to lock the device to either
portrait, or either landscape. So that 0/180 and 90/270 were allowed, while
other orientations are not.

Updated the note.

+  DisplayManager* display_manager = Shell::GetInstance()->display_manager();
+  if (!display_manager->HasInternalDisplay())
+    return;

[mlamouri (slow - plz ping), 2014/10/15 15:39:33]

I'm a bit confused by that. Is it expected? I mean, is that something that _can_
happen? I feel like we should whether fallback to primary or DCHECK but doing
nothing is unlikely a good idea.

[jonross, 2014/10/22 18:45:30]

This is a valid state for unit tests on ash. Since accelerometer rotation is
only supported on internal displays, we opted to exit early on this condition
rather than fail. Otherwise a large number of unrelated tests would have had to
have been updated.

+
+  gfx::Display::Rotation rotation = display_manager->
+    GetDisplayInfo(gfx::Display::InternalDisplayId()).rotation();
+  MaximizeModeController* controller =
+      Shell::GetInstance()->maximize_mode_controller();
+  if (natural_orientation_ == lock_orientation) {
+    if (rotation == gfx::Display::ROTATE_0 ||
+        rotation == gfx::Display::ROTATE_180){
+      controller->SetRotationLocked(true);
+    } else {
+      controller->LockRotation(gfx::Display::ROTATE_0);
+    }
+  } else {
+    if (rotation == gfx::Display::ROTATE_90 ||
+        rotation == gfx::Display::ROTATE_270) {
+      controller->SetRotationLocked(true);
+    } else {
+      controller->LockRotation(gfx::Display::ROTATE_90);
+    }
+  }
+}
+
+}  // namespace ash