NEW 65003
Remove the immediate repaint codepath and support, it doesn't do anything 
https://bugs.webkit.org/show_bug.cgi?id=65003
Summary Remove the immediate repaint codepath and support, it doesn't do anything
James Robinson
Reported 2011-07-21 18:18:13 PDT
Remove the immediate repaint codepath and support, it doesn't do anything
Attachments
Patch (84.06 KB, patch)
2011-07-21 19:20 PDT, James Robinson 		darin: review-
webkit.review.bot: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
James Robinson
Comment 1 2011-07-21 19:20:57 PDT
Created attachment 101687 [details] Patch
mitz
Comment 2 2011-07-21 20:17:21 PDT
This code appears to work perfectly well in WebKit1 in OS X Lion. I tested activating a link with the keyboard and activating a button with a keyboard, and in both cases the element painted in its active state, then after a short delay, repainted the inactive state. I think the patch will break this behavior.
WebKit Review Bot
Comment 3 2011-07-21 20:58:37 PDT
Comment on attachment 101687 [details] Patch Attachment 101687 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/9199971
WebKit Review Bot
Comment 4 2011-07-21 22:09:25 PDT
Comment on attachment 101687 [details] Patch Attachment 101687 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/9193968
Darin Adler
Comment 5 2011-07-21 23:09:11 PDT
Comment on attachment 101687 [details] Patch review- based on the build failures and Dan’s comments
Eric Seidel (no email)
Comment 6 2013-03-01 16:19:47 PST
Wow. Do we know if there is another way to get this behavior for Mac? If James is correct, this 80k patch of code is only to support Mac WK1. :) If we have to keep it for Mac, that's fine (and we can do bug 87054 instead, but if it's possible to get this behavior some other way, that seems best. I guess bug 87054 is the expedient solution since it's unlikely that James is the right person to change Mac WK1 here.
Note You need to log in before you can comment on or make changes to this bug.