commit | 8c9196789132507fb3062f7306cb36ef839b7dc4 | [log] [tgz] |
---|---|---|
author | Koji Ishii <kojii@chromium.org> | Thu Nov 15 07:54:26 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Thu Nov 15 07:54:26 2018 |
tree | e42b056f32be2baa93117b2adb3533e5925b03af | |
parent | 8d7b77a1205f89f36b216c301016599c87538354 [diff] |
[LayoutNG] Move should_create_box_fragment to LayoutInline This patch moves the flag whether to force to create a box fragment for LayoutInline or not from NGInlineItem to LayoutInline. This is part of an effort to reduce re-collecting NGInlineItem when styles were changed. Re-collecting is rather an expensive operation that changes in, for instance, background should not require it. Note, this patch itself does not reduce the collection, because we require re-collection whenever NeedsLayout is set. It is planned in future patches, but this change is needed to make it happen. Also benefits to keep the flag turned on once it's on, until the LayoutInline is reattached. The change has minor performance gain in most blink_perf.layout https://pinpoint-dot-chromeperf.appspot.com/job/15f8c74be40000 Bug: 636993 Change-Id: I8d4e7c421d5fad26fee60832cdfbacd575f7ad34 Reviewed-on: https://chromium-review.googlesource.com/c/1333588 Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by: Aleks Totic <atotic@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#608292}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .