[App banners] Update accessibility strings

chrome/android/java/strings/android_chrome_strings.grd

 <?xml version="1.0" encoding="UTF-8"?>
 <grit current_release="1" latest_public_release="0">
   <outputs>
     <output filename="values-am/android_chrome_strings.xml" lang="am" type="android" />
     <output filename="values-ar/android_chrome_strings.xml" lang="ar" type="android" />
     <output filename="values-bg/android_chrome_strings.xml" lang="bg" type="android" />
     <output filename="values-ca/android_chrome_strings.xml" lang="ca" type="android" />
     <output filename="values-cs/android_chrome_strings.xml" lang="cs" type="android" />
     <output filename="values-da/android_chrome_strings.xml" lang="da" type="android" />
     <output filename="values-de/android_chrome_strings.xml" lang="de" type="android" />
@@ skipping 811 matching lines @@
 
       <!-- App banner strings -->
       <message name="IDS_APP_BANNER_INSTALLING" desc="Button text indicating that an application is being installed. [CHAR-LIMIT=15]">
         Installing...
       </message>
       <message name="IDS_APP_BANNER_OPEN" desc="Text that indicates that clicking on the button will launch an application. [CHAR-LIMIT=15]">
         Open
       </message>
 
       <!-- App banner accessibility strings, used for touch exploration -->
-      <message name="IDS_APP_BANNER_VIEW_WEB_APP_ACCESSIBILITY" desc="Accessibililty text: Describes the banner content, including the app name and its URL.">
-        Prompt to add page to the homescreen.  App name: <ph name="APP_NAME">%1$s<ex>Gmail</ex></ph>.  App URL: <ph name="APP_URL">%2$s<ex>http://gmail.com</ex></ph>.
-      </message>
-      <message name="IDS_APP_BANNER_VIEW_NATIVE_APP_ACCESSIBILITY" desc="Accessibililty text: Describes the banner content, including the app name and rating.">
+      <message name="IDS_APP_BANNER_VIEW_NATIVE_APP_ACCESSIBILITY" desc="Accessibililty text: Describes a prompt promoting the installation of an app related to the current page, including its app name and rating.">
         Prompt to get app from the Google Play Store.  App name: <ph name="APP_NAME">%1$s<ex>Gmail</ex></ph>.  App average rating: <ph name="APP_RATING">%2$.1f<ex>4.2</ex></ph>.

[newt (away), 2015/02/13 23:49:59]

This reads oddly to me. How about: "Install app from the Google Play Store. App
name: Gmail. Rating: 4.2 out of 5."

[gone, 2015/02/13 23:52:09]

Sometimes it costs money.

[gone, 2015/02/13 23:57:13]

That and this message is used for the whole banner.  Clicking on it does
nothing.

       </message>
-      <message name="IDS_APP_BANNER_INSTALL_ACCESSIBILITY" desc="Accessibility text: Indicates that clicking on the button will purchase the app or install it.">
-        Get app from the Google Play Store: <ph name="APP_ACTION">%s<ex>Install</ex></ph>
+      <message name="IDS_APP_BANNER_VIEW_NATIVE_APP_ICON_ACCESSIBILITY" desc="Accessibililty text: Indicates that the user is hovering over the app icon and that clicking it will take the user to the Play Store.">
+        Click to open the Play Store page for the app: <ph name="APP_NAME">%1$s<ex>Gmail</ex></ph>.
+      </message>
+      <message name="IDS_APP_BANNER_VIEW_NATIVE_APP_INSTALL_ACCESSIBILITY" desc="Accessibility text: Indicates that clicking on the button will purchase the app or install it.">

[newt (away), 2015/02/13 23:49:58]

I don't think the button needs a content description. "Install" is quite clear.

[gone, 2015/02/13 23:52:09]

Sometimes it's a currency amount.

[newt (away), 2015/02/14 00:00:19]

True. But what does a sighted user see? "$0.99" or "Install". Why should we
provide extra/different information to TalkBack users?

Also, I'm imagining a TalkBack user exploring the views in this infobar and
hearing "Prompt to get app from the Google Play Store. App name: Gmail. App
average rating: 4.2. Click to open the Play Storage page for the app: Gmail. App
average rating: 4.2. Get the app from the Google Play Store: Install." It's
quite wordy.

+        Get the app from the Google Play Store: <ph name="APP_ACTION">%s<ex>Install</ex></ph>
+      </message>
+      <message name="IDS_APP_BANNER_VIEW_NATIVE_APP_RATING_ACCESSIBILITY" desc="Accessibililty text: Indicates the Play Store rating of the app being promoted by the current page.">
+        App average rating: <ph name="APP_RATING">%1$.1f<ex>4.2</ex></ph>.

[newt (away), 2015/02/13 23:49:58]

I'd remove "average" and maybe "app" too. It might be nice to add "out of 5"

+      </message>
+
+      <message name="IDS_APP_BANNER_VIEW_WEB_APP_ACCESSIBILITY" desc="Accessibililty text: Describes the banner content, including the app name and its URL.">
+        Prompt to add a promoted page to the homescreen.  App name: <ph name="APP_NAME">%1$s<ex>Gmail</ex></ph>.

[newt (away), 2015/02/13 23:49:58]

"Add web app to homescreen. App name: Gmail"

+      </message>
+      <message name="IDS_APP_BANNER_VIEW_WEB_APP_ICON_ACCESSIBILITY" desc="Accessibililty text: Indicates that the user is hovering over an icon representing an app being promoted by the current web page.">
+        Icon for the app: <ph name="APP_NAME">%1$s<ex>Gmail</ex></ph>.
+      </message>
+      <message name="IDS_APP_BANNER_VIEW_WEB_APP_INSTALL_ACCESSIBILITY" desc="Accessibility text: Indicates that the user is hovering over a button to add the app being promoted by the current web page to the user's homescreen">
+        Add the app to your homescreen: <ph name="APP_ACTION">%s<ex>Add to homescreen</ex></ph>
       </message>
 
       <!-- DOM Distiller strings -->
       <message name="IDS_DISTILLATION_QUALITY_QUESTION" desc="Question to ask the user about whether the text from a web page was extracted correctly.">
         Was the text extracted correctly?
       </message>
       <message name="IDS_DISTILLATION_QUALITY_ANSWER_YES" desc="Text for the answering whether the text from a web page was extracted correctly and the answer is yes.">
         Yes
       </message>
       <message name="IDS_DISTILLATION_QUALITY_ANSWER_NO" desc="Text for the answering whether the text from a web page was extracted correctly and the answer is no.">
@@ skipping 170 matching lines @@
         Description:
       </message>
       
       <!-- Document mode strings -->
       <message name="IDS_CLOSE_ALL_INCOGNITO_NOTIFICATION" desc="Message on the notification that closes all incognito tabs in document mode">
         Close all incognito tabs
       </message>
     </messages>
   </release>
 </grit>