Add an accessibility description for notification

ui/arc/notification/arc_notification_content_view.cc

Index: ui/arc/notification/arc_notification_content_view.cc
diff --git a/ui/arc/notification/arc_notification_content_view.cc b/ui/arc/notification/arc_notification_content_view.cc
index 312dcb97f903e63d66ce622d45743429f8169721..0ad970196c7c0cdf588169152c68783dd4da9f16 100644
--- a/ui/arc/notification/arc_notification_content_view.cc
+++ b/ui/arc/notification/arc_notification_content_view.cc
@@ -637,7 +637,7 @@ bool ArcNotificationContentView::HandleAccessibleAction(
 
 void ArcNotificationContentView::GetAccessibleNodeData(
     ui::AXNodeData* node_data) {
-  node_data->role = ui::AX_ROLE_BUTTON;

[David Tseng, 2017/06/02 17:56:11]

I would keep this.

[yoshiki, 2017/06/05 05:06:03]

Thank you for comment. if you don't think this change is necessary, I don't do.

[David Tseng, 2017/06/05 16:27:17]

No problem. It's is needed...see the other comment.

+  node_data->role = ui::AX_ROLE_NOTIFICATION;

[David Tseng, 2017/06/02 17:56:11]

And add this as role description string attribute.

[David Tseng, 2017/06/05 16:27:17]

A role is a programmatic identifier (string) that describes the essential usage
of the widget e.g. button, checkbox, etc. A role description is a localized
string describing the widget's role e.g. push button, dial, etc.

My suggestion was simply to add this string as a role description.

To add a new role, you would have to make ChromeVox aware of that role or re-use
an existing role.

Your change prompted me to hook up support for role descriptions in ChromeVox
(which was missed whenever it was added in Blink).

   node_data->SetName(accessible_name_);
 }