Fix Regression for Accessible Names appearing as NAMELESS.

The refactoring in r43558 caused SetAccessibleName and GetAccessibleName to be none virtual causing couple of components not having accessible names since they depend on overriding it. The following change sets the accessible name directly since view now stores it.

BUG=40343
TEST=With AccExplorer32 check if accessible names have been resolved.

Review URL: http://codereview.chromium.org/1616004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43584 0039d316-1c4b-4281-b951-d872f2087c98

5 files changed, 7 insertions, 20 deletions

diff --git a/chrome/browser/views/browser_actions_container.cc b/chrome/browser/views/browser_actions_container.cc
index 9a7c729..d4f6a42 100644
--- a/chrome/browser/views/browser_actions_container.cc
+++ b/chrome/browser/views/browser_actions_container.cc
@@ -299,6 +299,8 @@ BrowserActionView::BrowserActionView(Extension* extension,
   button_->SetDragController(panel_);
   AddChildView(button_);
   button_->UpdateState();
+  SetAccessibleName(
+      l10n_util::GetString(IDS_ACCNAME_EXTENSIONS_BROWSER_ACTION));
 }
 
 BrowserActionView::~BrowserActionView() {
@@ -331,12 +333,6 @@ bool BrowserActionView::GetAccessibleRole(AccessibilityTypes::Role* role) {
   return true;
 }
 
-bool BrowserActionView::GetAccessibleName(std::wstring* name) {
-  DCHECK(name);
-  *name = l10n_util::GetString(IDS_ACCNAME_EXTENSIONS_BROWSER_ACTION);
-  return true;
-}
-
 void BrowserActionView::Layout() {
   button_->SetBounds(0, kControlVertOffset, width(), kButtonSize);
 }
@@ -416,6 +412,8 @@ BrowserActionsContainer::BrowserActionsContainer(
   else
     chevron_->SetVisible(true);
   container_size_ = gfx::Size(IconCountToWidth(visible_actions), kButtonSize);
+
+  SetAccessibleName(l10n_util::GetString(IDS_ACCNAME_EXTENSIONS));
 }
 
 BrowserActionsContainer::~BrowserActionsContainer() {
@@ -825,12 +823,6 @@ bool BrowserActionsContainer::GetAccessibleRole(
   return true;
 }
 
-bool BrowserActionsContainer::GetAccessibleName(std::wstring* name) {
-  DCHECK(name);
-  *name = l10n_util::GetString(IDS_ACCNAME_EXTENSIONS);
-  return !name->empty();
-}
-
 void BrowserActionsContainer::MoveBrowserAction(
     const std::string& extension_id, size_t new_index) {
   ExtensionsService* service = profile_->GetExtensionsService();
diff --git a/chrome/browser/views/browser_actions_container.h b/chrome/browser/views/browser_actions_container.h
index de6048c..9934d8c 100644
--- a/chrome/browser/views/browser_actions_container.h
+++ b/chrome/browser/views/browser_actions_container.h
@@ -153,7 +153,6 @@ class BrowserActionView : public views::View {
 
   // Accessibility accessors, overridden from View.
   virtual bool GetAccessibleRole(AccessibilityTypes::Role* role);
-  virtual bool GetAccessibleName(std::wstring* name);
 
  private:
   virtual void Layout();
@@ -323,7 +322,6 @@ class BrowserActionsContainer
   virtual void OnDragExited();
   virtual int OnPerformDrop(const views::DropTargetEvent& event);
   virtual bool GetAccessibleRole(AccessibilityTypes::Role* role);
-  virtual bool GetAccessibleName(std::wstring* name);
 
   // Overridden from views::ViewMenuDelegate:
   virtual void RunMenu(View* source, const gfx::Point& pt);
diff --git a/chrome/browser/views/tabs/tab.cc b/chrome/browser/views/tabs/tab.cc
index 3d0f422..3f3da3f 100644
--- a/chrome/browser/views/tabs/tab.cc
+++ b/chrome/browser/views/tabs/tab.cc
@@ -196,11 +196,6 @@ bool Tab::GetAccessibleRole(AccessibilityTypes::Role* role) {
   return true;
 }
 
-bool Tab::GetAccessibleName(std::wstring* name) {
-  *name = GetTitle();
-  return !name->empty();
-}
-
 ///////////////////////////////////////////////////////////////////////////////
 // Tab, views::ContextMenuController implementation:
 
diff --git a/chrome/browser/views/tabs/tab.h b/chrome/browser/views/tabs/tab.h
index f491291..79b08f2 100644
--- a/chrome/browser/views/tabs/tab.h
+++ b/chrome/browser/views/tabs/tab.h
@@ -101,7 +101,6 @@ class Tab : public TabRenderer,
   virtual bool GetTooltipTextOrigin(const gfx::Point& p, gfx::Point* origin);
   virtual std::string GetClassName() const { return kTabClassName; }
   virtual bool GetAccessibleRole(AccessibilityTypes::Role* role);
-  virtual bool GetAccessibleName(std::wstring* name);
 
   // views::ContextMenuController overrides:
   virtual void ShowContextMenu(views::View* source,
diff --git a/chrome/browser/views/tabs/tab_renderer.cc b/chrome/browser/views/tabs/tab_renderer.cc
index f301b03..4f51b05 100644
--- a/chrome/browser/views/tabs/tab_renderer.cc
+++ b/chrome/browser/views/tabs/tab_renderer.cc
@@ -320,6 +320,9 @@ void TabRenderer::UpdateData(TabContents* contents,
     else
       data_.favicon = contents->GetFavIcon();
     data_.phantom = phantom;
+
+    // Sets the accessible name for the tab.
+    SetAccessibleName(data_.title);
   }
 
   // TODO(glen): Temporary hax.