Fix duplicate logging of ActiveBrowserChanged action on views.

chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer_browsertest.cc

Index: chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer_browsertest.cc
diff --git a/chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer_browsertest.cc b/chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer_browsertest.cc
index 6bbbd3061e0ff77d294609cf01c3123564e4dd4d..2993c76c59100ea35e36fc90a10c1d5a87d08b54 100644
--- a/chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer_browsertest.cc
+++ b/chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer_browsertest.cc
@@ -4,51 +4,27 @@
 
 #include "chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer.h"
 
-#include "base/memory/singleton.h"
 #include "base/run_loop.h"
-#include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_window.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
-#include "content/public/browser/notification_service.h"
 #include "content/public/test/test_utils.h"
 
 // Mock class for |ChromeVisibilityObserver| for testing.
 class MockChromeVisibilityObserver : public metrics::ChromeVisibilityObserver {
  public:
-  MockChromeVisibilityObserver() : is_active_(false) {
-    ResetRunLoop();
-  }
-
-  void Wait() {
-    run_loop_->Run();
-  }
-
-  void ResetRunLoop() {
-    run_loop_.reset(new base::RunLoop());
-  }
+  MockChromeVisibilityObserver() : is_active_(false) {}
 
   bool is_active() const { return is_active_; }
-  void set_is_active(bool is_active) { is_active_ = is_active; }
 
  private:
   void SendVisibilityChangeEvent(bool active,
                                  base::TimeDelta time_ago) override {
     is_active_ = active;
-    // In this test browser became inactive after browser is removed, thus
-    // in this case quit runloop in OnBrowserRemoved().
-    if (is_active_)
-      run_loop_->Quit();
-  }
-
-  void OnBrowserRemoved(Browser* browser) override {
-    metrics::ChromeVisibilityObserver::OnBrowserRemoved(browser);
-    run_loop_->Quit();
   }
 
   bool is_active_;
-  std::unique_ptr<base::RunLoop> run_loop_;
 
   DISALLOW_COPY_AND_ASSIGN(MockChromeVisibilityObserver);
 };
@@ -56,31 +32,30 @@ class MockChromeVisibilityObserver : public metrics::ChromeVisibilityObserver {
 class ChromeVisibilityObserverBrowserTest : public InProcessBrowserTest {};
 
 IN_PROC_BROWSER_TEST_F(ChromeVisibilityObserverBrowserTest, VisibilityTest) {
+  // Deactivate the initial browser window, to make observer start inactive.
+  browser()->window()->Deactivate();
+  base::RunLoop().RunUntilIdle();
+
   MockChromeVisibilityObserver observer;
 
   EXPECT_FALSE(observer.is_active());
   Browser* new_browser = CreateBrowser(browser()->profile());
-  observer.Wait();
+  base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(observer.is_active());
 
-  observer.ResetRunLoop();
-  observer.set_is_active(false);
   Browser* incognito_browser = CreateIncognitoBrowser();
-  observer.Wait();
+  base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(observer.is_active());
 
-  observer.ResetRunLoop();
   CloseBrowserSynchronously(incognito_browser);
-  observer.Wait();
+  base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(observer.is_active());
 
-  observer.ResetRunLoop();
   CloseBrowserSynchronously(new_browser);
-  observer.Wait();
+  base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(observer.is_active());
 
-  observer.ResetRunLoop();
   CloseBrowserSynchronously(browser());
-  observer.Wait();
+  base::RunLoop().RunUntilIdle();
   EXPECT_FALSE(observer.is_active());
 }