RESOLVED INVALID100156
isLinkVisited is unnecessarily convoluted 
https://bugs.webkit.org/show_bug.cgi?id=100156
Summary isLinkVisited is unnecessarily convoluted
Adam Barth
Reported 2012-10-23 13:06:06 PDT
isLinkVisited is unnecessarily convoluted
Attachments
Patch (3.06 KB, patch)
2012-10-23 13:08 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2012-10-23 13:08:02 PDT
Created attachment 170212 [details] Patch
Adam Barth
Comment 2 2012-10-23 13:08:46 PDT
This patch is based on the discussion in bug 99483.
Eric Seidel (no email)
Comment 3 2012-10-23 13:15:22 PDT
Comment on attachment 170212 [details] Patch LGTM.
Alexey Proskuryakov
Comment 4 2012-10-23 15:00:34 PDT
Comment on attachment 170212 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=170212&action=review > Source/WebCore/css/SelectorChecker.cpp:261 > +#error "Chromium is the only port that doesn't USE(PLATFORM_STRATEGIES)." Aren't Blackberry and Wx another two?
Adam Barth
Comment 5 2012-10-23 15:03:14 PDT
(In reply to comment #4) > (From update of attachment 170212 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=170212&action=review > > > Source/WebCore/css/SelectorChecker.cpp:261 > > +#error "Chromium is the only port that doesn't USE(PLATFORM_STRATEGIES)." > > Aren't Blackberry and Wx another two? /me will check before landing.
Darin Adler
Comment 6 2013-05-13 18:46:05 PDT
Comment on attachment 170212 [details] Patch Patch is Chromium-specific, hence obsolete.
Note You need to log in before you can comment on or make changes to this bug.