Bug 60173 - [Chromium] Change the --accelerated-2d-canvas flag to mean Ganesh, and stop using --enable-accelerated-drawing for that purpose
Summary: [Chromium] Change the --accelerated-2d-canvas flag to mean Ganesh, and stop u...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Justin Novosad
URL:
Keywords:
Depends on: 60168
Blocks:
  Show dependency treegraph
 
Reported: 2011-05-04 08:56 PDT by Justin Novosad
Modified: 2011-05-05 15:39 PDT (History)
6 users (show)

See Also:


Attachments
Patch (10.71 KB, patch)
2011-05-05 13:00 PDT, Justin Novosad
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.