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

Unified Diff: chrome/browser/views/frame/browser_window_factory.cc

Issue 10896: Re-do the way browser windows are shown:... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 12 years, 1 month 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/views/frame/browser_window_factory.cc
===================================================================
--- chrome/browser/views/frame/browser_window_factory.cc (revision 5374)
+++ chrome/browser/views/frame/browser_window_factory.cc (working copy)
@@ -17,12 +17,10 @@
// BrowserWindow, public:
// static
-BrowserWindow* BrowserWindow::CreateBrowserWindow(Browser* browser,
- const gfx::Rect& bounds,
- int show_command) {
+BrowserWindow* BrowserWindow::CreateBrowserWindow(Browser* browser) {
BrowserView* browser_view = new BrowserView(browser);
BrowserFrame::CreateForBrowserView(BrowserFrame::GetActiveFrameType(),
- browser_view, bounds, show_command);
+ browser_view);
return browser_view;
}
@@ -37,16 +35,14 @@
// static
BrowserFrame* BrowserFrame::CreateForBrowserView(BrowserFrame::FrameType type,
- BrowserView* browser_view,
- const gfx::Rect& bounds,
- int show_command) {
+ BrowserView* browser_view) {
if (type == FRAMETYPE_OPAQUE) {
OpaqueFrame* frame = new OpaqueFrame(browser_view);
- frame->Init(NULL, bounds);
+ frame->Init();
return frame;
} else if (type == FRAMETYPE_AERO_GLASS) {
AeroGlassFrame* frame = new AeroGlassFrame(browser_view);
- frame->Init(bounds);
+ frame->Init();
return frame;
}
NOTREACHED() << "Unsupported frame type";

Powered by Google App Engine
This is Rietveld 408576698