Chromium Code Reviews| 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 { |