Deduplicate Document::encoding().
authorakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 31 Jan 2014 19:49:30 +0000 (19:49 +0000)
committerakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 31 Jan 2014 19:49:30 +0000 (19:49 +0000)
<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):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@163184 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/dom/Document.cpp
Source/WebCore/dom/Document.h

index 8f6ded87799aad3d96f607e4ade0e7308e2ba299..945ddc54eaa2be418009964a6877b3a2b46fb4d1 100644 (file)
@@ -1,3 +1,19 @@
+2014-01-31  Andreas Kling  <akling@apple.com>
+
+        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  <simon.fraser@apple.com>
 
         Even when in fixed layout mode, some platforms need to do layout after a viewport change
index 7d70dcc3ee8ed7a0627d4e389bf014d92120aa74..d8f3cac689baed2e95820af971931f0468fffcef 100644 (file)
@@ -1270,7 +1270,7 @@ void Document::setVisualUpdatesAllowedByClient(bool visualUpdatesAllowedByClient
         setVisualUpdatesAllowed(true);
 }
 
-String Document::encoding() const
+AtomicString Document::encoding() const
 {
     if (TextResourceDecoder* d = decoder())
         return d->encoding().domName();
index 79bee34ca7d9febf339d77b327d570f4bc98107c..31b067988dc0cd7ded1dd52ff17e13446dcee33d 100644 (file)
@@ -414,7 +414,7 @@ public:
     String charset() const { return Document::encoding(); }
     String characterSet() const { return Document::encoding(); }
 
-    String encoding() const;
+    AtomicString encoding() const;
 
     void setCharset(const String&);