| Index: chrome/browser/ui/exclusive_access/fullscreen_controller_state_unittest.cc
|
| diff --git a/chrome/browser/ui/exclusive_access/fullscreen_controller_state_unittest.cc b/chrome/browser/ui/exclusive_access/fullscreen_controller_state_unittest.cc
|
| index 9bd0d34ede37a9019e858f2abdc4fbb30c5a845a..21bd59b9c385eb44602372f5fc19758379cad1c4 100644
|
| --- a/chrome/browser/ui/exclusive_access/fullscreen_controller_state_unittest.cc
|
| +++ b/chrome/browser/ui/exclusive_access/fullscreen_controller_state_unittest.cc
|
| @@ -3,10 +3,12 @@
|
| // found in the LICENSE file.
|
|
|
| #include "base/command_line.h"
|
| +#include "base/test/histogram_tester.h"
|
| #include "build/build_config.h"
|
| #include "chrome/browser/ui/browser.h"
|
| #include "chrome/browser/ui/browser_tabstrip.h"
|
| #include "chrome/browser/ui/exclusive_access/exclusive_access_context.h"
|
| +#include "chrome/browser/ui/exclusive_access/exclusive_access_manager.h"
|
| #include "chrome/browser/ui/exclusive_access/fullscreen_controller.h"
|
| #include "chrome/browser/ui/exclusive_access/fullscreen_controller_state_test.h"
|
| #include "chrome/browser/ui/tabs/tab_strip_model.h"
|
| @@ -20,6 +22,10 @@
|
| // the FullscreenController through all permutations of events. The behavior
|
| // of the BrowserWindow is mocked via FullscreenControllerTestWindow.
|
|
|
| +namespace {
|
| +
|
| +const char kFullscreenReshowHistogramName[] =
|
| + "ExclusiveAccess.BubbleReshowsPerSession.Fullscreen";
|
|
|
| // FullscreenControllerTestWindow ----------------------------------------------
|
|
|
| @@ -246,6 +252,8 @@ void FullscreenControllerTestWindow::UpdateExclusiveAccessExitBubbleContent(
|
|
|
| void FullscreenControllerTestWindow::OnExclusiveAccessUserInput() {}
|
|
|
| +} // namespace
|
| +
|
| // FullscreenControllerStateUnitTest -------------------------------------------
|
|
|
| // Unit test fixture testing Fullscreen Controller through its states. Most of
|
| @@ -474,15 +482,25 @@ TEST_F(FullscreenControllerStateUnitTest,
|
|
|
| // Test that switching tabs takes the browser out of tab fullscreen.
|
| TEST_F(FullscreenControllerStateUnitTest, ExitTabFullscreenViaSwitchingTab) {
|
| + base::HistogramTester histogram_tester;
|
| +
|
| AddTab(browser(), GURL(url::kAboutBlankURL));
|
| AddTab(browser(), GURL(url::kAboutBlankURL));
|
| ASSERT_TRUE(InvokeEvent(TAB_FULLSCREEN_TRUE));
|
| ASSERT_TRUE(InvokeEvent(WINDOW_CHANGE));
|
| ASSERT_TRUE(browser()->window()->IsFullscreen());
|
| + histogram_tester.ExpectTotalCount(kFullscreenReshowHistogramName, 0);
|
|
|
| browser()->tab_strip_model()->SelectNextTab();
|
| ChangeWindowFullscreenState();
|
| EXPECT_FALSE(browser()->window()->IsFullscreen());
|
| +
|
| + // Do a simple test that histograms are being recorded upon exiting the
|
| + // fullscreen session (when simplified-fullscreen-ui is enabled).
|
| + if (ExclusiveAccessManager::IsSimplifiedFullscreenUIEnabled())
|
| + histogram_tester.ExpectUniqueSample(kFullscreenReshowHistogramName, 0, 1);
|
| + else
|
| + histogram_tester.ExpectTotalCount(kFullscreenReshowHistogramName, 0);
|
| }
|
|
|
| // Test that switching tabs via detaching the active tab (which is in tab
|
|
|