Index: chrome/browser/ui/views/frame/opaque_browser_frame_view.cc |
diff --git a/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc b/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc |
index 19ad04299e523b4456ea48419dfa68b213a8b27f..f370c85a8039cadb8d6536f4986bf91686247f51 100644 |
--- a/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc |
+++ b/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc |
@@ -428,10 +428,8 @@ void OpaqueBrowserFrameView::Observe( |
bool OpaqueBrowserFrameView::ShouldShowWindowIcon() const { |
views::WidgetDelegate* delegate = frame()->widget_delegate(); |
- // ShouldShowTitleBar defaults to true. |
- if (platform_observer_ && !platform_observer_->ShouldShowTitleBar()) |
- return false; |
- return delegate && delegate->ShouldShowWindowIcon(); |
+ return platform_observer_->ShouldShowTitleBar() && delegate && |
+ delegate->ShouldShowWindowIcon(); |
} |
bool OpaqueBrowserFrameView::ShouldShowWindowTitle() const { |
@@ -439,10 +437,8 @@ bool OpaqueBrowserFrameView::ShouldShowWindowTitle() const { |
// a window is being destroyed. |
// See more discussion at http://crosbug.com/8958 |
views::WidgetDelegate* delegate = frame()->widget_delegate(); |
- // ShouldShowTitleBar defaults to true. |
- if (platform_observer_ && !platform_observer_->ShouldShowTitleBar()) |
- return false; |
- return delegate && delegate->ShouldShowWindowTitle(); |
+ return platform_observer_->ShouldShowTitleBar() && delegate && |
+ delegate->ShouldShowWindowTitle(); |
} |
base::string16 OpaqueBrowserFrameView::GetWindowTitle() const { |
@@ -470,8 +466,6 @@ gfx::Size OpaqueBrowserFrameView::GetBrowserViewMinimumSize() const { |
bool OpaqueBrowserFrameView::ShouldShowCaptionButtons() const { |
if (!OpaqueBrowserFrameViewLayout::ShouldAddDefaultCaptionButtons()) |
return false; |
- if (!platform_observer_) |
- return true; |
return platform_observer_->ShouldShowCaptionButtons(); |
} |