MD Settings: Make Protected Content setting available to all platforms.

MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

This also updates the string to the new text.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=687783, 686886
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2677003004
Cr-Commit-Position: refs/heads/master@{#448680}
Committed: https://chromium.googlesource.com/chromium/src/+/45f66398546e5f402fb9768e05c1321893dbee6b

[tommycli, 2017-02-06 23:59:03]

Description was changed from

==========
MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=686430
==========

to

==========
MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=686430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tommycli, 2017-02-06 23:59:07]

The CQ bit was checked by tommycli@chromium.org to run a CQ dry run

[tommycli, 2017-02-06 23:59:31]

Description was changed from

==========
MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=686430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

This also updates the string to the new text.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=686430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2017-02-06 23:59:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tommycli, 2017-02-07 00:18:17]

tommycli@chromium.org changed reviewers:
+ stevenjb@chromium.org

[tommycli, 2017-02-07 00:18:18]

stevenjb: PTAL, thanks!

[stevenjb, 2017-02-07 00:22:52]

lgtm

[tommycli, 2017-02-07 00:33:54]

The CQ bit was unchecked by tommycli@chromium.org

[tommycli, 2017-02-07 00:34:01]

On 2017/02/07 00:22:52, stevenjb wrote:
> lgtm

thanks!

[tommycli, 2017-02-07 00:34:04]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2017-02-07 00:34:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-07 01:26:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-07 01:26:44]

Failed to apply patch for
chrome/browser/resources/settings/privacy_page/privacy_page.html:
While running git apply --index -p1;
  error: patch failed:
chrome/browser/resources/settings/privacy_page/privacy_page.html:449
  error: chrome/browser/resources/settings/privacy_page/privacy_page.html: patch
does not apply

Patch:       chrome/browser/resources/settings/privacy_page/privacy_page.html
Index: chrome/browser/resources/settings/privacy_page/privacy_page.html
diff --git a/chrome/browser/resources/settings/privacy_page/privacy_page.html
b/chrome/browser/resources/settings/privacy_page/privacy_page.html
index
3bb8aeee8b9260893538907b88be04ec25ea5ff5..633d475be82583eb66e447b3f0a835f278e4636d
100644
--- a/chrome/browser/resources/settings/privacy_page/privacy_page.html
+++ b/chrome/browser/resources/settings/privacy_page/privacy_page.html
@@ -438,7 +438,6 @@
           </site-data-details-subpage>
         </settings-subpage>
       </template>
-<if expr="chromeos or is_win">
       <template is="dom-if" route-path="/content/protectedContent" no-search>
         <settings-subpage page-title="$i18n{siteSettingsProtectedContent}">
           <div class="settings-box first">
@@ -449,7 +448,6 @@
           </div>
         </settings-subpage>
       </template>
-</if>
     </settings-animated-pages>
   </template>
   <script src="privacy_page.js"></script>

[tommycli, 2017-02-07 17:46:54]

The CQ bit was checked by tommycli@chromium.org

[tommycli, 2017-02-07 17:46:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
stevenjb@chromium.org
Link to the patchset: https://codereview.chromium.org/2677003004/#ps20001
(title: "Merge branch 'master' of https://chromium.googlesource.com/chromium/src
into 399-settings-make-enab…")

[commit-bot: I haz the power, 2017-02-07 17:47:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-07 18:53:33]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486489614150020,
"parent_rev": "33480670615a7a5229e9c16cd0ccc641143380a9", "commit_rev":
"45f66398546e5f402fb9768e05c1321893dbee6b"}

[commit-bot: I haz the power, 2017-02-07 18:55:13]

Description was changed from

==========
MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

This also updates the string to the new text.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=686430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

This also updates the string to the new text.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=686430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2677003004
Cr-Commit-Position: refs/heads/master@{#448680}
Committed:
https://chromium.googlesource.com/chromium/src/+/45f66398546e5f402fb9768e05c1...
==========

[commit-bot: I haz the power, 2017-02-07 18:55:14]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/45f66398546e5f402fb9768e05c1...

[Will Harris, 2017-02-07 18:59:12]

the bug for this is crbug.com/686886 I think?

[tommycli, 2017-02-07 19:40:29]

Description was changed from

==========
MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

This also updates the string to the new text.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=686430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2677003004
Cr-Commit-Position: refs/heads/master@{#448680}
Committed:
https://chromium.googlesource.com/chromium/src/+/45f66398546e5f402fb9768e05c1...
==========

to

==========
MD Settings: Make Protected Content setting available to all platforms.

The Protected Content setting has been re-designed to toggle DRM on all
platforms. This makes the setting available on all platforms.

This also updates the string to the new text.

MD Settings equivalent to this patch:
https://codereview.chromium.org/2679723003/

BUG=687783,686886
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2677003004
Cr-Commit-Position: refs/heads/master@{#448680}
Committed:
https://chromium.googlesource.com/chromium/src/+/45f66398546e5f402fb9768e05c1...
==========

[tommycli, 2017-02-07 19:40:40]

On 2017/02/07 18:59:12, Will Harris wrote:
> the bug for this is crbug.com/686886 I think?

Ah, that's right. I updated the bug numbers.