diff options
| author | nick@chromium.org <nick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-02 18:24:53 +0000 |
|---|---|---|
| committer | nick@chromium.org <nick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-02 18:24:53 +0000 |
| commit | 9155324b8d7e4b465553e42366319a4e1587ac20 (patch) | |
| tree | 6faecbce10a5163ab46782cf654e49c937ce6e1a /chrome/browser/tabs | |
| parent | c60bf3db722742386dcf4f8c9dc86e7a812688b7 (diff) | |
| download | chromium_src-9155324b8d7e4b465553e42366319a4e1587ac20.zip chromium_src-9155324b8d7e4b465553e42366319a4e1587ac20.tar.gz chromium_src-9155324b8d7e4b465553e42366319a4e1587ac20.tar.bz2 | |
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
Diffstat (limited to 'chrome/browser/tabs')
| -rw-r--r-- | chrome/browser/tabs/tab_strip_model.h | 3 | ||||
| -rw-r--r-- | chrome/browser/tabs/tab_strip_model_unittest.cc | 1 |
2 files changed, 0 insertions, 4 deletions
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 |