Refactor BrowserActivationWaiter as test support and move PopupBlockerBrowserTest to interactive_ui…

chrome/test/base/ui_test_utils.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_TEST_BASE_UI_TEST_UTILS_H_
 #define CHROME_TEST_BASE_UI_TEST_UTILS_H_
 
 #include <map>
 #include <queue>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/strings/string16.h"
+#include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_list_observer.h"
 #include "components/history/core/browser/history_service.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/test/test_utils.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/base/window_open_disposition.h"
 #include "ui/events/keycodes/keyboard_codes.h"
 #include "ui/gfx/native_widget_types.h"
@@ skipping 253 matching lines @@
       const base::Closure& quit_task,
       history::QueryResults* results);
 
   std::vector<GURL> urls_;
 
   base::CancelableTaskTracker tracker_;
 
   DISALLOW_COPY_AND_ASSIGN(HistoryEnumerator);
 };
 
+// Use in browser interactive uitests to wait until a browser is set to active.
+// To use create and call WaitForActivation().
+// TODO(warx): check if code base exists the requirement for deactivation sync,
+// this class can be modified to support that.
+class BrowserActivationWaiter : public chrome::BrowserListObserver {
+ public:
+  explicit BrowserActivationWaiter(Browser* browser);
+  ~BrowserActivationWaiter() override;
+
+  // Returns when the |browser_| which is supplied by the constructor becomes
+  // activated. If it is not activated, a message loop is used until the
+  // |browser_| is activated, otherwise this returns immediately.
+  void WaitForActivation();
+
+ private:
+  // chrome::BrowserListObserver override:
+  void OnBrowserSetLastActive(Browser* browser) override;
+
+  Browser* browser_;
+  bool observed_;
+  scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
+
+  DISALLOW_COPY_AND_ASSIGN(BrowserActivationWaiter);
+};
+
 }  // namespace ui_test_utils
 
 #endif  // CHROME_TEST_BASE_UI_TEST_UTILS_H_