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

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

Issue 1001003: Allow dynamic switching in and out of sidetabs mode. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 9 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/browser/views/frame/browser_frame_win.cc
===================================================================
--- chrome/browser/views/frame/browser_frame_win.cc (revision 41630)
+++ chrome/browser/views/frame/browser_frame_win.cc (working copy)
@@ -69,9 +69,6 @@
return this;
}
-void BrowserFrameWin::TabStripCreated(BaseTabStrip* tabstrip) {
-}
-
int BrowserFrameWin::GetMinimizeButtonOffset() const {
TITLEBARINFOEX titlebar_info;
titlebar_info.cbSize = sizeof(TITLEBARINFOEX);
@@ -127,6 +124,12 @@
browser_frame_view_->PaintTabStripShadow(canvas);
}
+void BrowserFrameWin::TabStripDisplayModeChanged() {
+ GetRootView()->Layout();
sky 2010/03/19 16:14:02 Why do you need to layout twice? If you really do,
+ UpdateDWMFrame();
+ GetRootView()->Layout();
+}
+
///////////////////////////////////////////////////////////////////////////////
// BrowserFrame, views::WindowWin overrides:
@@ -294,7 +297,7 @@
// In maximized mode, we only have a titlebar strip of glass, no side/bottom
// borders.
if (!browser_view_->IsFullscreen()) {
- if (browser_view_->UsingSideTabs()) {
+ if (browser_view_->UseVerticalTabs()) {
margins.cxLeftWidth +=
GetBoundsForTabStrip(browser_view_->tabstrip()).right();
margins.cyTopHeight += GetSystemMetrics(SM_CYSIZEFRAME);
@@ -309,7 +312,7 @@
DwmExtendFrameIntoClientArea(GetNativeView(), &margins);
DWORD window_style = GetWindowLong(GWL_STYLE);
- if (browser_view_->UsingSideTabs()) {
+ if (browser_view_->UseVerticalTabs()) {
if (window_style & WS_CAPTION)
SetWindowLong(GWL_STYLE, window_style & ~WS_CAPTION);
} else {

Powered by Google App Engine
This is Rietveld 408576698