Media control: Fix background color of the play button loaded via MHTML.

This fixes a regression by crrev.com/578281. Bug: 871132 Change-Id: If2773697618fb2d1a33bb7f9b7b48847e6d11c8d Reviewed-on: https://chromium-review.googlesource.com/1166612 Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#581823}

Media control: Fix background color of the play button loaded via MHTML.
This fixes a regression by crrev.com/578281. Bug: 871132 Change-Id: If2773697618fb2d1a33bb7f9b7b48847e6d11c8d Reviewed-on: https://chromium-review.googlesource.com/1166612 Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#581823}
0956bbd2 · Kent Tamura · Commit Bot · 068abd92 · 0956bbd2 · 0956bbd2
Commit 0956bbd2 authored Aug 09, 2018 by Kent Tamura Committed by Commit Bot Aug 09, 2018
4 changed files
--- a/third_party/WebKit/LayoutTests/.gitattributes
+++ b/third_party/WebKit/LayoutTests/.gitattributes
@@ -141,6 +141,7 @@ http/tests/security/isolatedWorld/resources/iframe.html -crlf
 http/tests/security/isolatedWorld/resources/userGestureEvents-second-window.html -crlf
 http/tests/security/isolatedWorld/userGestureEvents.html -crlf
 http/tests/security/resources/empty-svg.php -crlf
+media/controls/*.mht -crlf
 media/track/captions-webvtt/captions-multiline-lf.vtt -crlf
 mhtml/*.mht -crlf
 platform/win/fast/events/panScroll-event-fired.html -crlf

--- a/third_party/WebKit/LayoutTests/media/controls/offline-play-button-expected.html
+++ b/third_party/WebKit/LayoutTests/media/controls/offline-play-button-expected.html
+<video controls>
--- a/third_party/WebKit/LayoutTests/media/controls/offline-play-button.mht
+++ b/third_party/WebKit/LayoutTests/media/controls/offline-play-button.mht
+From: <Saved by Blink>
+Snapshot-Content-Location: data:text/html,<video controls>
+Subject: 
+Date: Wed, 8 Aug 2018 03:40:01 -0000
+MIME-Version: 1.0
+Content-Type: multipart/related;
+	type="text/html";
+	boundary="----MultipartBoundary--vMHcSo6MOQ8bNqjzscEyfel3Kzz93I3pZg8BXOlrNK----"
+
+
+------MultipartBoundary--vMHcSo6MOQ8bNqjzscEyfel3Kzz93I3pZg8BXOlrNK----
+Content-Type: text/html
+Content-ID: <frame-EABF2B3BFD6D225D86B7362F1A016751@mhtml.blink>
+Content-Transfer-Encoding: quoted-printable
+Content-Location: data:text/html,<video controls>
+
+<html><head><meta http-equiv=3D"Content-Type" content=3D"text/html; charset=
+=3Dwindows-1252"></head><body><video controls=3D""></video></body></html>
+------MultipartBoundary--vMHcSo6MOQ8bNqjzscEyfel3Kzz93I3pZg8BXOlrNK------
--- a/third_party/blink/renderer/modules/media_controls/resources/modernMediaControls.css
+++ b/third_party/blink/renderer/modules/media_controls/resources/modernMediaControls.css
@@ -383,6 +383,15 @@ video::-webkit-media-controls-overlay-play-button {
  transition: opacity 0.25s cubic-bezier(0.25, 0.1, 0.25, 1);
 }

+/**
+ * The overlay-play-button is disabled if the video element is loaded via
+ * MHTML, and a ruleset for input[type=button]:disabled in themeWin.css has
+ * higher priority than the above ruleset.
+ */
+video::-webkit-media-controls-overlay-play-button:disabled {
+  background: transparent;
+}
+
 video::-webkit-media-controls-overlay-play-button.hidden {
  opacity: 0;
  transition: opacity 0.75s cubic-bezier(0.25, 0.1, 0.25, 1);