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

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

Issue 2581023002: Add tab status to accessibility labels (Closed)
Patch Set: Add check for active browser tab index to prevent crashes Created 4 years 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 63f6bff9f5240be5a07f2efab5834f8f74e6b138..6c766d211f7fefdd2f1b2be1684c635b217aa74a 100644
--- a/chrome/browser/ui/views/tabs/browser_tab_strip_controller.h
+++ b/chrome/browser/ui/views/tabs/browser_tab_strip_controller.h
@@ -76,6 +76,7 @@ class BrowserTabStripController : public TabStripController,
void OnStoppedDraggingTabs() override;
void CheckFileSupported(const GURL& url) override;
SkColor GetToolbarTopSeparatorColor() const override;
+ base::string16 GetAccessibleTabName() const override;
// TabStripModelObserver implementation:
void TabInsertedAt(TabStripModel* tab_strip_model,
@@ -101,6 +102,8 @@ class BrowserTabStripController : public TabStripController,
void TabBlockedStateChanged(content::WebContents* contents,
int model_index) override;
+ const Browser* browser() const { return browser_; }
+
protected:
// The context in which SetTabRendererDataFromModel is being called.
enum TabStatus {
@@ -118,8 +121,6 @@ class BrowserTabStripController : public TabStripController,
const TabStrip* tabstrip() const { return tabstrip_; }
- const Browser* browser() const { return browser_; }
-
private:
class TabContextMenuContents;

Powered by Google App Engine
This is Rietveld 408576698