Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1355)

Unified Diff: chrome/browser/ui/views/tabs/browser_tab_strip_controller.h

Issue 7065052: Improve large tab strip by leveraging touch icons when present (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: removed extra line Created 9 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/tabs/browser_tab_strip_controller.h
diff --git a/chrome/browser/ui/views/tabs/browser_tab_strip_controller.h b/chrome/browser/ui/views/tabs/browser_tab_strip_controller.h
index b1baac40247d1d8a20ba8c50d5c13f476078c5bb..d988cffc05fe90de48cd1206b447f24af1b2e658 100644
--- a/chrome/browser/ui/views/tabs/browser_tab_strip_controller.h
+++ b/chrome/browser/ui/views/tabs/browser_tab_strip_controller.h
@@ -97,17 +97,25 @@ class BrowserTabStripController : public TabStripController,
const NotificationSource& source,
const NotificationDetails& details) OVERRIDE;
+ protected:
+ // Sets the TabRendererData from the TabStripModel.
+ virtual void SetTabRendererDataFromModel(TabContents* contents,
+ int model_index,
+ TabRendererData* data);
+
+ TabStripModel* model_;
+
+ BaseTabStrip* tabstrip_;
+
+ // Non-owning pointer to the browser which is using this controller.
+ Browser* browser_;
+
private:
class TabContextMenuContents;
// Invokes tabstrip_->SetTabData.
void SetTabDataAt(TabContentsWrapper* contents, int model_index);
- // Sets the TabRendererData from the TabStripModel.
- void SetTabRendererDataFromModel(TabContents* contents,
- int model_index,
- TabRendererData* data);
-
void StartHighlightTabsForCommand(
TabStripModel::ContextMenuCommand command_id,
BaseTab* tab);
@@ -117,13 +125,6 @@ class BrowserTabStripController : public TabStripController,
Profile* profile() const { return model_->profile(); }
- TabStripModel* model_;
-
- BaseTabStrip* tabstrip_;
-
- // Non-owning pointer to the browser which is using this controller.
- Browser* browser_;
-
// If non-NULL it means we're showing a menu for the tab.
scoped_ptr<TabContextMenuContents> context_menu_contents_;

Powered by Google App Engine
This is Rietveld 408576698