Added tab notification to accessibility_event_router_views.

chrome/browser/ui/views/tabs/tab_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_TAB_CONTROLLER_H_
 #define CHROME_BROWSER_UI_VIEWS_TABS_TAB_CONTROLLER_H_
 
 #include "chrome/browser/ui/views/tabs/tab_strip_types.h"
 
 class Tab;
@@ skipping 71 matching lines @@
                                  const ui::MouseEvent& event) = 0;
 
   // Returns true if |tab| needs to be painted. If false is returned the tab is
   // not painted. If true is returned the tab should be painted and |clip| is
   // set to the clip (if |clip| is empty means no clip).
   virtual bool ShouldPaintTab(const Tab* tab, gfx::Rect* clip) = 0;
 
   // Returns true if tabs painted in the rectangular light-bar style.
   virtual bool IsImmersiveStyle() const = 0;
 
+  // Returns the index of the specified tab in the model coordinate system, or
+  // -1 if tab is closing or not valid.
+  virtual int GetModelIndexOfTab(const Tab* tab) const = 0;

[sky, 2014/07/17 04:06:10]

If we can I would like to avoid exposing this state. Could you instead add an
explicit function that adds the state you care about for accessibility.
Something like GetTabAccessibilityState(Tab* ...)?

[evy, 2014/07/21 17:54:15]

That's a great way to do it! Thanks

I wasn't sure if you meant "explicit" as in using the explicit keyword - I'm not
as familiar with it. I looked into it a bit and I noticed it's mostly used for
constructors, so I didn't include it here but I figured I'd ask just in case.

+
+  // Gets the number of Tabs in the tab strip.
+  virtual int GetTabCount() const = 0;
+
  protected:
   virtual ~TabController() {}
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_TAB_CONTROLLER_H_