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

Unified Diff: chrome/browser/metrics/desktop_engagement/audible_contents_tracker_browsertest.cc

Issue 2333113002: Rename DesktopEngagement* to DesktopSessionDuration*. (Closed)
Patch Set: Change name Created 4 years, 3 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/metrics/desktop_engagement/audible_contents_tracker_browsertest.cc
diff --git a/chrome/browser/metrics/desktop_engagement/audible_contents_tracker_browsertest.cc b/chrome/browser/metrics/desktop_engagement/audible_contents_tracker_browsertest.cc
deleted file mode 100644
index d5e92d01a6b8f26c19998b77e5aeed6049626a12..0000000000000000000000000000000000000000
--- a/chrome/browser/metrics/desktop_engagement/audible_contents_tracker_browsertest.cc
+++ /dev/null
@@ -1,82 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/metrics/desktop_engagement/audible_contents_tracker.h"
-
-#include "base/path_service.h"
-#include "chrome/test/base/in_process_browser_test.h"
-#include "chrome/test/base/ui_test_utils.h"
-#include "content/public/test/browser_test_base.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace {
-
-// Observer for testing AudibleContentsTracker.
-class MockAudibleContentsObserver
- : public metrics::AudibleContentsTracker::Observer {
- public:
- MockAudibleContentsObserver() {}
-
- // AudibleContentsTracker::Observer:
- void OnAudioStart() override { is_audio_playing_ = true; }
- void OnAudioEnd() override { is_audio_playing_ = false; }
-
- bool is_audio_playing() const { return is_audio_playing_; }
-
- private:
- bool is_audio_playing_ = false;
-
- DISALLOW_COPY_AND_ASSIGN(MockAudibleContentsObserver);
-};
-
-} // namespace
-
-class AudibleContentsTrackerTest : public InProcessBrowserTest {
- public:
- AudibleContentsTrackerTest() {}
-
- void SetUp() override {
- observer_.reset(new MockAudibleContentsObserver());
- tracker_.reset(new metrics::AudibleContentsTracker(observer()));
- InProcessBrowserTest::SetUp();
- }
-
- void TearDown() override {
- InProcessBrowserTest::TearDown();
- tracker_.reset();
- observer_.reset();
- }
-
- MockAudibleContentsObserver* observer() const { return observer_.get(); }
-
- private:
- std::unique_ptr<MockAudibleContentsObserver> observer_ = nullptr;
- std::unique_ptr<metrics::AudibleContentsTracker> tracker_ = nullptr;
-
- DISALLOW_COPY_AND_ASSIGN(AudibleContentsTrackerTest);
-};
-
-IN_PROC_BROWSER_TEST_F(AudibleContentsTrackerTest, TestAudioNotifications) {
- MockAudibleContentsObserver* audio_observer = observer();
- EXPECT_FALSE(audio_observer->is_audio_playing());
-
- // For serving audio.
- ASSERT_TRUE(embedded_test_server()->Start());
- base::FilePath test_data_dir;
- ASSERT_TRUE(PathService::Get(base::DIR_SOURCE_ROOT, &test_data_dir));
- embedded_test_server()->ServeFilesFromDirectory(
- test_data_dir.AppendASCII("chrome/test/data/"));
- ui_test_utils::NavigateToURL(
- browser(), embedded_test_server()->GetURL("/autoplay_audio.html"));
-
- // Wait until the audio starts.
- while (!audio_observer->is_audio_playing()) {
- base::RunLoop().RunUntilIdle();
- }
-
- // Wait until the audio stops.
- while (audio_observer->is_audio_playing()) {
- base::RunLoop().RunUntilIdle();
- }
-}

Powered by Google App Engine
This is Rietveld 408576698