Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8)

Issue 2700943003: Invalidate custom pseudo elements for RuleSet invalidations. (Closed)

Created:
3 years, 10 months ago by rune
Modified:
3 years, 10 months ago
Reviewers:
Eric Willigers
CC:
chromium-reviews, blink-reviews-style_chromium.org, blink-reviews-css, sof, eae+blinkwatch, blink-reviews-dom_chromium.org, dglazkov+blink, apavlov+blink_chromium.org, darktears, blink-reviews, rwlbuis
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Invalidate custom pseudo elements for RuleSet invalidations. When we have selectors containing custom pseudo elements matching elements inside the UA shadow, and no id, class, or attribute selectors present, do like we do for type selectors and invalidate as part of an invalidation set scheduled on the root node of the tree scope for RuleSet invalidations. We utilize the same invalidation set as for type invalidations and mark it as invalidating custom pseudo elements as well as marking it as tree-boundary-crossing to allow drilling into the UA shadow. This means we will traverse into all shadow sub-trees, but it should at least be better than the existing recalc all behavior. The full recalc for custom pseudo elements caused a full recalc for one of the stylesheets on facebook.com. R=ericwilligers@chromium.org BUG=680549 Review-Url: https://codereview.chromium.org/2700943003 Cr-Commit-Position: refs/heads/master@{#451578} Committed: https://chromium.googlesource.com/chromium/src/+/248934089f7a250a94842e1c3059167d704e72fd

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+56 lines, -15 lines) Patch
M third_party/WebKit/LayoutTests/fast/css/invalidation/sheet-ruleset-invalidation.html View 2 chunks +22 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/css/RuleFeature.h View 1 chunk +0 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/css/RuleFeature.cpp View 2 chunks +16 lines, -13 lines 0 comments Download
M third_party/WebKit/Source/core/css/RuleFeatureSetTest.cpp View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/core/dom/StyleEngineTest.cpp View 1 chunk +17 lines, -0 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 11 (7 generated)
rune
ptal
3 years, 10 months ago (2017-02-17 13:38:45 UTC) #3
Eric Willigers
lgtm
3 years, 10 months ago (2017-02-20 02:00:09 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2700943003/1
3 years, 10 months ago (2017-02-20 06:06:08 UTC) #8
commit-bot: I haz the power
3 years, 10 months ago (2017-02-20 08:04:46 UTC) #11
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/248934089f7a250a94842e1c3059...

Powered by Google App Engine
This is Rietveld 408576698