Bug 75298

Summary: QuerySelector should not have side effect.
Product: WebKit Reporter: Shinya Kawanaka <shinyak>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: kling, koivisto, shinyak, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Shinya Kawanaka 2011-12-28 01:19:54 PST
Currently QuerySelector seems to have side effect that parentStyle's flags are set if some pseudo type (e.g. first-of-type) was used to query.
Comment 1 Shinya Kawanaka 2011-12-28 01:24:35 PST
Created attachment 120643 [details]
Patch
Comment 2 Andreas Kling 2011-12-28 09:15:52 PST
Comment on attachment 120643 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=120643&action=review

> Source/WebCore/ChangeLog:11
> +        No new tests. Covered by existing tests.

If this is covered by existing tests, why aren't you unskipping something that used to fail?
Comment 3 Antti Koivisto 2011-12-30 06:23:00 PST
(In reply to comment #2)
> If this is covered by existing tests, why aren't you unskipping something that used to fail?

The flags that these end up setting affect style recalc performance optimizations and are not easily testable. Selector queries itself should should have reasonable coverage.
Comment 4 WebKit Review Bot 2012-01-11 23:07:30 PST
Comment on attachment 120643 [details]
Patch

Clearing flags on attachment: 120643

Committed r104793: <http://trac.webkit.org/changeset/104793>
Comment 5 WebKit Review Bot 2012-01-11 23:07:35 PST
All reviewed patches have been landed.  Closing bug.