Fix the overlay play button position on android
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Feb 2013 23:21:16 +0000 (23:21 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Feb 2013 23:21:16 +0000 (23:21 +0000)
https://bugs.webkit.org/show_bug.cgi?id=110638

Patch by Min Qin <qinmin@chromium.org> on 2013-02-22
Reviewed by Ojan Vafai.

Because we are using flex box, specifying the height is not necessary.
Setting height to 100% will make the enclosure go outside the control panel.
There are no media layout test expecations for android, will do that in another patch.

* css/mediaControlsChromiumAndroid.css:
(video::-webkit-media-controls-overlay-enclosure):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@143809 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/css/mediaControlsChromiumAndroid.css

index e6adf7b..2636f94 100644 (file)
@@ -1,3 +1,17 @@
+2013-02-22  Min Qin  <qinmin@chromium.org>
+
+        Fix the overlay play button position on android
+        https://bugs.webkit.org/show_bug.cgi?id=110638
+
+        Reviewed by Ojan Vafai.
+
+        Because we are using flex box, specifying the height is not necessary.
+        Setting height to 100% will make the enclosure go outside the control panel.
+        There are no media layout test expecations for android, will do that in another patch.
+
+        * css/mediaControlsChromiumAndroid.css:
+        (video::-webkit-media-controls-overlay-enclosure):
+
 2013-02-22  Kentaro Hara  <haraken@chromium.org>
 
         Unreviewed build fix. Added missing #if PLATFORM(CHROMIUM).
 2013-02-22  Kentaro Hara  <haraken@chromium.org>
 
         Unreviewed build fix. Added missing #if PLATFORM(CHROMIUM).
index e8d3389..3e99d8a 100644 (file)
@@ -86,7 +86,6 @@ video::-webkit-media-controls-overlay-enclosure {
     -webkit-align-items: center;
     -webkit-flex: 1 1;
     width: 100%;
     -webkit-align-items: center;
     -webkit-flex: 1 1;
     width: 100%;
-    height: 100%;
     max-width: 800px;
     text-indent: 0;
     box-sizing: border-box;
     max-width: 800px;
     text-indent: 0;
     box-sizing: border-box;