MD Settings: Google Play Store: Add subpage and polish

chrome/app/settings_strings.grdp

Index: chrome/app/settings_strings.grdp
diff --git a/chrome/app/settings_strings.grdp b/chrome/app/settings_strings.grdp
index 5d79e46473d4165717cb85d1be79cb21ae49cbe4..656de0f3ab01deb488d9f5ba2cfc7c1f6abc6b8a 100644
--- a/chrome/app/settings_strings.grdp
+++ b/chrome/app/settings_strings.grdp
@@ -211,11 +211,17 @@
     <message name="IDS_SETTINGS_ANDROID_APPS_TITLE" desc="The title of Google Play Store (Arc++ / Android Apps) section.">
       Google Play Store (beta)
     </message>
-    <message name="IDS_SETTINGS_ANDROID_APPS_ENABLE" desc="Label for the checkbox that enables Google Play Store (Arc++) apps.">
-      Enable Google Play Store on your Chromebook.
+    <message name="IDS_SETTINGS_ANDROID_APPS_ENABLE" desc="Label for the button that enables Google Play Store (Arc++) apps.">
+      Turn on
+    </message>
+    <message name="IDS_SETTINGS_ANDROID_APPS_SUBTEXT" desc="Description for the section for enabling and managing Google Play Store (Android) apps.">
+      Install apps and games from Google Play. <ph name="BEGIN_LINK">&lt;a target="_blank" href="$1"&gt;</ph>Learn more<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph>

[Dan Beam, 2017/03/30 13:24:22]

these <ph> are a little odd to me.  can we just make one for the URL instead? 
that's the only real replacement that's happening here, as far as I can tell

Install apps and games from Google Play. &lt;a target="_blank" href="<ph
name="URL">$1<ex>https://google.com/</ex></ph>"&gt;Learn more&lt;/a&gt;

[stevenjb, 2017/03/30 23:05:36]

I agree this looks better. FWIW, this is the pattern we use everywhere else
except for one example in Downloads added by you :). I'm fine with changing the
pattern, but there are dozen other examples in this file that we should consider
cleaning up also.

[Dan Beam, 2017/03/30 23:35:19]

people will keep copy pasting the ones that don't make much sense until they're
cleaned up, so sgtm

     </message>
     <message name="IDS_SETTINGS_ANDROID_APPS_MANAGE_APPS" desc="Label for launching Android apps settings.">
-      Manage your Android preferences
+      Manage Android preferences
+    </message>
+    <message name="IDS_SETTINGS_ANDROID_APPS_REMOVE" desc="Label for the control to open a dialog confirming removal of the Google Play Store.">
+      Remove Google Play Store
     </message>
     <message name="IDS_SETTINGS_ANDROID_APPS_LEARN_MORE" desc="Link for more informaiton about installing Android apps on Chrome OS.">
       Learn more
@@ -225,11 +231,9 @@
       Remove Android apps?
     </message>
     <message name="IDS_SETTINGS_ANDROID_APPS_DISABLE_DIALOG_MESSAGE" desc="Describes what will happen if the user opts out of android apps.">
-      Apps you’ve downloaded from Google Play will be deleted from this Chromebook.
-      <ph name="LINE_BREAKS1">&lt;br&gt;&lt;br&gt;</ph>
-      Content you’ve purchased such as movies, TV shows, music, books, or other in-app purchases may also be deleted.
-      <ph name="LINE_BREAKS2">&lt;br&gt;&lt;br&gt;</ph>
-      This doesn’t affect apps or content on other devices.
+      Apps and content from Google Play will be deleted from your Chromebook.
+      This only affects apps and content on this device.
+      <ph name="BEGIN_LINK">&lt;a target="_blank" href="$1"&gt;</ph>Learn more<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph>

[Dan Beam, 2017/03/30 13:24:22]

same note regarding <ph>, I don't think they're that useful with HTML in them

[stevenjb, 2017/03/30 23:05:36]

Done.

     </message>
   </if>