Title: [147486] trunk/Source/WebCore
Revision
147486
Author
[email protected]
Date
2013-04-02 12:53:35 -0700 (Tue, 02 Apr 2013)

Log Message

Fix css style for CC button on android
https://bugs.webkit.org/show_bug.cgi?id=113815

Patch by Min Qin <[email protected]> on 2013-04-02
Reviewed by Adam Barth.

The default CC button size is 16px as defined in mediaControls.css. Android should use 35px.
This change fixed that problem.
Also fixed a problem that r142947 does not change the display of fullscreen button to -webkit-flex
Android does not run media layout tests currently. Will fix the test expectations when layout tests are run in content shell.

* css/mediaControlsChromiumAndroid.css:
(video::-webkit-media-controls-fullscreen-button):
(audio::-webkit-media-controls-toggle-closed-captions-button, video::-webkit-media-controls-toggle-closed-captions-button):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (147485 => 147486)


--- trunk/Source/WebCore/ChangeLog	2013-04-02 19:52:22 UTC (rev 147485)
+++ trunk/Source/WebCore/ChangeLog	2013-04-02 19:53:35 UTC (rev 147486)
@@ -1,3 +1,19 @@
+2013-04-02  Min Qin  <[email protected]>
+
+        Fix css style for CC button on android
+        https://bugs.webkit.org/show_bug.cgi?id=113815
+
+        Reviewed by Adam Barth.
+
+        The default CC button size is 16px as defined in mediaControls.css. Android should use 35px.
+        This change fixed that problem.
+        Also fixed a problem that r142947 does not change the display of fullscreen button to -webkit-flex
+        Android does not run media layout tests currently. Will fix the test expectations when layout tests are run in content shell.
+
+        * css/mediaControlsChromiumAndroid.css:
+        (video::-webkit-media-controls-fullscreen-button):
+        (audio::-webkit-media-controls-toggle-closed-captions-button, video::-webkit-media-controls-toggle-closed-captions-button):
+
 2013-04-02  Alexey Proskuryakov  <[email protected]>
 
         [Mac] Remove WKCreateNSURLConnectionDelegateProxy

Modified: trunk/Source/WebCore/css/mediaControlsChromiumAndroid.css (147485 => 147486)


--- trunk/Source/WebCore/css/mediaControlsChromiumAndroid.css	2013-04-02 19:52:22 UTC (rev 147485)
+++ trunk/Source/WebCore/css/mediaControlsChromiumAndroid.css	2013-04-02 19:53:35 UTC (rev 147486)
@@ -161,7 +161,7 @@
 
 video::-webkit-media-controls-fullscreen-button {
     -webkit-appearance: media-enter-fullscreen-button;
-    display: inline;
+    display: -webkit-flex;
     border: none;
     box-sizing: border-box;
     width: 35px;
@@ -172,6 +172,19 @@
     padding: 0;
 }
 
+audio::-webkit-media-controls-toggle-closed-captions-button, video::-webkit-media-controls-toggle-closed-captions-button {
+    -webkit-appearance: media-toggle-closed-captions-button;
+    display: -webkit-flex;
+    border: none;
+    box-sizing: border-box;
+    width: 35px;
+    height: 35px;
+    line-height: 35px;
+    margin-left: -5px;
+    margin-right: 9px;
+    padding: 0;
+}
+
 input[type="range"]::-webkit-media-slider-container {
     display: -webkit-flex;
     -webkit-align-items: center;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to