Title: [163184] trunk/Source/WebCore
Revision
163184
Author
[email protected]
Date
2014-01-31 11:49:30 -0800 (Fri, 31 Jan 2014)

Log Message

Deduplicate Document::encoding().
<https://webkit.org/b/128000>

Make Document::encoding() return an AtomicString so we don't duplicate
the string every time it's called.

215 KB progression on Membuster3.

Reviewed by Simon Fraser.

* dom/Document.h:
* dom/Document.cpp:
(WebCore::Document::encoding):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (163183 => 163184)


--- trunk/Source/WebCore/ChangeLog	2014-01-31 19:48:35 UTC (rev 163183)
+++ trunk/Source/WebCore/ChangeLog	2014-01-31 19:49:30 UTC (rev 163184)
@@ -1,3 +1,19 @@
+2014-01-31  Andreas Kling  <[email protected]>
+
+        Deduplicate Document::encoding().
+        <https://webkit.org/b/128000>
+
+        Make Document::encoding() return an AtomicString so we don't duplicate
+        the string every time it's called.
+
+        215 KB progression on Membuster3.
+
+        Reviewed by Simon Fraser.
+
+        * dom/Document.h:
+        * dom/Document.cpp:
+        (WebCore::Document::encoding):
+
 2014-01-31  Simon Fraser  <[email protected]>
 
         Even when in fixed layout mode, some platforms need to do layout after a viewport change

Modified: trunk/Source/WebCore/dom/Document.cpp (163183 => 163184)


--- trunk/Source/WebCore/dom/Document.cpp	2014-01-31 19:48:35 UTC (rev 163183)
+++ trunk/Source/WebCore/dom/Document.cpp	2014-01-31 19:49:30 UTC (rev 163184)
@@ -1270,7 +1270,7 @@
         setVisualUpdatesAllowed(true);
 }
 
-String Document::encoding() const
+AtomicString Document::encoding() const
 {
     if (TextResourceDecoder* d = decoder())
         return d->encoding().domName();

Modified: trunk/Source/WebCore/dom/Document.h (163183 => 163184)


--- trunk/Source/WebCore/dom/Document.h	2014-01-31 19:48:35 UTC (rev 163183)
+++ trunk/Source/WebCore/dom/Document.h	2014-01-31 19:49:30 UTC (rev 163184)
@@ -414,7 +414,7 @@
     String charset() const { return Document::encoding(); }
     String characterSet() const { return Document::encoding(); }
 
-    String encoding() const;
+    AtomicString encoding() const;
 
     void setCharset(const String&);
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to