Desktop Capture Picker New UI: Non Mac Structure Change

chrome/app/google_chrome_strings.grd

Index: chrome/app/google_chrome_strings.grd
diff --git a/chrome/app/google_chrome_strings.grd b/chrome/app/google_chrome_strings.grd
index a9edbad423538ca587366326f126f84edd1dffb0..a01e5985debc64ec6e406b71bed974539b3e4199 100644
--- a/chrome/app/google_chrome_strings.grd
+++ b/chrome/app/google_chrome_strings.grd
@@ -1124,6 +1124,11 @@ Signing in anyway will merge Chrome information like bookmarks, history, and oth
       <message name="IDS_PAGE_INFO_SECURITY_TAB_SECURE_IDENTITY_CT_INVALID" desc="The text of the identity section when the page is secure, but it contains a Signed Certificate Timestamp from a known Certificate Transparency log that failed to verify.">
         Chrome verified that <ph name="ISSUER">$1<ex>VeriSign</ex></ph> issued this website's certificate. The server supplied Certificate Transparency information, but it was invalid.
       </message>
+      
+      <!-- Desktop Capture -->
+      <message name="IDS_DESKTOP_MEDIA_PICKER_SOURCE_TYPE_TAB" desc="Text for the button on the window picker dialog, clicking which takes one to tab sharing">
+        Chromium Tab

[msw, 2016/05/02 23:49:57]

Shouldn't this be "Chrome Tab"? Please just add a single shared string with a
"<ph name="PRODUCT_NAME">$1<ex>Chrome</ex></ph>" field for IDS_PRODUCT_NAME,
like eg. IDS_SYNC_UPGRADE_CLIENT.

[qiangchen, 2016/05/05 22:17:53]

That way the "git cl upload" thinks it is an error and tells me to do like this.
Maybe we changed the regulation at some time? Theoretically this way is more
translator friendly.

[msw, 2016/05/05 22:59:49]

That doesn't make sense to me. What is the git cl upload error?

[qiangchen, 2016/05/05 23:36:26]

Don't use PRODUCT_NAME placeholders in string resources. Instead, add separate
strings to google_chrome_strings.grd and chromium_strings.grd. See
http://goo.gl/6614MQ for more information.Problems with this check? Contact
dubroy@chromium.org.

The reason is for translation.

+      </message>
 
       <!-- Runtime permission strings -->
       <if expr="is_android">