From 0fbb214b6345b8490e559c1a4f8f66d704df72d2 Mon Sep 17 00:00:00 2001 From: "tc@google.com" Date: Wed, 4 Mar 2009 05:37:01 +0000 Subject: Take 2 at moving SupportsWindowFeature from BrowserView to Browser. Turns out the test regression was actually caused by r10806 which removed an anonymous namespace causing the TestView class to collide with the same class in render_widget_host_unittest.cc. For now, fix by adding the TestView in the view unittest to the anon namespace. shess is going to try to re-add the namespace to render_widget_host_unittest.cc too. TBR=pkasting Review URL: http://codereview.chromium.org/40058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10873 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/views/frame/browser_view.cc | 35 ++++++++++-------------------- chrome/browser/views/frame/browser_view.h | 15 ------------- 2 files changed, 11 insertions(+), 39 deletions(-) (limited to 'chrome/browser/views') diff --git a/chrome/browser/views/frame/browser_view.cc b/chrome/browser/views/frame/browser_view.cc index de73484..fc0bca3 100644 --- a/chrome/browser/views/frame/browser_view.cc +++ b/chrome/browser/views/frame/browser_view.cc @@ -323,12 +323,12 @@ int BrowserView::GetTabStripHeight() const { } bool BrowserView::IsToolbarVisible() const { - return SupportsWindowFeature(FEATURE_TOOLBAR) || - SupportsWindowFeature(FEATURE_LOCATIONBAR); + return browser_->SupportsWindowFeature(Browser::FEATURE_TOOLBAR) || + browser_->SupportsWindowFeature(Browser::FEATURE_LOCATIONBAR); } bool BrowserView::IsTabStripVisible() const { - return SupportsWindowFeature(FEATURE_TABSTRIP); + return browser_->SupportsWindowFeature(Browser::FEATURE_TABSTRIP); } bool BrowserView::IsOffTheRecord() const { @@ -706,7 +706,7 @@ void BrowserView::DestroyBrowser() { } bool BrowserView::IsBookmarkBarVisible() const { - return SupportsWindowFeature(FEATURE_BOOKMARKBAR) && + return browser_->SupportsWindowFeature(Browser::FEATURE_BOOKMARKBAR) && active_bookmark_bar_ && (active_bookmark_bar_->GetPreferredSize().height() != 0); } @@ -972,7 +972,7 @@ views::View* BrowserView::GetInitiallyFocusedView() { } bool BrowserView::ShouldShowWindowTitle() const { - return SupportsWindowFeature(FEATURE_TITLEBAR); + return browser_->SupportsWindowFeature(Browser::FEATURE_TITLEBAR); } SkBitmap BrowserView::GetWindowIcon() { @@ -982,7 +982,7 @@ SkBitmap BrowserView::GetWindowIcon() { } bool BrowserView::ShouldShowWindowIcon() const { - return SupportsWindowFeature(FEATURE_TITLEBAR); + return browser_->SupportsWindowFeature(Browser::FEATURE_TITLEBAR); } bool BrowserView::ExecuteWindowsCommand(int command_id) { @@ -1289,21 +1289,6 @@ void BrowserView::InitSystemMenu() { } } -bool BrowserView::SupportsWindowFeature(WindowFeature feature) const { - unsigned int features = FEATURE_INFOBAR | FEATURE_DOWNLOADSHELF; - if (IsBrowserTypeNormal()) - features |= FEATURE_BOOKMARKBAR; - if (!fullscreen_) { - if (IsBrowserTypeNormal()) - features |= FEATURE_TABSTRIP | FEATURE_TOOLBAR; - else - features |= FEATURE_TITLEBAR; - if ((browser_->type() & Browser::TYPE_APP) == 0) - features |= FEATURE_LOCATIONBAR; - } - return !!(features & feature); -} - bool BrowserView::ShouldForwardToTabStrip( const views::DropTargetEvent& event) { if (!tabstrip_->IsVisible()) @@ -1395,7 +1380,7 @@ int BrowserView::LayoutBookmarkBar(int top) { } int BrowserView::LayoutInfoBar(int top) { - bool visible = SupportsWindowFeature(FEATURE_INFOBAR); + bool visible = browser_->SupportsWindowFeature(Browser::FEATURE_INFOBAR); int height = visible ? infobar_container_->GetPreferredSize().height() : 0; infobar_container_->SetVisible(visible); infobar_container_->SetBounds(0, top, width(), height); @@ -1409,7 +1394,8 @@ void BrowserView::LayoutTabContents(int top, int bottom) { int BrowserView::LayoutDownloadShelf() { int bottom = height(); if (active_download_shelf_) { - bool visible = SupportsWindowFeature(FEATURE_DOWNLOADSHELF); + bool visible = browser_->SupportsWindowFeature( + Browser::FEATURE_DOWNLOADSHELF); int height = visible ? active_download_shelf_->GetPreferredSize().height() : 0; active_download_shelf_->SetVisible(visible); @@ -1433,7 +1419,8 @@ void BrowserView::LayoutStatusBubble(int top) { bool BrowserView::MaybeShowBookmarkBar(TabContents* contents) { views::View* new_bookmark_bar_view = NULL; - if (SupportsWindowFeature(FEATURE_BOOKMARKBAR) && contents) { + if (browser_->SupportsWindowFeature(Browser::FEATURE_BOOKMARKBAR) + && contents) { if (!bookmark_bar_view_.get()) { bookmark_bar_view_.reset(new BookmarkBarView(contents->profile(), browser_.get())); diff --git a/chrome/browser/views/frame/browser_view.h b/chrome/browser/views/frame/browser_view.h index 06dc8da..af9a75f 100644 --- a/chrome/browser/views/frame/browser_view.h +++ b/chrome/browser/views/frame/browser_view.h @@ -157,17 +157,6 @@ class BrowserView : public BrowserWindow, // to add or delete entries. void PrepareToRunSystemMenu(HMENU menu); - // Possible elements of the Browser window. - enum WindowFeature { - FEATURE_TITLEBAR = 1, - FEATURE_TABSTRIP = 2, - FEATURE_TOOLBAR = 4, - FEATURE_LOCATIONBAR = 8, - FEATURE_BOOKMARKBAR = 16, - FEATURE_INFOBAR = 32, - FEATURE_DOWNLOADSHELF = 64 - }; - // Returns true if the Browser object associated with this BrowserView is a // normal-type window (i.e. a browser window, not an app or popup). bool IsBrowserTypeNormal() const { @@ -301,10 +290,6 @@ class BrowserView : public BrowserWindow, // Creates the system menu. void InitSystemMenu(); - // Returns true if the Browser object associated with this BrowserView - // supports the specified feature. - bool SupportsWindowFeature(WindowFeature feature) const; - // Returns true if the event should be forwarded to the TabStrip. This // returns true if y coordinate is less than the bottom of the tab strip, and // is not over another child view. -- cgit v1.1