Title: [275620] trunk/Source/WebCore
- Revision
- 275620
- Author
- [email protected]
- Date
- 2021-04-07 12:25:52 -0700 (Wed, 07 Apr 2021)
Log Message
[LFC][IFC] Remove InlineLines and InlineLineBoxes inline capacity of 10
https://bugs.webkit.org/show_bug.cgi?id=224296
Reviewed by Antti Koivisto.
A casual browsing session shows that the average line count is ~1.4. Having 10 as initial capacity seems a little excessive.
(and neither PerformanceTests/Layout/line-layout-inline-block.html nor PerformanceTests/Layout/line-layout-inline-level-boxes.html microbenchmarks show regression locally)
* layout/inlineformatting/InlineFormattingState.h:
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (275619 => 275620)
--- trunk/Source/WebCore/ChangeLog 2021-04-07 19:16:47 UTC (rev 275619)
+++ trunk/Source/WebCore/ChangeLog 2021-04-07 19:25:52 UTC (rev 275620)
@@ -1,3 +1,15 @@
+2021-04-07 Zalan Bujtas <[email protected]>
+
+ [LFC][IFC] Remove InlineLines and InlineLineBoxes inline capacity of 10
+ https://bugs.webkit.org/show_bug.cgi?id=224296
+
+ Reviewed by Antti Koivisto.
+
+ A casual browsing session shows that the average line count is ~1.4. Having 10 as initial capacity seems a little excessive.
+ (and neither PerformanceTests/Layout/line-layout-inline-block.html nor PerformanceTests/Layout/line-layout-inline-level-boxes.html microbenchmarks show regression locally)
+
+ * layout/inlineformatting/InlineFormattingState.h:
+
2021-04-07 Chris Dumez <[email protected]>
Drop unnecessary NDEBUG checks in ServiceWorkerContainer
Modified: trunk/Source/WebCore/layout/inlineformatting/InlineFormattingState.h (275619 => 275620)
--- trunk/Source/WebCore/layout/inlineformatting/InlineFormattingState.h 2021-04-07 19:16:47 UTC (rev 275619)
+++ trunk/Source/WebCore/layout/inlineformatting/InlineFormattingState.h 2021-04-07 19:25:52 UTC (rev 275620)
@@ -38,8 +38,8 @@
namespace Layout {
using InlineItems = Vector<InlineItem>;
-using InlineLines = Vector<InlineLineGeometry, 10>;
-using InlineLineBoxes = Vector<LineBox, 10>;
+using InlineLines = Vector<InlineLineGeometry>;
+using InlineLineBoxes = Vector<LineBox>;
using InlineLineRuns = Vector<LineRun>;
// InlineFormattingState holds the state for a particular inline formatting context tree.
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
