Title: [121320] trunk/Source/WebCore
Revision
121320
Author
macpher...@chromium.org
Date
2012-06-26 22:40:18 -0700 (Tue, 26 Jun 2012)

Log Message

Return correct value for css variables enabled runtime flag.
https://bugs.webkit.org/show_bug.cgi?id=90040

Reviewed by Dimitri Glazkov.

Was always returning true for the runtime flag when the compile time flag was on. That was good for testing,
but not so much for production.

* page/Settings.h:
(WebCore::Settings::cssVariablesEnabled):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (121319 => 121320)


--- trunk/Source/WebCore/ChangeLog	2012-06-27 04:22:47 UTC (rev 121319)
+++ trunk/Source/WebCore/ChangeLog	2012-06-27 05:40:18 UTC (rev 121320)
@@ -1,3 +1,16 @@
+2012-06-26  Luke Macpherson  <macpher...@chromium.org>
+
+        Return correct value for css variables enabled runtime flag.
+        https://bugs.webkit.org/show_bug.cgi?id=90040
+
+        Reviewed by Dimitri Glazkov.
+
+        Was always returning true for the runtime flag when the compile time flag was on. That was good for testing,
+        but not so much for production.
+
+        * page/Settings.h:
+        (WebCore::Settings::cssVariablesEnabled):
+
 2012-06-26  Xueqing Huang  <huangxueq...@baidu.com>
 
         DragData::asFilenames should not push same file names to result in Windows.

Modified: trunk/Source/WebCore/page/Settings.h (121319 => 121320)


--- trunk/Source/WebCore/page/Settings.h	2012-06-27 04:22:47 UTC (rev 121319)
+++ trunk/Source/WebCore/page/Settings.h	2012-06-27 05:40:18 UTC (rev 121320)
@@ -333,7 +333,7 @@
 
 #if ENABLE(CSS_VARIABLES)
         void setCSSVariablesEnabled(bool enabled) { m_cssVariablesEnabled = enabled; }
-        bool cssVariablesEnabled() const { return true; }
+        bool cssVariablesEnabled() const { return m_cssVariablesEnabled; }
 #else
         void setCSSVariablesEnabled(bool) { }
         bool cssVariablesEnabled() const { return false; }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to