Changeset 275483 in webkit

Timestamp:

Apr 5, 2021 8:08:15 PM (3 years ago)

Author:

Simon Fraser

Message:

Wasted vector capacity in SVGPathByteStream
​https://bugs.webkit.org/show_bug.cgi?id=224165

Reviewed by Said Abou-Hallawa.

On youtube.com, about 50KB of vector capacity is wasted in SVGPathByteStream,
so shrink them after parsing.

• svg/SVGPathByteStream.h:

(WebCore::SVGPathByteStream::shrinkToFit):

• svg/SVGPathParser.cpp:

(WebCore::SVGPathParser::parseToByteStream):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• svg/SVGPathByteStream.h (modified) (1 diff)
• svg/SVGPathParser.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2021-04-05  Simon Fraser  <simon.fraser@apple.com>
+
+        Wasted vector capacity in SVGPathByteStream
+        https://bugs.webkit.org/show_bug.cgi?id=224165
+
+        Reviewed by Said Abou-Hallawa.
+
+        On youtube.com, about 50KB of vector capacity is wasted in SVGPathByteStream,
+        so shrink them after parsing.
+
+        * svg/SVGPathByteStream.h:
+        (WebCore::SVGPathByteStream::shrinkToFit):
+        * svg/SVGPathParser.cpp:
+        (WebCore::SVGPathParser::parseToByteStream):
+
 2021-04-05  Simon Fraser  <simon.fraser@apple.com>
 

trunk/Source/WebCore/svg/SVGPathByteStream.h

@@ -98 +98 @@
     bool isEmpty() const { return m_data.isEmpty(); }
     unsigned size() const { return m_data.size(); }
+    void shrinkToFit() { m_data.shrinkToFit(); }
 
 private:

trunk/Source/WebCore/svg/SVGPathParser.cpp

@@ -44 +44 @@
 {
     SVGPathByteStreamBuilder builder(byteStream);
-    return parse(source, builder, mode, checkForInitialMoveTo);
+    auto result = parse(source, builder, mode, checkForInitialMoveTo);
+    byteStream.shrinkToFit();
+    return result;
 }