WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
126189
Add the pseudo class :focus to the Selector Code Generator
https://bugs.webkit.org/show_bug.cgi?id=126189
Summary
Add the pseudo class :focus to the Selector Code Generator
Benjamin Poulain
Reported
2013-12-23 16:18:30 PST
Add the pseudo class :focus to the Selector Code Generator
Attachments
Patch
(4.82 KB, patch)
2013-12-23 16:18 PST
,
Benjamin Poulain
rniwa
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Benjamin Poulain
Comment 1
2013-12-23 16:18:46 PST
Created
attachment 219940
[details]
Patch
Ryosuke Niwa
Comment 2
2013-12-23 16:30:36 PST
Comment on
attachment 219940
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=219940&action=review
> Source/WebCore/cssjit/SelectorCompiler.cpp:244 > + m_functionType = std::max(m_functionType, addPseudoType(selector->pseudoType(), fragment.pseudoClasses));
Maybe we should replace std::max with some descriptive inline function? Or maybe addPseudoType should be a member function of SelectorCodeGenerator that auto-updates m_functionType. Or it should take m_functionType as an argument?
Benjamin Poulain
Comment 3
2013-12-23 17:19:41 PST
Committed
r161041
: <
http://trac.webkit.org/changeset/161041
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug