[MD settings] settings for pdf documents

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 e63deda26b96e0f2e1fc67adacf53262aa8fc68c..4521b5fa906bfab474048c17b669974f2c19375c 100644
--- a/chrome/app/settings_strings.grdp
+++ b/chrome/app/settings_strings.grdp
@@ -1664,6 +1664,12 @@
   <message name="IDS_SETTINGS_SITE_SETTINGS_FLASH" desc="Label for the Flash site settings.">
     Flash
   </message>
+  <message name="IDS_SETTINGS_SITE_SETTINGS_PDF_DOCUMENTS" desc="Label for the PDF documents site settings.">
+    PDF documents
+  </message>
+  <message name="IDS_SETTINGS_SITE_SETTINGS_PDF_DEFAULT_VIEWER" desc="Label for openning PDF documents using the default PDF viewer application.">

[dpapad, 2016/11/11 22:28:03]

Does this setting apply to both Chromium and Chrome?

If yes, should there be a different string for each case (in
settings_chromium_strings.grdp and settings_google_chrome_strings.grdp)?

If no, should this string be in an "<if expr=" block (and similarly within an
ifdef at md_settings_localized_strings_provider.cc)? 

[dschuyler, 2016/11/12 00:11:53]

There's another problem here. The text can give the impression that
Chrome will become the default PDF viewer (like when opening a PDF
file from the desktop, outside of the browser). That's not accurate.
I've changed the text and sent the new text to Alan for approval.
I believe the new text is good enough to go in now - I believe it's 
likely to be approved or only need a small change (which will be 
an easy CL).

+    Use Chrome as your default PDF viewer
+  </message>
   <message name="IDS_SETTINGS_SITE_SETTINGS_POPUPS" desc="Label for the popups site settings.">
     Popups
   </message>