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

Unified Diff: third_party/WebKit/Source/modules/media_controls/MediaControlsRotateToFullscreenDelegateTest.cpp

Issue 2904263002: [Media Controls] Tests for rotate-to-fullscreen meets orientation lock (Closed)
Patch Set: Nits Created 3 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
Index: third_party/WebKit/Source/modules/media_controls/MediaControlsRotateToFullscreenDelegateTest.cpp
diff --git a/third_party/WebKit/Source/modules/media_controls/MediaControlsRotateToFullscreenDelegateTest.cpp b/third_party/WebKit/Source/modules/media_controls/MediaControlsRotateToFullscreenDelegateTest.cpp
index 37d492f5a089d44fa53e3409f79d47203d52d5d3..2467c32f966381ef54206e2ab5b623d2c6cd2325 100644
--- a/third_party/WebKit/Source/modules/media_controls/MediaControlsRotateToFullscreenDelegateTest.cpp
+++ b/third_party/WebKit/Source/modules/media_controls/MediaControlsRotateToFullscreenDelegateTest.cpp
@@ -608,8 +608,8 @@ TEST_F(MediaControlsRotateToFullscreenDelegateTest,
EXPECT_TRUE(GetVideo().paused());
EXPECT_FALSE(ObservedVisibility());
- // Rotate screen to portrait. This relies on the screen orientation no longer
- // being locked by MediaControlsOrientationLockDelegate.
+ // Rotate screen to portrait. This relies on the screen orientation not being
+ // locked by MediaControlsOrientationLockDelegate (which has its own tests).
RotateTo(kWebScreenOrientationPortraitPrimary);
// Should exit fullscreen.
@@ -639,8 +639,8 @@ TEST_F(MediaControlsRotateToFullscreenDelegateTest,
EXPECT_TRUE(GetVideo().paused());
EXPECT_FALSE(ObservedVisibility());
- // Rotate screen to portrait. This relies on the screen orientation no longer
- // being locked by MediaControlsOrientationLockDelegate.
+ // Rotate screen to landscape. This relies on the screen orientation not being
+ // locked by MediaControlsOrientationLockDelegate (which has its own tests).
RotateTo(kWebScreenOrientationLandscapePrimary);
// Should exit fullscreen.

Powered by Google App Engine
This is Rietveld 408576698