Title: [131195] trunk/Source/WebCore
Revision
131195
Author
[email protected]
Date
2012-10-12 09:58:53 -0700 (Fri, 12 Oct 2012)

Log Message

RenderBR should share its constant newline string between instances.
<http://webkit.org/b/99121>

Reviewed by Anders Carlsson.

* rendering/RenderBR.cpp:
(WebCore::newlineString):
(WebCore::RenderBR::RenderBR):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (131194 => 131195)


--- trunk/Source/WebCore/ChangeLog	2012-10-12 16:57:41 UTC (rev 131194)
+++ trunk/Source/WebCore/ChangeLog	2012-10-12 16:58:53 UTC (rev 131195)
@@ -1,3 +1,14 @@
+2012-10-12  Andreas Kling  <[email protected]>
+
+        RenderBR should share its constant newline string between instances.
+        <http://webkit.org/b/99121>
+
+        Reviewed by Anders Carlsson.
+
+        * rendering/RenderBR.cpp:
+        (WebCore::newlineString):
+        (WebCore::RenderBR::RenderBR):
+
 2012-10-12  Alexander Pavlov  <[email protected]>
 
         Web Inspector: [Styles] !important priority not honored inside the same declaration

Modified: trunk/Source/WebCore/rendering/RenderBR.cpp (131194 => 131195)


--- trunk/Source/WebCore/rendering/RenderBR.cpp	2012-10-12 16:57:41 UTC (rev 131194)
+++ trunk/Source/WebCore/rendering/RenderBR.cpp	2012-10-12 16:58:53 UTC (rev 131195)
@@ -28,8 +28,14 @@
 
 namespace WebCore {
 
+static PassRefPtr<StringImpl> newlineString()
+{
+    DEFINE_STATIC_LOCAL(const String, string, (ASCIILiteral("\n")));
+    return string.impl();
+}
+
 RenderBR::RenderBR(Node* node)
-    : RenderText(node, StringImpl::create("\n"))
+    : RenderText(node, newlineString())
     , m_lineHeight(-1)
 {
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to