Title: [166604] trunk/Source/WebCore
Revision
166604
Author
benja...@webkit.org
Date
2014-04-01 13:32:33 -0700 (Tue, 01 Apr 2014)

Log Message

Remove a couple of useless static strings
https://bugs.webkit.org/show_bug.cgi?id=131003

Patch by Benjamin Poulain <bpoul...@apple.com> on 2014-04-01
Reviewed by David Kilzer.

There is zero value in keeping those strings alive.

* css/StyleProperties.cpp:
(WebCore::isInitialOrInherit):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (166603 => 166604)


--- trunk/Source/WebCore/ChangeLog	2014-04-01 20:05:08 UTC (rev 166603)
+++ trunk/Source/WebCore/ChangeLog	2014-04-01 20:32:33 UTC (rev 166604)
@@ -1,3 +1,15 @@
+2014-04-01  Benjamin Poulain  <bpoul...@apple.com>
+
+        Remove a couple of useless static strings
+        https://bugs.webkit.org/show_bug.cgi?id=131003
+
+        Reviewed by David Kilzer.
+
+        There is zero value in keeping those strings alive.
+
+        * css/StyleProperties.cpp:
+        (WebCore::isInitialOrInherit):
+
 2014-04-01  Myles C. Maxfield  <mmaxfi...@apple.com>
 
         svg/text/text-overflow-ellipsis-svgfont-kerning-ligatures.html and svg/css/font-face-crash.html frequently assert in ComplexTextController::offsetForPosition

Modified: trunk/Source/WebCore/css/StyleProperties.cpp (166603 => 166604)


--- trunk/Source/WebCore/css/StyleProperties.cpp	2014-04-01 20:05:08 UTC (rev 166603)
+++ trunk/Source/WebCore/css/StyleProperties.cpp	2014-04-01 20:32:33 UTC (rev 166604)
@@ -50,9 +50,7 @@
 
 static bool isInitialOrInherit(const String& value)
 {
-    DEPRECATED_DEFINE_STATIC_LOCAL(String, initial, ("initial"));
-    DEPRECATED_DEFINE_STATIC_LOCAL(String, inherit, ("inherit"));
-    return value.length() == 7 && (value == initial || value == inherit);
+    return value.length() == 7 && (value == "initial" || value == "inherit");
 }
 
 PassRef<ImmutableStyleProperties> ImmutableStyleProperties::create(const CSSProperty* properties, unsigned count, CSSParserMode cssParserMode)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to