Adds support and exposure of additional accessibility roles. Includes naming ...

views/controls/image_view.cc

Index: views/controls/image_view.cc
===================================================================
--- views/controls/image_view.cc	(revision 23961)
+++ views/controls/image_view.cc	(working copy)
@@ -136,6 +136,23 @@
   }
 }
 
+bool ImageView::GetAccessibleName(std::wstring* name) {
+  if (tooltip_text_.empty()) {
+    return false;
+  } else {
+    *name = GetTooltipText();

[sky, 2009/08/21 21:33:39]

Both of these should be consistent. Either use tooltip_text_ in both, or
GetTooltipText in both. Would also be nice if the format of this matched
GetAccessibleRole, eg:

if (!tooltip_text_.empty())
  return false;
...

+    return true;
+  }
+}
+
+bool ImageView::GetAccessibleRole(AccessibilityTypes::Role* role) {
+  if (!role)
+    return false;
+
+  *role = AccessibilityTypes::ROLE_GRAPHIC;
+  return true;
+}
+
 void ImageView::SetHorizontalAlignment(Alignment ha) {
   if (ha != horiz_alignment_) {
     horiz_alignment_ = ha;
@@ -170,7 +187,7 @@
   if (tooltip_text_.empty()) {
     return false;
   } else {
-    * tooltip =  GetTooltipText();
+    *tooltip = GetTooltipText();
     return true;
   }
 }