From 9155324b8d7e4b465553e42366319a4e1587ac20 Mon Sep 17 00:00:00 2001 From: "nick@chromium.org" Date: Mon, 2 Aug 2010 18:24:53 +0000 Subject: Revert 54560 - Change removing method, GetBrowser from TabContentsDelegate, as this was breaking an abstraction layer. This routine was originally added in CL 434046, which required the Browser* to construct extension popup views from within Chrome-Frame instances. I changed all accesses to Browser instances from usage of the above method, to either iterating the BrowserList using the situation-specific profile as a search key, or modifying the appropriate delegate interfaces to provide the functionality that was previously used directly via the Browser. BUG=None TEST=None Review URL: http://codereview.chromium.org/2941001 TBR=twiz@google.com Review URL: http://codereview.chromium.org/3012042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54565 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/tabs/tab_strip_model.h | 3 --- chrome/browser/tabs/tab_strip_model_unittest.cc | 1 - 2 files changed, 4 deletions(-) (limited to 'chrome/browser/tabs') diff --git a/chrome/browser/tabs/tab_strip_model.h b/chrome/browser/tabs/tab_strip_model.h index 92cecfd..f93bfc5 100644 --- a/chrome/browser/tabs/tab_strip_model.h +++ b/chrome/browser/tabs/tab_strip_model.h @@ -241,9 +241,6 @@ class TabStripModelDelegate { // Toggles the use of the vertical tabstrip. virtual void ToggleUseVerticalTabs() = 0; - // Returns true if the tab strip can use large icons. - virtual bool LargeIconsPermitted() const = 0; - protected: virtual ~TabStripModelDelegate() {} }; diff --git a/chrome/browser/tabs/tab_strip_model_unittest.cc b/chrome/browser/tabs/tab_strip_model_unittest.cc index 6f53630..c885d5f 100644 --- a/chrome/browser/tabs/tab_strip_model_unittest.cc +++ b/chrome/browser/tabs/tab_strip_model_unittest.cc @@ -80,7 +80,6 @@ class TabStripDummyDelegate : public TabStripModelDelegate { virtual void BookmarkAllTabs() {} virtual bool UseVerticalTabs() const { return false; } virtual void ToggleUseVerticalTabs() {} - virtual bool LargeIconsPermitted() const { return true; } private: // A dummy TabContents we give to callers that expect us to actually build a -- cgit v1.1