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

Unified Diff: chrome/test/base/test_browser_window.cc

Issue 1198313003: Fix the browser match rules. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use scoped_ptr<Browser> instead. Created 5 years, 5 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/test/base/test_browser_window.cc
diff --git a/chrome/test/base/test_browser_window.cc b/chrome/test/base/test_browser_window.cc
index 8bfc54c0b946c7117030471a426adce64233eb70..d9140ccb3d4bc926ed4c51bdc79149b1557598f3 100644
--- a/chrome/test/base/test_browser_window.cc
+++ b/chrome/test/base/test_browser_window.cc
@@ -8,7 +8,6 @@
#include "chrome/browser/ui/browser_list_observer.h"
#include "ui/gfx/geometry/rect.h"
-
// Helpers --------------------------------------------------------------------
namespace chrome {
@@ -38,12 +37,32 @@ class TestBrowserWindowOwner : public chrome::BrowserListObserver {
} // namespace
-Browser* CreateBrowserWithTestWindowForParams(Browser::CreateParams* params) {
+scoped_ptr<Browser> CreateBrowserWithTestWindowForParams(
+ Browser::CreateParams* params) {
TestBrowserWindow* window = new TestBrowserWindow;
new TestBrowserWindowOwner(window);
params->window = window;
- return new Browser(*params);
+ return make_scoped_ptr(new Browser(*params));
+}
+
+#if defined(USE_AURA)
+scoped_ptr<Browser> CreateBrowserWithAuraTestWindowForParams(
+ scoped_ptr<aura::Window> window,
+ Browser::CreateParams* params) {
+ if (window.get() == nullptr) {
+ window.reset(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 =
+ new TestBrowserWindowAura(window.Pass());
+ new TestBrowserWindowOwner(browser_window);
+ return browser_window->CreateBrowser(params);
}
+#endif // defined(USE_AURA)
} // namespace chrome
@@ -231,3 +250,40 @@ void TestBrowserWindow::ExecuteExtensionCommand(
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();
+}
+
+scoped_ptr<Browser> TestBrowserWindowAura::CreateBrowser(
+ Browser::CreateParams* params) {
+ params->window = this;
+ browser_ = new Browser(*params);
+ return make_scoped_ptr(browser_);
+}
+
+#endif
« chrome/test/base/test_browser_window.h ('K') | « chrome/test/base/test_browser_window.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698