Add tab status to accessibility labels

chrome/browser/ui/views/frame/browser_view.cc

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index d31ac99ae903b9296a4609cd19f2951544232288..17b6e0d3d5b038aee69498fe9c5cb3a915150f65 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -1594,13 +1594,9 @@ base::string16 BrowserView::GetWindowTitle() const {
 }
 
 base::string16 BrowserView::GetAccessibleWindowTitle() const {
-  const bool include_app_name = false;
-  if (IsIncognito()) {
-    return l10n_util::GetStringFUTF16(
-        IDS_ACCESSIBLE_INCOGNITO_WINDOW_TITLE_FORMAT,
-        browser_->GetWindowTitleForCurrentTab(include_app_name));
-  }
-  return browser_->GetWindowTitleForCurrentTab(include_app_name);

[sky, 2016/11/26 17:19:33]

GetWindowTitleForCurrentTab() has a couple of heuristics for getting the title
that you're missing. I would prefer GetAccessibleTabLabel is here, and not the
in the tab class, mostly because I think you want parts of what
GetWindowTitleForCurrentTab does.

[edwardjung, 2016/12/06 20:23:16]

Done.

+  Tab* current_tab =
+      tabstrip_->tab_at(browser_->tab_strip_model()->active_index());
+  return current_tab->GetAccessibleTabLabel();
 }
 
 views::View* BrowserView::GetInitiallyFocusedView() {