REGRESSION(r173441): [GTK] All buttons appear insensitive
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 12 Sep 2014 15:44:42 +0000 (15:44 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 12 Sep 2014 15:44:42 +0000 (15:44 +0000)
https://bugs.webkit.org/show_bug.cgi?id=136614

Reviewed by Sergio Villar Senin.

Do not use RenderTheme::isReadOnlyControl() to decide whether to
render non text based forms sensitive or unsensitive, isEnabled()
is enough in those cases.

* rendering/RenderThemeGtk.cpp:
(WebCore::paintToggle):
(WebCore::renderButton):
(WebCore::RenderThemeGtk::paintSliderTrack):
(WebCore::RenderThemeGtk::paintSliderThumb):

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderThemeGtk.cpp

index 87aa1f3..92873ba 100644 (file)
@@ -1,5 +1,22 @@
 2014-09-12  Carlos Garcia Campos  <cgarcia@igalia.com>
 
+        REGRESSION(r173441): [GTK] All buttons appear insensitive
+        https://bugs.webkit.org/show_bug.cgi?id=136614
+
+        Reviewed by Sergio Villar Senin.
+
+        Do not use RenderTheme::isReadOnlyControl() to decide whether to
+        render non text based forms sensitive or unsensitive, isEnabled()
+        is enough in those cases.
+
+        * rendering/RenderThemeGtk.cpp:
+        (WebCore::paintToggle):
+        (WebCore::renderButton):
+        (WebCore::RenderThemeGtk::paintSliderTrack):
+        (WebCore::RenderThemeGtk::paintSliderThumb):
+
+2014-09-12  Carlos Garcia Campos  <cgarcia@igalia.com>
+
         REGRESSION(r173423): CertificateInfo is never sent to the UI process when using shared secondary process model
         https://bugs.webkit.org/show_bug.cgi?id=136669
 
index 518d814..053fa89 100644 (file)
@@ -515,7 +515,7 @@ static void paintToggle(const RenderThemeGtk* theme, GType widgetType, const Ren
     gtk_style_context_add_class(context, widgetType == GTK_TYPE_CHECK_BUTTON ? GTK_STYLE_CLASS_CHECK : GTK_STYLE_CLASS_RADIO);
 
     guint flags = 0;
-    if (!theme->isEnabled(renderObject) || theme->isReadOnlyControl(renderObject))
+    if (!theme->isEnabled(renderObject))
         flags |= GTK_STATE_FLAG_INSENSITIVE;
     else if (theme->isHovered(renderObject))
         flags |= GTK_STATE_FLAG_PRELIGHT;
@@ -575,7 +575,7 @@ static void renderButton(RenderTheme* theme, GtkStyleContext* context, const Ren
     IntRect buttonRect(rect);
 
     guint flags = 0;
-    if (!theme->isEnabled(renderObject) || theme->isReadOnlyControl(renderObject))
+    if (!theme->isEnabled(renderObject))
         flags |= GTK_STATE_FLAG_INSENSITIVE;
     else if (theme->isHovered(renderObject))
         flags |= GTK_STATE_FLAG_PRELIGHT;
@@ -1105,7 +1105,7 @@ bool RenderThemeGtk::paintSliderTrack(const RenderObject& renderObject, const Pa
     applySliderStyleContextClasses(context, part);
     gtk_style_context_add_class(context, GTK_STYLE_CLASS_TROUGH);
 
-    if (!isEnabled(renderObject) || isReadOnlyControl(renderObject))
+    if (!isEnabled(renderObject))
         gtk_style_context_set_state(context, GTK_STATE_FLAG_INSENSITIVE);
 
     gtk_render_background(context, paintInfo.context->platformContext()->cr(), rect.x(), rect.y(), rect.width(), rect.height());
@@ -1136,7 +1136,7 @@ bool RenderThemeGtk::paintSliderThumb(const RenderObject& renderObject, const Pa
     gtk_style_context_add_class(context, GTK_STYLE_CLASS_SLIDER);
 
     guint flags = 0;
-    if (!isEnabled(renderObject) || isReadOnlyControl(renderObject))
+    if (!isEnabled(renderObject))
         flags |= GTK_STATE_FLAG_INSENSITIVE;
     else if (isHovered(renderObject))
         flags |= GTK_STATE_FLAG_PRELIGHT;