Bug 77247

Summary: [chromium] Remove unnecessary retry logic in LayerRendererChromium initialization for accelerated painting
Product: WebKit Reporter: James Robinson <jamesr>
Component: New BugsAssignee: James Robinson <jamesr>
Status: RESOLVED FIXED    
Severity: Normal CC: cc-bugs, dglazkov, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
now with 50% more compilation kbr: review+

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
now with 50% more compilation (3.63 KB, patch)
2012-01-27 15:13 PST, James Robinson
kbr: review+
James Robinson
Comment 1 2012-01-27 14:41:09 PST
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
Note You need to log in before you can comment on or make changes to this bug.