Add Help & feedback screens to Android Chromoting client

remoting/resources/remoting_strings.grd

Index: remoting/resources/remoting_strings.grd
diff --git a/remoting/resources/remoting_strings.grd b/remoting/resources/remoting_strings.grd
index 48cd896d8ec9fbfa5cc716529fe65b0f36ea4452..9b1bc3d2a35561bccbd370331782b7874c24b95d 100644
--- a/remoting/resources/remoting_strings.grd
+++ b/remoting/resources/remoting_strings.grd
@@ -378,9 +378,18 @@
       </if>
 
       <if expr="is_android">
-        <message desc="Text displayed in the Android action-bar overflow menu for showing the Help and Feedback screen" name="IDS_ACTIONBAR_HELP" formatter_data="android_java">
+        <message desc="Title displayed in the action-bar for the Android Help screen" name="IDS_ACTIONBAR_HELP_TITLE" formatter_data="android_java">

[Jamie, 2014/02/25 22:01:18]

Make it clear in the description that this is a verb; e.g, append something
similar to what you have for the Feedback string: "Pressing this button gives
the user help on the current screen of the app."

[Lambros, 2014/02/26 21:15:45]

It's the title of the Help & feedback screen - just the word "Help".

+          Help
+        </message>
+        <message desc="Android action-bar menu item for showing the Help and Feedback screen" name="IDS_ACTIONBAR_HELP" formatter_data="android_java">
           Help &amp; feedback

[Jamie, 2014/02/25 22:01:18]

Do we still need this one?

[Lambros, 2014/02/26 21:15:45]

Yes, this is unchanged.

         </message>
+        <message desc="Label for the Feedback button displayed in the Android Help screen. Pressing this button causes the Feedback screen to be shown." name="IDS_ACTIONBAR_FEEDBACK" formatter_data="android_java">
+          Feedback
+        </message>
+        <message desc="Android action-bar menu item for viewing the Play Store page for the application" name="IDS_ACTIONBAR_PLAY_STORE" formatter_data="android_java">
+          View in Google Play Store

[Jamie, 2014/02/25 22:01:18]

Was this feature mandated by the UX review? It seems non-standard.

[Lambros, 2014/02/26 21:15:45]

Yes, I think it's new. Haven't seen it in other apps, but it's described in our
guidelines.

+        </message>
       </if>
 
       <message desc="Label for the access code entry box. This is where the client user enters the code that permits access to the host." name="IDS_ACCESS_CODE">