2010-12-27 Martin Robinson <mrobinson@igalia.com>
authormrobinson@webkit.org <mrobinson@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Dec 2010 22:37:15 +0000 (22:37 +0000)
committermrobinson@webkit.org <mrobinson@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Dec 2010 22:37:15 +0000 (22:37 +0000)
        Reviewed by Daniel Bates.

        [GTK] RenderThemeGtk::m_HScale and RenderThemeGtk::m_VScale are not initialized
        https://bugs.webkit.org/show_bug.cgi?id=51654

        Properly initialize slider members of RenderThemeGtk. This can lead to test crashes
        on some debug configurations.

        No new tests. This is already covered by current slider tests. It isn't crashing on
        the bots, but it does crash locally.

        * platform/gtk/RenderThemeGtk.cpp:
        (WebCore::RenderThemeGtk::RenderThemeGtk): Properly initialize members.

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

WebCore/ChangeLog
WebCore/platform/gtk/RenderThemeGtk.cpp

index 9a250a5..a05ddd0 100644 (file)
@@ -1,3 +1,19 @@
+2010-12-27  Martin Robinson  <mrobinson@igalia.com>
+
+        Reviewed by Daniel Bates.
+
+        [GTK] RenderThemeGtk::m_HScale and RenderThemeGtk::m_VScale are not initialized
+        https://bugs.webkit.org/show_bug.cgi?id=51654
+
+        Properly initialize slider members of RenderThemeGtk. This can lead to test crashes
+        on some debug configurations.
+
+        No new tests. This is already covered by current slider tests. It isn't crashing on
+        the bots, but it does crash locally.
+
+        * platform/gtk/RenderThemeGtk.cpp:
+        (WebCore::RenderThemeGtk::RenderThemeGtk): Properly initialize members.
+
 2010-12-27  Anton Muhin  <antonm@chromium.org>
 
         Reviewed by David Levin.
index d34847d..d92c505 100644 (file)
@@ -141,6 +141,8 @@ RenderThemeGtk::RenderThemeGtk()
     , m_gtkButton(0)
     , m_gtkEntry(0)
     , m_gtkTreeView(0)
+    , m_gtkVScale(0)
+    , m_gtkHScale(0)
     , m_panelColor(Color::white)
     , m_sliderColor(Color::white)
     , m_sliderThumbColor(Color::white)