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

Unified Diff: chrome/browser/ui/fullscreen/fullscreen_controller_state_interactive_browsertest.cc

Issue 325443002: Move about://-related constants from //content to //url (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Android ifx Created 6 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: chrome/browser/ui/fullscreen/fullscreen_controller_state_interactive_browsertest.cc
diff --git a/chrome/browser/ui/fullscreen/fullscreen_controller_state_interactive_browsertest.cc b/chrome/browser/ui/fullscreen/fullscreen_controller_state_interactive_browsertest.cc
index 33bcea7a8a921b3411f36a6375f8f443ee5e98bf..01c7ac3029364765c91fdb0abb1b4620a6c3acb2 100644
--- a/chrome/browser/ui/fullscreen/fullscreen_controller_state_interactive_browsertest.cc
+++ b/chrome/browser/ui/fullscreen/fullscreen_controller_state_interactive_browsertest.cc
@@ -54,8 +54,7 @@ Browser* FullscreenControllerStateInteractiveTest::GetBrowser() {
IN_PROC_BROWSER_TEST_F(FullscreenControllerStateInteractiveTest,
DISABLED_TransitionsForEachState) {
// A tab is needed for tab fullscreen.
- AddTabAtIndex(0, GURL(content::kAboutBlankURL),
- content::PAGE_TRANSITION_TYPED);
+ AddTabAtIndex(0, GURL(url::kAboutBlankURL), content::PAGE_TRANSITION_TYPED);
TestTransitionsForEachState();
// Progress of test can be examined via LOG(INFO) << GetAndClearDebugLog();
}
@@ -66,17 +65,16 @@ IN_PROC_BROWSER_TEST_F(FullscreenControllerStateInteractiveTest,
// An "empty" test is included as part of each "TEST_EVENT" because it makes
// running the entire test suite less flaky on MacOS. All of the tests pass
// when run individually.
-#define TEST_EVENT(state, event) \
- IN_PROC_BROWSER_TEST_F(FullscreenControllerStateInteractiveTest, \
- DISABLED_##state##__##event##__Empty) { \
- } \
- IN_PROC_BROWSER_TEST_F(FullscreenControllerStateInteractiveTest, \
- DISABLED_##state##__##event) { \
- AddTabAtIndex(0, GURL(content::kAboutBlankURL), \
- content::PAGE_TRANSITION_TYPED); \
- ASSERT_NO_FATAL_FAILURE(TestStateAndEvent(state, event)) \
- << GetAndClearDebugLog(); \
- }
+#define TEST_EVENT(state, event) \
+ IN_PROC_BROWSER_TEST_F(FullscreenControllerStateInteractiveTest, \
+ DISABLED_##state##__##event##__Empty) {} \
+ IN_PROC_BROWSER_TEST_F(FullscreenControllerStateInteractiveTest, \
+ DISABLED_##state##__##event) { \
+ AddTabAtIndex( \
+ 0, GURL(url::kAboutBlankURL), content::PAGE_TRANSITION_TYPED); \
+ ASSERT_NO_FATAL_FAILURE(TestStateAndEvent(state, event)) \
+ << GetAndClearDebugLog(); \
+ }
// Progress of tests can be examined by inserting the following line:
// LOG(INFO) << GetAndClearDebugLog(); }
@@ -89,8 +87,7 @@ IN_PROC_BROWSER_TEST_F(FullscreenControllerStateInteractiveTest,
IN_PROC_BROWSER_TEST_F(FullscreenControllerStateInteractiveTest,
DISABLED_ManualTest) {
// A tab is needed for tab fullscreen.
- AddTabAtIndex(0, GURL(content::kAboutBlankURL),
- content::PAGE_TRANSITION_TYPED);
+ AddTabAtIndex(0, GURL(url::kAboutBlankURL), content::PAGE_TRANSITION_TYPED);
ASSERT_TRUE(InvokeEvent(TOGGLE_FULLSCREEN)) << GetAndClearDebugLog();
ASSERT_TRUE(InvokeEvent(WINDOW_CHANGE)) << GetAndClearDebugLog();
ASSERT_TRUE(InvokeEvent(TAB_FULLSCREEN_TRUE)) << GetAndClearDebugLog();

Powered by Google App Engine
This is Rietveld 408576698