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