Changeset 233728 in webkit

Timestamp:

Jul 11, 2018 6:30:14 AM (6 years ago)

Author:

Alan Bujtas

Message:

SimpleLineLayout::FlowContents wastes 54KB of Vector capacity on nytimes.com
​https://bugs.webkit.org/show_bug.cgi?id=186709
<rdar://problem/41173793>

Reviewed by Simon Fraser.

The size of the m_segments vector in SimpleLineLayoutFlowContents is alway pre-computed and don't change after the initial append.

Not testable.

• rendering/SimpleLineLayoutFlowContents.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/SimpleLineLayoutFlowContents.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2018-07-11  Zalan Bujtas  <zalan@apple.com>
+
+        SimpleLineLayout::FlowContents wastes 54KB of Vector capacity on nytimes.com
+        https://bugs.webkit.org/show_bug.cgi?id=186709
+        <rdar://problem/41173793>
+
+        Reviewed by Simon Fraser.
+
+        The size of the m_segments vector in SimpleLineLayoutFlowContents is alway pre-computed and don't change after the initial append.  
+
+        Not testable.
+
+        * rendering/SimpleLineLayoutFlowContents.h:
+
 2018-07-10  Youenn Fablet  <youenn@apple.com>
 

trunk/Source/WebCore/rendering/SimpleLineLayoutFlowContents.h

@@ -59 +59 @@
 private:
     unsigned segmentIndexForRunSlow(unsigned start, unsigned end) const;
-    const Vector<Segment, 8> m_segments;
+    const Vector<Segment> m_segments;
     mutable unsigned m_lastSegmentIndex { 0 };
 };