Enhancements to https://codereview.chromium.org/12494033/ for zmo@

chrome/app/chromium_strings.grd

 <?xml version="1.0" encoding="UTF-8"?>
 <!-- This file contains definitions of strings that are distribution specific.
 These strings will not be translated and only english versions will
 be available for now. -->
 
 <grit base_dir="." latest_public_release="0" current_release="1"
       source_lang_id="en" enc_check="möl">
   <outputs>
     <output filename="grit/chromium_strings.h" type="rc_header">
       <emit emit_type='prepend'></emit>
@@ skipping 409 matching lines @@
       </if>
       <if expr="not pp_ifdef('use_titlecase')">
         <message name="IDS_STATUS_TRAY_KEEP_CHROME_RUNNING_IN_BACKGROUND" desc="The checkbox in the status tray context menu that controls whether chrome keeps running in the background after the last window is closed">
           Let Chromium run in the background
         </message>
       </if>
       <message name="IDS_OPTIONS_SYSTEM_ENABLE_BACKGROUND_MODE" desc="The label of the check-box that enables background mode">
         Continue running background apps when Chromium is closed
       </message>
       <message name="IDS_OPTIONS_SYSTEM_ENABLE_HARDWARE_ACCELERATION_MODE" desc="The label of the check-box that enables hardware acceleration mode">
-        Use hardware acceleration when available (requires Chromium restart)
+        Use hardware acceleration when available
+      </message>
+      <message name="IDS_OPTIONS_SYSTEM_ENABLE_HARDWARE_ACCELERATION_MODE_RESTART" desc="The restart string after changing hardware acceleration mode">
+        (requires Chromium <ph name="BEGIN_LINK">&lt;a id="gpu-mode-reset-restart-link" href="#"&gt;</ph>restart<ph name="END_LINK">&lt;/a&gt;</ph>)

[Dan Beam, 2013/03/22 00:32:56]

looking the patch, perhaps rewording to "click here to restart chrom{e,ium}"
would be better as (or anything to make it more clear the user *could* lose some
potential work from restarting chrome)

[arv (Not doing code reviews), 2013/03/22 14:35:48]

In general, it is bad to put HTML in here. Can you rethink your solution?

[arv (Not doing code reviews), 2013/03/22 14:35:48]

This should not be a link. It should be a button since opening this in a new tab
or bookmarking it makes no sense.

[Dan Beam, 2013/03/22 19:24:31]

I agree on the link, but I don't agree on the HTML point as previous zmo@ had 3
different translations (i.e. prefix, link, suffix) which doesn't work well.

[Dan Beam, 2013/03/22 19:25:16]

Is there a better way to do

  Start translation <link in middle> end translation.

?

       </message>
       <message name="IDS_CANT_WRITE_USER_DIRECTORY_SUMMARY" desc="Summary of problem displayed in dialog when we can't create a directory for this user.">
 Chromium cannot read and write to its data directory:
 
 <ph name="USER_DATA_DIRECTORY">$1<ex>C:\Documents and Settings\devint\Local Settings\Application Data\Google\Chrome</ex></ph>
       </message>
       <message name="IDS_EULA_CHECKBOX_ENABLE_LOGGING" desc="The label of the checkbox to enable/disable crash and user metrics logging">
         Optional: Help make Chromium better by automatically sending usage statistics and crash reports to Google.
       </message>
       <message name="IDS_PROFILE_TOO_NEW_ERROR" desc="Error displayed on startup when the profile is from a newer version of the product and can not be read">
@@ skipping 518 matching lines @@
 
       <!-- Chrome OS OOBE Terms of Service screen-->
       <if expr="pp_ifdef('chromeos')">
         <message name="IDS_TERMS_OF_SERVICE_SCREEN_SUBHEADING" desc="Subheading at the top of the Terms of Service screen.">
           <ph name="DOMAIN">$1<ex>example.com</ex></ph> requires that you read and accept the following Terms of Service before using this device. These terms do not expand, modify or limit the Chromium OS Terms.
         </message>
       </if>
     </messages>
   </release>
 </grit>