Bug 60173

Summary: [Chromium] Change the --accelerated-2d-canvas flag to mean Ganesh, and stop using --enable-accelerated-drawing for that purpose
Product: WebKit Reporter: Justin Novosad <junov>
Component: Layout and RenderingAssignee: Justin Novosad <junov>
Status: RESOLVED FIXED    
Severity: Normal CC: alokp, commit-queue, jamesr, kbr, senorblanco, vangelis
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on: 60168    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Description Justin Novosad 2011-05-04 08:56:08 PDT
This is phase 2 of the delivery of the solution for https://bugs.webkit.org/show_bug.cgi?id=59929
This change will make the GPU accelerated 2D canvas lab use skia for rendering, which will likely require re-baselining several of the gpu layout test
and also adding additional failures to test_expectaions.txt for the unresolved issues.
Comment 1 Justin Novosad 2011-05-05 10:24:11 PDT
Currently blocked on:
http://code.google.com/p/chromium/issues/detail?id=81674
Comment 2 Justin Novosad 2011-05-05 13:00:50 PDT
Created attachment 92454 [details]
Patch
Comment 3 Kenneth Russell 2011-05-05 14:42:37 PDT
Comment on attachment 92454 [details]
Patch

Thumbs up assuming it's been tested and won't turn the tree red. Please watch the canaries (see http://www.chromium.org/developers/how-tos/webkit-merge-1 ).
Comment 4 Justin Novosad 2011-05-05 14:53:35 PDT
(In reply to comment #3)
> (From update of attachment 92454 [details])
> Thumbs up assuming it's been tested and won't turn the tree red. Please watch the canaries (see http://www.chromium.org/developers/how-tos/webkit-merge-1 ).

The try-bots came back clear on all platforms.
Comment 5 James Robinson 2011-05-05 15:04:59 PDT
Did you run through the layout trybots (linux_layout, mac_layout, win_layout)?
Comment 6 James Robinson 2011-05-05 15:07:12 PDT
http://build.chromium.org/p/tryserver.chromium/builders/linux_layout/builds/638 shows a lot of crashes on canvas tests
Comment 7 Justin Novosad 2011-05-05 15:09:48 PDT
(In reply to comment #6)
> http://build.chromium.org/p/tryserver.chromium/builders/linux_layout/builds/638 shows a lot of crashes on canvas tests

No worries, that run did not have brian's fix in it.  I launched a new try with --revision=84277.  It came back clean on win_layout_rel.  Still haven't heard back from the debug bot though
Comment 8 WebKit Commit Bot 2011-05-05 15:39:26 PDT
Comment on attachment 92454 [details]
Patch

Clearing flags on attachment: 92454

Committed r85895: <http://trac.webkit.org/changeset/85895>
Comment 9 WebKit Commit Bot 2011-05-05 15:39:31 PDT
All reviewed patches have been landed.  Closing bug.