Changeset 121320 in webkit

Timestamp:

Jun 26, 2012, 10:40:18 PM (14 years ago)

Author:

macpherson@chromium.org

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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/Settings.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-06-26  Luke Macpherson  <macpherson@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  <huangxueqing@baidu.com>
 

trunk/Source/WebCore/page/Settings.h

@@ -334 +334 @@
 #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) { }