Fix the browser match rules.

chrome/test/base/test_browser_window.cc

 // 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.
 
 #include "chrome/test/base/test_browser_window.h"
 
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_list_observer.h"
 #include "ui/gfx/geometry/rect.h"
 
-
 // Helpers --------------------------------------------------------------------
 
 namespace chrome {
 
 namespace {
 
 // Handles destroying a TestBrowserWindow when the Browser it is attached to is
 // destroyed.
 class TestBrowserWindowOwner : public chrome::BrowserListObserver {
  public:
@@ skipping 16 matching lines @@
 
 }  // namespace
 
 Browser* CreateBrowserWithTestWindowForParams(Browser::CreateParams* params) {
   TestBrowserWindow* window = new TestBrowserWindow;
   new TestBrowserWindowOwner(window);
   params->window = window;
   return new Browser(*params);
 }
 
+#if defined(USE_AURA)
+Browser* CreateBrowserWithAuraTestWindowForParams(
+    const Browser::CreateParams& params) {
+  scoped_ptr<aura::Window> window(new aura::Window(nullptr));
+  window->set_id(0);
+  window->SetType(ui::wm::WINDOW_TYPE_NORMAL);
+  window->Init(ui::LAYER_TEXTURED);
+  window->Show();
+
+  TestBrowserWindowAura* browser_window =

[msw, 2015/07/08 23:58:00]

nit: if we keep this function, make it just call
CreateBrowserForAuraTestWindowAndParams here (instead of duplicating this code).

+      new TestBrowserWindowAura(window.Pass());
+  new TestBrowserWindowOwner(browser_window);
+  return browser_window->CreateBrowser(params);
+}
+
+Browser* CreateBrowserForAuraTestWindowAndParams(
+    scoped_ptr<aura::Window> window,
+    const Browser::CreateParams& params) {

[msw, 2015/07/08 23:58:00]

Should these functions take a params pointer, like
CreateBrowserWithTestWindowForParams?

+  TestBrowserWindowAura* browser_window =
+      new TestBrowserWindowAura(window.Pass());
+  new TestBrowserWindowOwner(browser_window);
+  return browser_window->CreateBrowser(params);
+}
+
+#endif  // defined(USE_AURA)
+
 }  // namespace chrome
 
 
 // TestBrowserWindow::TestLocationBar -----------------------------------------
 
 GURL TestBrowserWindow::TestLocationBar::GetDestinationURL() const {
   return GURL();
 }
 
 WindowOpenDisposition
@@ skipping 166 matching lines @@
   return 0;
 }
 
 void TestBrowserWindow::ExecuteExtensionCommand(
     const extensions::Extension* extension,
     const extensions::Command& command) {}
 
 ExclusiveAccessContext* TestBrowserWindow::GetExclusiveAccessContext() {
   return nullptr;
 }
+
+#if defined(USE_AURA)
+
+// TestBrowserWindowAura -------------------------------------------------------
+
+TestBrowserWindowAura::TestBrowserWindowAura(
+    scoped_ptr<aura::Window> native_window)
+    : native_window_(native_window.Pass()) {
+}
+
+TestBrowserWindowAura::~TestBrowserWindowAura() {
+}
+
+gfx::NativeWindow TestBrowserWindowAura::GetNativeWindow() const {
+  return native_window_.get();
+}
+
+void TestBrowserWindowAura::Show() {
+  native_window_->Show();
+}
+
+void TestBrowserWindowAura::Hide() {
+  native_window_->Hide();
+}
+
+gfx::Rect TestBrowserWindowAura::GetBounds() const {
+  return native_window_->bounds();
+}
+
+Browser* TestBrowserWindowAura::CreateBrowser(
+    const Browser::CreateParams& params) {
+  Browser::CreateParams create_params = params;
+  create_params.window = this;
+  browser_ = new Browser(create_params);
+  return browser_;
+}
+
+#endif