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