Changeset 83445 in webkit

Timestamp:

Apr 11, 2011 9:32:36 AM (13 years ago)

Author:

Dimitri Glazkov

Message:

2011-04-11 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Martin Robinson.

REGRESSION(r83397): Most GTK media controls are hidden when no valid source is specified
​https://bugs.webkit.org/show_bug.cgi?id=58204

Covered by existing tests.

• platform/gtk/RenderThemeGtk.h: (WebCore::RenderThemeGtk::hasOwnDisabledStateHandlingFor): Let GTK handle

the the state of media controls.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/gtk/RenderThemeGtk.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-04-11  Dimitri Glazkov  <dglazkov@chromium.org>
+
+        Reviewed by Martin Robinson.
+
+        REGRESSION(r83397): Most GTK media controls are hidden when no valid source is specified
+        https://bugs.webkit.org/show_bug.cgi?id=58204
+
+        Covered by existing tests.
+
+        * platform/gtk/RenderThemeGtk.h:
+        (WebCore::RenderThemeGtk::hasOwnDisabledStateHandlingFor): Let GTK handle
+            the the state of media controls.
+
 2011-04-11  Dimitri Glazkov  <dglazkov@chromium.org>
 

trunk/Source/WebCore/platform/gtk/RenderThemeGtk.h

@@ -148 +148 @@
     void initMediaButtons();
     void adjustMediaSliderThumbSize(RenderObject*) const;
+    virtual bool hasOwnDisabledStateHandlingFor(ControlPart) const { return true; }
     virtual bool paintMediaFullscreenButton(RenderObject*, const PaintInfo&, const IntRect&);
     virtual bool paintMediaPlayButton(RenderObject*, const PaintInfo&, const IntRect&);