RESOLVED FIXED 77247
[chromium] Remove unnecessary retry logic in LayerRendererChromium initialization for accelerated painting 
https://bugs.webkit.org/show_bug.cgi?id=77247
Summary [chromium] Remove unnecessary retry logic in LayerRendererChromium initializa...
James Robinson
Reported 2012-01-27 14:39:35 PST
[chromium] Remove unnecessary retry logic in LayerRendererChromium initialization for accelerated painting
Attachments
Patch (3.63 KB, patch)
2012-01-27 14:41 PST, James Robinson 		no flags
DetailsFormatted DiffDiff
now with 50% more compilation (3.63 KB, patch)
2012-01-27 15:13 PST, James Robinson 		kbr: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
James Robinson
Comment 1 2012-01-27 14:41:09 PST
Created attachment 124371 [details] Patch
Adrienne Walker
Comment 2 2012-01-27 14:46:27 PST
Comment on attachment 124371 [details] Patch Looks good to me. It's kind of awkward to use creation failure as a signal, and I can't see any reason to do that.
WebKit Review Bot
Comment 3 2012-01-27 14:49:20 PST
Comment on attachment 124371 [details] Patch Attachment 124371 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/11265858
James Robinson
Comment 4 2012-01-27 14:49:43 PST
(In reply to comment #3) > (From update of attachment 124371 [details]) > Attachment 124371 [details] did not pass chromium-ews (chromium-xvfb): > Output: http://queues.webkit.org/results/11265858 Whoops, parenthesis are hard!
James Robinson
Comment 5 2012-01-27 15:13:31 PST
Created attachment 124379 [details] now with 50% more compilation
Kenneth Russell
Comment 6 2012-01-30 15:29:42 PST
Comment on attachment 124379 [details] now with 50% more compilation Looks fine to me.
James Robinson
Comment 7 2012-01-30 17:23:24 PST
Committed r106306: <http://trac.webkit.org/changeset/106306>
Note You need to log in before you can comment on or make changes to this bug.