RESOLVED FIXED 133295
CSS JIT: add support for the pseudo classes :hover and :active 
https://bugs.webkit.org/show_bug.cgi?id=133295
Summary CSS JIT: add support for the pseudo classes :hover and :active
Benjamin Poulain
Reported 2014-05-26 20:23:03 PDT
CSS JIT: add support for the pseudo classes :hover and :active
Attachments
Patch (38.07 KB, patch)
2014-05-26 20:31 PDT, Benjamin Poulain 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Benjamin Poulain
Comment 1 2014-05-26 20:31:17 PDT
Created attachment 232104 [details] Patch
Andreas Kling
Comment 2 2014-05-27 10:22:39 PDT
Comment on attachment 232104 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=232104&action=review > Source/WebCore/dom/Document.cpp:767 > +void Document::setCompatibilityMode(DocumentCompatibilityMode mode) Does this function need to do any additional invalidation?
Benjamin Poulain
Comment 3 2014-05-27 15:47:31 PDT
(In reply to comment #2) > (From update of attachment 232104 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=232104&action=review > > > Source/WebCore/dom/Document.cpp:767 > > +void Document::setCompatibilityMode(DocumentCompatibilityMode mode) > > Does this function need to do any additional invalidation? That should not be needed. With compiled selectors we don't even need the clearSelectorQueryCache(), it is just there for the old code.
Andreas Kling
Comment 4 2014-05-29 15:37:55 PDT
Comment on attachment 232104 [details] Patch r=me
Benjamin Poulain
Comment 5 2014-05-29 17:15:44 PDT
Comment on attachment 232104 [details] Patch Clearing flags on attachment: 232104 Committed r169478: <http://trac.webkit.org/changeset/169478>
Benjamin Poulain
Comment 6 2014-05-29 17:15:49 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.