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

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: Fix MSVC warning Created 3 years, 6 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 0745e5ecd4c3c98ca12b92afb53bcf891b9ab379..814bc8dee90126083732949b52bab035c317d526 100644
--- a/third_party/WebKit/Source/modules/media_controls/MediaControlsRotateToFullscreenDelegateTest.cpp
+++ b/third_party/WebKit/Source/modules/media_controls/MediaControlsRotateToFullscreenDelegateTest.cpp
@@ -602,8 +602,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.
@@ -632,8 +632,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