Adding a "Learn more" link to the Google DNS selection

chrome/app/chromeos_strings.grdp

 <?xml version="1.0" encoding="utf-8"?>
 <!-- ChromeOS-specific strings (included from generated_resources.grd).
      Everything in this file is wrapped in <if expr="pp_ifdef('chromeos')">. -->
 <grit-part>
   <!-- Idle Logout Dialog Strings -->
   <message name="IDS_IDLE_LOGOUT_TITLE" desc="Dialog title for the idle logout dialog">
     Are you still there?
   </message>
   <message name="IDS_IDLE_LOGOUT_WARNING_RESTART" desc="Restart label for the idle logout dialog">
     This computer will reset in <ph name="LOGOUT_TIME_LEFT">$1<ex>20</ex></ph> seconds.
@@ skipping 1190 matching lines @@
   <message name="IDS_OPTIONS_SETTINGS_INTERNET_CONNECTION_STATE" desc="In settings Internet options, the label for connection status.">
     Connection status:
   </message>
   <message name="IDS_OPTIONS_SETTINGS_INTERNET_IP_AUTOMATIC_CONFIGURATION" desc="In the IP address fields, labels automatic connection toggle">
     Configure automatically
   </message>
   <message name="IDS_OPTIONS_SETTINGS_INTERNET_AUTOMATIC_NAME_SERVERS" desc="In the name server type radio for static ip configuration, labels the automatic DNS selection.">
     Automatic name servers
   </message>
   <message name="IDS_OPTIONS_SETTINGS_INTERNET_GOOGLE_NAME_SERVERS" desc="In the name server type radio for static ip configuration, labels the Google DNS selection.">
-    Google name servers
+    Google name servers - <ph name="BEGIN_LINK">&lt;a href="https://developers.google.com/speed/public-dns" target="_blank"&gt;</ph>Learn more<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph>

[pneubeck (no reviews), 2013/03/20 09:09:02]

you don't have an example for the first <ph>
better make it consistent and drop it from the second <ph> too.

Alternative which I prefer:
make only the "Learn more" translatable with another <message>
Would be much more readable and wouldn't "misuse" the <ph>.

[Greg Spencer (Chromium), 2013/03/20 17:31:03]

Good point, done.  The <ex> was pretty useless there anyhow.
Well, but I don't want to do that because another language might make it hard to
substitute that.  The surrounding text might have to be a vague tense or gender
or something to account for the fact that the substituted message might contain
anything.  I'd rather have the entire string in one message so that the
translators get the entire context at once.

   </message>
   <message name="IDS_OPTIONS_SETTINGS_INTERNET_USER_NAME_SERVERS" desc="In the name server type radio for static ip configuration, labels the user DNS selection.">
     Custom name servers
   </message>
   <message name="IDS_OPTIONS_SETTINGS_INTERNET_USER_NAME_SERVER_1" desc="In the name server type radio for static ip configuration, is placeholder text for server 1">
     Server 1
   </message>
   <message name="IDS_OPTIONS_SETTINGS_INTERNET_USER_NAME_SERVER_2" desc="In the name server type radio for static ip configuration, is placeholder text for server 2">
     Server 2
   </message>
@@ skipping 2322 matching lines @@
   <message name="IDS_LOGIN_PUBLIC_ACCOUNT_SIGNOUT_REMINDER" desc="Text shown in the public account user pod, reminding the user to log out.">
     Your information will be removed when you sign out.
   </message>
   <message name="IDS_LOGIN_PUBLIC_ACCOUNT_ENTER" desc="Label text for the sign-in button in the public account user pod.">
     Enter
   </message>
   <message name="IDS_LOGIN_PUBLIC_ACCOUNT_ENTER_ACCESSIBLE_NAME" desc="Text to be spoken when focus is set to the sign-in button in the public account user pod.">
     Enter public session
   </message>
 </grit-part>