MD Settings: Update Protected Content for redesign

chrome/browser/resources/settings/privacy_page/privacy_page.html

 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/cr_elements/policy/cr_policy_vars_css.html">
 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout-classes.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/neon-animation/neon-animatable.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button-light.html">
 <link rel="import" href="../clear_browsing_data_dialog/clear_browsing_data_dialog.html">
 <link rel="import" href="../controls/settings_toggle_button.html">
 <link rel="import" href="../lifetime_browser_proxy.html">
@@ skipping 400 matching lines @@
           <paper-button class="subpage-title-extra"
               on-tap="onRemoveAllCookiesFromSite_">
             $i18n{siteSettingsCookieRemoveAll}
           </paper-button>
           <site-data-details-subpage page-title="{{pageTitle}}">
           </site-data-details-subpage>
         </settings-subpage>
       </template>
       <template is="dom-if" route-path="/content/protectedContent" no-search>
         <settings-subpage page-title="$i18n{siteSettingsProtectedContent}">
+          <div class="settings-box first">
+            <!-- TODO(dschuyler): change i18n to i18nStr. -->
+            <settings-toggle-button class="start"
+                pref="{{prefs.webkit.webprefs.encrypted_media_enabled}}"
+                label="[[getStringTernary_(
+                    prefs.webkit.webprefs.encrypted_media_enabled.value,
+                    '$i18n{siteSettingsProtectedContentEnable}',
+                    '$i18n{siteSettingsBlocked}')]]">

[Dan Beam, 2017/02/10 17:23:53]

this will break on '

[tommycli, 2017/02/10 18:46:31]

+dschuyler

Indeed this does break on '. I thought we escaped that, but we only escape
double-quotes. Does it seem worth it to additionally escape this in
i18n_source_stream? If not, we probably need to abandon all usages of this
pattern in Settings. (There are many in site_settings_page.)

[dschuyler, 2017/02/10 19:07:58]

A CL to fix it is here: https://codereview.chromium.org/2681623002/
And a doc discussing it further is here:
https://docs.google.com/document/d/1qWU0tJfLbmE0clsCdLUy7etk3Of8hUb8qG87tYKM6...

[tommycli, 2017/02/10 19:17:00]

Hey thanks, that explains things.

In PS4, I've converted the code to use JavaScript i18n as stevenjb recommended
earlier. This should allow us to get unblocked and fix the protected content. We
can convert back to the ternary expression later once i18nStr is landed.

+            </settings-toggle-button>
+          </div>
+<if expr="chromeos or is_win">
           <div class="settings-box first two-line">
-            $i18n{siteSettingsProtectedContentExplanation}
+            $i18n{siteSettingsProtectedContentIdentifiersExplanation}
           </div>
           <div class="settings-box first">
             <!-- TODO(dschuyler): change i18n to i18nStr. -->
             <settings-toggle-button class="start"
                 pref="{{prefs.settings.privacy.drm_enabled}}"
                 label="[[getStringTernary_(
                     prefs.settings.privacy.drm_enabled.value,
-                    '$i18n{siteSettingsProtectedContentEnable}',
+                    '$i18n{siteSettingsProtectedContentEnableIdentifiers}',
                     '$i18n{siteSettingsBlocked}')]]">
             </settings-toggle-button>
           </div>
+</if>
 <if expr="chromeos">
-          <category-setting-exceptions
-              category="{{ContentSettingsTypes.PROTECTED_CONTENT}}">
-          </category-setting-exceptions>
+          <template is="dom-if"
+              if="[[prefs.settings.privacy.drm_enabled.value]]">
+            <category-setting-exceptions
+                category="{{ContentSettingsTypes.PROTECTED_CONTENT}}">
+            </category-setting-exceptions>
+          </template>
 </if>
         </settings-subpage>
       </template>
     </settings-animated-pages>
   </template>
   <script src="privacy_page.js"></script>
 </dom-module>