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

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: Fixed URL to ignore fragment and changed item handling 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..b3b71c0b12d3539ee6b4b337a24d6f5616ce92f2 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,27 @@ 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);
+
+ Profile* profile() const { return model_->profile(); }
+
+ 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);
@@ -115,15 +125,6 @@ class BrowserTabStripController : public TabStripController,
TabStripModel::ContextMenuCommand command_id,
BaseTab* tab);
- 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