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

Unified Diff: trunk/src/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc

Issue 271133002: Revert 268794 "Lock Rotation when screen is manually rotated" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 7 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
« no previous file with comments | « trunk/src/ash/wm/maximize_mode/maximize_mode_controller.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc
===================================================================
--- trunk/src/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc (revision 269259)
+++ trunk/src/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc (working copy)
@@ -148,7 +148,7 @@
// Tests that when the hinge is nearly vertically aligned, the current state
// persists as the computed angle is highly inaccurate in this orientation.
TEST_F(MaximizeModeControllerTest, HingeAligned) {
- // Laptop in normal orientation lid open 90 degrees.
+ // Laptop in normal orientation lid open 90 degrees.
TriggerAccelerometerUpdate(gfx::Vector3dF(0.0f, 0.0f, 1.0f),
gfx::Vector3dF(-1.0f, 0.0f, 0.0f));
EXPECT_FALSE(IsMaximizeModeStarted());
@@ -309,7 +309,7 @@
EXPECT_EQ(0u, counter.event_count());
counter.reset();
- // Touch should not be blocked.
+ // Touch should not be blocked.
event_generator.PressTouch();
event_generator.ReleaseTouch();
EXPECT_GT(counter.event_count(), 0u);
@@ -331,7 +331,7 @@
// Feeds in sample accelerometer data and verifies that there are no
// transitions into touchview / maximize mode while shaking the device around
// with the hinge at less than 180 degrees.
- ASSERT_EQ(0u, kAccelerometerLaptopModeTestDataLength % 6);
+ ASSERT_TRUE(kAccelerometerLaptopModeTestDataLength % 6 == 0);
for (size_t i = 0; i < kAccelerometerLaptopModeTestDataLength / 6; ++i) {
gfx::Vector3dF base(kAccelerometerLaptopModeTestData[i * 6],
kAccelerometerLaptopModeTestData[i * 6 + 1],
@@ -355,7 +355,7 @@
// Feeds in sample accelerometer data and verifies that there are no
// transitions out of touchview / maximize mode while shaking the device
// around.
- ASSERT_EQ(0u, kAccelerometerFullyOpenTestDataLength % 6);
+ ASSERT_TRUE(kAccelerometerFullyOpenTestDataLength % 6 == 0);
for (size_t i = 0; i < kAccelerometerFullyOpenTestDataLength / 6; ++i) {
gfx::Vector3dF base(kAccelerometerFullyOpenTestData[i * 6],
kAccelerometerFullyOpenTestData[i * 6 + 1],
@@ -394,27 +394,26 @@
EXPECT_EQ(gfx::Display::ROTATE_90, GetInternalDisplayRotation());
}
-// Tests that if the display was rotated before entering maximize mode that
-// rotation becomes locked.
-TEST_F(MaximizeModeControllerTest,
- RotatedDisplayLocksRotationUponEnteringMaximizeMode) {
- DisplayManager* display_manager = Shell::GetInstance()->display_manager();
- display_manager->SetDisplayRotation(gfx::Display::InternalDisplayId(),
- gfx::Display::ROTATE_90);
+// Tests that when MaximizeModeController turns off MaximizeMode that on the
+// next accelerometer update the rotation lock is cleared.
+TEST_F(MaximizeModeControllerTest, ExitingMaximizeModeClearRotationLock) {
+ // The base remains steady.
+ gfx::Vector3dF base(0.0f, 0.0f, 1.0f);
- // Trigger a non-maximize mode accelerometer update. Ensure that the display
- // does not rotate.
- TriggerAccelerometerUpdate(gfx::Vector3dF(0.0f, 0.0f, 1.0f),
- gfx::Vector3dF(-1.0f, 0.0f, 0.0f));
- EXPECT_FALSE(IsMaximizeModeStarted());
- EXPECT_EQ(gfx::Display::ROTATE_90, GetInternalDisplayRotation());
-
// Trigger maximize mode by opening to 270.
TriggerAccelerometerUpdate(gfx::Vector3dF(0.0f, 0.0f, -1.0f),
gfx::Vector3dF(-1.0f, 0.0f, 0.0f));
ASSERT_TRUE(IsMaximizeModeStarted());
- EXPECT_TRUE(maximize_mode_controller()->rotation_locked());
- EXPECT_EQ(gfx::Display::ROTATE_90, GetInternalDisplayRotation());
+
+ maximize_mode_controller()->set_rotation_locked(true);
+
+ // Open 90 degrees.
+ TriggerAccelerometerUpdate(base, gfx::Vector3dF(-1.0f, 0.0f, 0.0f));
+ EXPECT_FALSE(IsMaximizeModeStarted());
+
+ // Send an update that would not relaunch MaximizeMode. 90 degrees.
+ TriggerAccelerometerUpdate(base, gfx::Vector3dF(-1.0f, 0.0f, 0.0f));
+ EXPECT_FALSE(maximize_mode_controller()->rotation_locked());
}
} // namespace ash
« no previous file with comments | « trunk/src/ash/wm/maximize_mode/maximize_mode_controller.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698