Title: [120554] trunk/Source/WebCore
Revision
120554
Author
[email protected]
Date
2012-06-17 16:06:12 -0700 (Sun, 17 Jun 2012)

Log Message

REGRESSION (r111041): Missing element type check in RenderThemeMac::paintMediaFullscreenButton
https://bugs.webkit.org/show_bug.cgi?id=89270

Reviewed by Oliver Hunt.

* rendering/RenderThemeMac.mm:
(WebCore::RenderThemeMac::paintMediaFullscreenButton): Use the proper idiom for getting
a media control element's type.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (120553 => 120554)


--- trunk/Source/WebCore/ChangeLog	2012-06-17 21:19:15 UTC (rev 120553)
+++ trunk/Source/WebCore/ChangeLog	2012-06-17 23:06:12 UTC (rev 120554)
@@ -1,3 +1,14 @@
+2012-06-15  Darin Adler  <[email protected]>
+
+        REGRESSION (r111041): Missing element type check in RenderThemeMac::paintMediaFullscreenButton
+        https://bugs.webkit.org/show_bug.cgi?id=89270
+
+        Reviewed by Oliver Hunt.
+
+        * rendering/RenderThemeMac.mm:
+        (WebCore::RenderThemeMac::paintMediaFullscreenButton): Use the proper idiom for getting
+        a media control element's type.
+
 2012-06-17  Zeev Lieber  <[email protected]>
 
         [Chromium] Compositor should avoid drawing quads when cached textures are available and contents unchanged

Modified: trunk/Source/WebCore/rendering/RenderThemeMac.mm (120553 => 120554)


--- trunk/Source/WebCore/rendering/RenderThemeMac.mm	2012-06-17 21:19:15 UTC (rev 120553)
+++ trunk/Source/WebCore/rendering/RenderThemeMac.mm	2012-06-17 23:06:12 UTC (rev 120554)
@@ -1833,9 +1833,9 @@
     if (!node)
         return false;
 
-    if (MediaControlFullscreenButtonElement* btn = static_cast<MediaControlFullscreenButtonElement*>(o->node())) {
+    if (node->isMediaControlElement()) {
         LocalCurrentGraphicsContext localContext(paintInfo.context);
-        wkDrawMediaUIPart(btn->displayType(), mediaControllerTheme(), localContext.cgContext(), r, getMediaUIPartStateFlags(node));
+        wkDrawMediaUIPart(mediaControlElementType(node), mediaControllerTheme(), localContext.cgContext(), r, getMediaUIPartStateFlags(node));
     }
     return false;
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to