Title: [233728] trunk/Source/WebCore
Revision
233728
Author
za...@apple.com
Date
2018-07-11 06:30:14 -0700 (Wed, 11 Jul 2018)

Log 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:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (233727 => 233728)


--- trunk/Source/WebCore/ChangeLog	2018-07-11 10:02:05 UTC (rev 233727)
+++ trunk/Source/WebCore/ChangeLog	2018-07-11 13:30:14 UTC (rev 233728)
@@ -1,3 +1,17 @@
+2018-07-11  Zalan Bujtas  <za...@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  <you...@apple.com>
 
         Make fetch() use "same-origin" credentials by default

Modified: trunk/Source/WebCore/rendering/SimpleLineLayoutFlowContents.h (233727 => 233728)


--- trunk/Source/WebCore/rendering/SimpleLineLayoutFlowContents.h	2018-07-11 10:02:05 UTC (rev 233727)
+++ trunk/Source/WebCore/rendering/SimpleLineLayoutFlowContents.h	2018-07-11 13:30:14 UTC (rev 233728)
@@ -58,7 +58,7 @@
 
 private:
     unsigned segmentIndexForRunSlow(unsigned start, unsigned end) const;
-    const Vector<Segment, 8> m_segments;
+    const Vector<Segment> m_segments;
     mutable unsigned m_lastSegmentIndex { 0 };
 };
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to