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

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: Resynced (again) 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..71ca58aa4fea60bcdd2fd6f39a493d6ada817469 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,31 @@ class BrowserTabStripController : public TabStripController,
const NotificationSource& source,
const NotificationDetails& details) OVERRIDE;
+ protected:
+ // The context in which SetTabRendererDataFromModel is being called.
+ enum TabStatus {
+ NEW_TAB,
+ EXISTING_TAB
+ };
+
+ // Sets the TabRendererData from the TabStripModel.
+ virtual void SetTabRendererDataFromModel(TabContents* contents,
+ int model_index,
+ TabRendererData* data,
+ TabStatus tab_status);
+
+ Profile* profile() const { return model_->profile(); }
+
+ const BaseTabStrip* tabstrip() const { return tabstrip_; }
+
+ const Browser* browser() const { return 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,8 +129,6 @@ class BrowserTabStripController : public TabStripController,
TabStripModel::ContextMenuCommand command_id,
BaseTab* tab);
- Profile* profile() const { return model_->profile(); }
-
TabStripModel* model_;
BaseTabStrip* tabstrip_;

Powered by Google App Engine
This is Rietveld 408576698