Commit 1e3b9890 authored by Rune Lillesveen's avatar Rune Lillesveen Committed by Commit Bot

Don't use Bloom filter for layout tree rebuild.

Layout tree rebuild and re-attachment now only matches selectors for
::first-letter. It's not worth populating the Bloom filter with
ancestor attributes just for optimizing ::first-letter matching.

Bug: 843520, 836126
Change-Id: I22c3a4d6518314ded66ef9b214db53c3ff16aac2
Reviewed-on: https://chromium-review.googlesource.com/1158230Reviewed-by: default avatarAnders Ruud <andruud@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579774}
parent 4e98bec3
......@@ -2114,8 +2114,6 @@ void Element::AttachLayoutTree(AttachContext& context) {
rare_data->ClearPseudoElements();
}
SelectorFilterParentScope filter_scope(*this);
AttachContext children_context(context);
LayoutObject* layout_object = GetLayoutObject();
......@@ -2500,7 +2498,6 @@ void Element::RebuildLayoutTree(WhitespaceAttacher& whitespace_attacher) {
whitespace_attacher.DidReattachElement(this,
reattach_context.previous_in_flow);
} else {
SelectorFilterParentScope filter_scope(*this);
// We create a local WhitespaceAttacher when rebuilding children of an
// element with a LayoutObject since whitespace nodes do not rely on layout
// objects further up the tree. Also, if this Element's layout object is an
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment