Quick fix for rendering controls regression.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 25 Mar 2012 03:59:40 +0000 (03:59 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 25 Mar 2012 03:59:40 +0000 (03:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=82125

Patch by Victor Carbune <vcarbune@adobe.com> on 2012-03-24
Reviewed by Eric Carlson.

Only temporary fix. Tests to cover the problem should be added

* html/shadow/MediaControlElements.cpp:
(WebCore::MediaControlPanelElement::makeOpaque): The display:none
property should be toggled only by the media element now.
(WebCore::MediaControlPanelElement::makeTransparent): The timer
that toggles the dispalay:none shouldn't be used now.

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

Source/WebCore/ChangeLog
Source/WebCore/html/shadow/MediaControlElements.cpp

index a5f49e8..a68bab3 100644 (file)
@@ -1,3 +1,18 @@
+2012-03-24  Victor Carbune  <vcarbune@adobe.com>
+
+        Quick fix for rendering controls regression.
+        https://bugs.webkit.org/show_bug.cgi?id=82125
+
+        Reviewed by Eric Carlson.
+
+        Only temporary fix. Tests to cover the problem should be added
+
+        * html/shadow/MediaControlElements.cpp:
+        (WebCore::MediaControlPanelElement::makeOpaque): The display:none
+        property should be toggled only by the media element now.
+        (WebCore::MediaControlPanelElement::makeTransparent): The timer
+        that toggles the dispalay:none shouldn't be used now.
+
 2012-03-24  Kevin Ollivier  <kevino@theolliviers.com>
 
         [wx] Unreviewed. Add header needed for compilation.
index 6059f4e..0f34538 100644 (file)
@@ -242,7 +242,9 @@ void MediaControlPanelElement::makeOpaque()
 
     m_opaque = true;
 
-    show();
+    // FIXME(BUG 79347): The display:none property should be toggled below only
+    // when display logic is introduced.
+    // show();
 }
 
 void MediaControlPanelElement::makeTransparent()
@@ -256,7 +258,9 @@ void MediaControlPanelElement::makeTransparent()
 
     m_opaque = false;
 
-    startTimer();
+    // FIXME(BUG 79347): The display:none property should be toggled below
+    // (through the timer start) when display logic is introduced.
+    // startTimer();
 }
 
 void MediaControlPanelElement::defaultEventHandler(Event* event)