Changeset 163184 in webkit


Ignore:
Timestamp:
Jan 31, 2014 11:49:30 AM (10 years ago)
Author:
akling@apple.com
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):

Location:
trunk/Source/WebCore
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r163182 r163184  
     12014-01-31  Andreas Kling  <akling@apple.com>
     2
     3        Deduplicate Document::encoding().
     4        <https://webkit.org/b/128000>
     5
     6        Make Document::encoding() return an AtomicString so we don't duplicate
     7        the string every time it's called.
     8
     9        215 KB progression on Membuster3.
     10
     11        Reviewed by Simon Fraser.
     12
     13        * dom/Document.h:
     14        * dom/Document.cpp:
     15        (WebCore::Document::encoding):
     16
    1172014-01-31  Simon Fraser  <simon.fraser@apple.com>
    218
  • trunk/Source/WebCore/dom/Document.cpp

    r163079 r163184  
    12711271}
    12721272
    1273 String Document::encoding() const
     1273AtomicString Document::encoding() const
    12741274{
    12751275    if (TextResourceDecoder* d = decoder())
  • trunk/Source/WebCore/dom/Document.h

    r163092 r163184  
    415415    String characterSet() const { return Document::encoding(); }
    416416
    417     String encoding() const;
     417    AtomicString encoding() const;
    418418
    419419    void setCharset(const String&);
Note: See TracChangeset for help on using the changeset viewer.