Index: chrome/browser/ui/fullscreen/fullscreen_controller_state_unittest.cc |
=================================================================== |
--- chrome/browser/ui/fullscreen/fullscreen_controller_state_unittest.cc (revision 178499) |
+++ chrome/browser/ui/fullscreen/fullscreen_controller_state_unittest.cc (working copy) |
@@ -317,26 +317,6 @@ |
// Tests ----------------------------------------------------------------------- |
-// Soak tests: |
- |
-// Tests all states with all permutations of multiple events to detect lingering |
-// state issues that would bleed over to other states. |
-// I.E. for each state test all combinations of events E1, E2, E3. |
-// |
-// This produces coverage for event sequences that may happen normally but |
-// would not be exposed by traversing to each state via TransitionToState(). |
-// TransitionToState() always takes the same path even when multiple paths |
-// exist. |
-TEST_F(FullscreenControllerStateUnitTest, TransitionsForEachState) { |
- // A tab is needed for tab fullscreen. |
- AddTab(browser(), GURL(chrome::kAboutBlankURL)); |
- TestTransitionsForEachState(); |
- // Progress of test can be examined via LOG(INFO) << GetAndClearDebugLog(); |
-} |
- |
- |
-// Individual tests for each pair of state and event: |
- |
#define TEST_EVENT_INNER(state, event, reentrant, reentrant_id) \ |
TEST_F(FullscreenControllerStateUnitTest, \ |
state##__##event##reentrant_id) { \ |
@@ -351,6 +331,8 @@ |
TEST_EVENT_INNER(state, event, false, ); \ |
TEST_EVENT_INNER(state, event, true, _Reentrant); |
+// Individual tests for each pair of state and event: |
+ |
TEST_EVENT(STATE_NORMAL, TOGGLE_FULLSCREEN); |
TEST_EVENT(STATE_NORMAL, TAB_FULLSCREEN_TRUE); |
TEST_EVENT(STATE_NORMAL, TAB_FULLSCREEN_FALSE); |
@@ -447,7 +429,6 @@ |
TEST_EVENT(STATE_TO_TAB_FULLSCREEN, BUBBLE_DENY); |
TEST_EVENT(STATE_TO_TAB_FULLSCREEN, WINDOW_CHANGE); |
- |
// Specific one-off tests for known issues: |
// TODO(scheib) Toggling Tab fullscreen while pending Tab or |
@@ -481,3 +462,20 @@ |
#endif |
} |
+// Soak tests: |
+ |
+// Tests all states with all permutations of multiple events to detect lingering |
+// state issues that would bleed over to other states. |
+// I.E. for each state test all combinations of events E1, E2, E3. |
+// |
+// This produces coverage for event sequences that may happen normally but |
+// would not be exposed by traversing to each state via TransitionToState(). |
+// TransitionToState() always takes the same path even when multiple paths |
+// exist. |
+TEST_F(FullscreenControllerStateUnitTest, TransitionsForEachState) { |
+ // A tab is needed for tab fullscreen. |
+ AddTab(browser(), GURL(chrome::kAboutBlankURL)); |
+ TestTransitionsForEachState(); |
+ // Progress of test can be examined via LOG(INFO) << GetAndClearDebugLog(); |
+} |
+ |