Add tab status to accessibility labels

chrome/browser/ui/views/tabs/browser_tab_strip_controller.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_TABS_BROWSER_TAB_STRIP_CONTROLLER_H_
 #define CHROME_BROWSER_UI_VIEWS_TABS_BROWSER_TAB_STRIP_CONTROLLER_H_
 
 #include <memory>
 
 #include "base/compiler_specific.h"
@@ skipping 15 matching lines @@
 
 namespace ui {
 class ListSelectionModel;
 }
 
 // An implementation of TabStripController that sources data from the
 // WebContentses in a TabStripModel.
 class BrowserTabStripController : public TabStripController,
                                   public TabStripModelObserver {
  public:
-  BrowserTabStripController(Browser* browser, TabStripModel* model);
+  BrowserTabStripController(Browser* browser,
+                            TabStripModel* model,
+                            BrowserView* browser_view);

[sky, 2017/01/03 23:11:41]

BrowserView has a browser() function, so that now that this depends upon
BrowserView you shouldn't need the Browser* member.

[edwardjung, 2017/01/05 16:59:45]

Done.

   ~BrowserTabStripController() override;
 
   void InitFromModel(TabStrip* tabstrip);
 
   TabStripModel* model() const { return model_; }
 
   bool IsCommandEnabledForTab(TabStripModel::ContextMenuCommand command_id,
                               Tab* tab) const;
   void ExecuteCommandForTab(TabStripModel::ContextMenuCommand command_id,
                             Tab* tab);
@@ skipping 22 matching lines @@
   void PerformDrop(bool drop_before, int index, const GURL& url) override;
   bool IsCompatibleWith(TabStrip* other) const override;
   void CreateNewTab() override;
   void CreateNewTabWithLocation(const base::string16& loc) override;
   bool IsIncognito() override;
   void StackedLayoutMaybeChanged() override;
   void OnStartedDraggingTabs() override;
   void OnStoppedDraggingTabs() override;
   void CheckFileSupported(const GURL& url) override;
   SkColor GetToolbarTopSeparatorColor() const override;
+  base::string16 GetAccessibleTabName(const Tab* tab) const override;
 
   // TabStripModelObserver implementation:
   void TabInsertedAt(TabStripModel* tab_strip_model,
                      content::WebContents* contents,
                      int model_index,
                      bool is_active) override;
   void TabDetachedAt(content::WebContents* contents, int model_index) override;
   void TabSelectionChanged(TabStripModel* tab_strip_model,
                            const ui::ListSelectionModel& old_model) override;
   void TabMoved(content::WebContents* contents,
                 int from_model_index,
                 int to_model_index) override;
   void TabChangedAt(content::WebContents* contents,
                     int model_index,
                     TabChangeType change_type) override;
   void TabReplacedAt(TabStripModel* tab_strip_model,
                      content::WebContents* old_contents,
                      content::WebContents* new_contents,
                      int model_index) override;
   void TabPinnedStateChanged(TabStripModel* tab_strip_model,
                              content::WebContents* contents,
                              int model_index) override;
   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 {
     NEW_TAB,
     EXISTING_TAB
   };
 
   // Sets the TabRendererData from the TabStripModel.
   virtual void SetTabRendererDataFromModel(content::WebContents* contents,
                                            int model_index,
                                            TabRendererData* data,
                                            TabStatus tab_status);
 
   Profile* profile() const { return model_->profile(); }
 
   const TabStrip* tabstrip() const { return tabstrip_; }
 
-  const Browser* browser() const { return browser_; }
-
  private:
   class TabContextMenuContents;
 
   // Invokes tabstrip_->SetTabData.
   void SetTabDataAt(content::WebContents* web_contents, int model_index);
 
   void StartHighlightTabsForCommand(
       TabStripModel::ContextMenuCommand command_id,
       Tab* tab);
   void StopHighlightTabsForCommand(
@@ skipping 11 matching lines @@
   void OnFindURLMimeTypeCompleted(const GURL& url,
                                   const std::string& mime_type);
 
   TabStripModel* model_;
 
   TabStrip* tabstrip_;
 
   // Non-owning pointer to the browser which is using this controller.
   Browser* browser_;
 
+  BrowserView* browser_view_;
+
   // If non-NULL it means we're showing a menu for the tab.
   std::unique_ptr<TabContextMenuContents> context_menu_contents_;
 
   // Helper for performing tab selection as a result of dragging over a tab.
   HoverTabSelector hover_tab_selector_;
 
   // Forces the tabs to use the regular (non-immersive) style and the
   // top-of-window views to be revealed when the user is dragging |tabstrip|'s
   // tabs.
   std::unique_ptr<ImmersiveRevealedLock> immersive_reveal_lock_;
 
   PrefChangeRegistrar local_pref_registrar_;
 
   base::WeakPtrFactory<BrowserTabStripController> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserTabStripController);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_BROWSER_TAB_STRIP_CONTROLLER_H_