[subresource_filter] Add Page Info subtitle on desktop + placeholder string

components/page_info_strings.grdp

Index: components/page_info_strings.grdp
diff --git a/components/page_info_strings.grdp b/components/page_info_strings.grdp
index f1af7c44018c98eeb738cc6587cd787780207a36..39d444fb4ad7c3227c67d6ee316852f7e01194a1 100644
--- a/components/page_info_strings.grdp
+++ b/components/page_info_strings.grdp
@@ -296,6 +296,9 @@
     <message name="IDS_PAGE_INFO_PERMISSION_AUTOMATICALLY_BLOCKED" desc="The label used underneath a permission listed in the Page Info bubble if the permission was blocked by Chrome on behalf of the user.">
       Automatically blocked
     </message>
+    <message name="IDS_PAGE_INFO_PERMISSION_SUBRESOURCE_FILTER_SUBTITLE" desc="The label used underneath the subresource filter permission in the Page Info UI. Used on desktop and Android platforms" translateable="false" formatter_data="android_java">

[lgarron, 2017/05/17 20:38:11]

Could you remove the translateable="false", like with
https://codereview.chromium.org/2884813003/?

I'm not familiar with formatter_data="android_java"; does that do something
useful here?

[Charlie Harrison, 2017/05/17 20:50:13]

I don't think we should remove translateable=false for strings that are not
identical to existing strings. I was told by translation folks to add that for
placeholder strings that aren't really translateable.

formatter_data="android_java" makes it possible to share the string on Android
so it is accessible with R.string...

+       Subresource filter activated (subtitle)

[lgarron, 2017/05/17 20:38:11]

Nit: remove "(subtitle)", assuming that's meant as a placeholder.

[Charlie Harrison, 2017/05/17 20:50:13]

Done.

+    </message>
 
     <!-- Permission change infobar. -->
     <if expr="not is_android">