Make sure the new streaming parser invalidates CSS correctly [chromium/src : main]

0 views

Skip to first unread message

Philip Jägenstedt (Gerrit)

unread,

4:38 AM (15 hours ago) 4:38 AM

to Noam Rosenthal, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, loading-rev...@chromium.org, kinuko...@chromium.org, blink-rev...@chromium.org, blink-re...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org

Attention needed from Noam Rosenthal

Philip Jägenstedt added 2 comments

Patchset-level comments

File-level comment, Patchset 5 (Latest):

Philip Jägenstedt . resolved

The commit message talks about insertions, but only removals are changed here. Are insertions already not treated as parser-inserted?

File third_party/blink/renderer/core/html/parser/html_construction_site.cc

Line 295, Patchset 5 (Latest): task.parent->RemoveChildren();

Philip Jägenstedt . unresolved

Can you leave a comment here explaining why it's not ParserRemoveChildren() as one might expect from the surrounding code?

Open in Gerrit

Related details

Attention is currently required from:

Noam Rosenthal

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Reply all

Reply to author

Forward

0 new messages