Bug 98082

Summary: Add new GraphicsLayer::create overload to all ports
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: cc-bugs, gustavo, jamesr, mifenton, noam, philn, rwlbuis, webkit.review.bot, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch kling: review+, buildbot: commit-queue-

Description Anders Carlsson 2012-10-01 15:36:33 PDT 
Add new GraphicsLayer::create overload to all ports
Comment 1 Anders Carlsson 2012-10-01 15:38:10 PDT 
Created attachment 166561 [details]
Patch
Comment 2 James Robinson 2012-10-01 16:01:58 PDT 
Can I ask what this is for?
Comment 3 Early Warning System Bot 2012-10-01 16:02:46 PDT 
Comment on attachment 166561 [details]
Patch

Attachment 166561 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14089954
Comment 4 Early Warning System Bot 2012-10-01 16:10:58 PDT 
Comment on attachment 166561 [details]
Patch

Attachment 166561 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/14088952
Comment 5 Build Bot 2012-10-01 16:21:03 PDT 
Comment on attachment 166561 [details]
Patch

Attachment 166561 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14131006

New failing tests:
svg/animations/smil-leak-dynamically-added-element-instances.svg
Comment 6 Anders Carlsson 2012-10-01 16:21:25 PDT 
Created attachment 166568 [details]
Patch
Comment 7 Early Warning System Bot 2012-10-01 16:37:46 PDT 
Comment on attachment 166568 [details]
Patch

Attachment 166568 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14118258
Comment 8 Anders Carlsson 2012-10-01 16:42:54 PDT 
Created attachment 166570 [details]
Patch
Comment 9 Anders Carlsson 2012-10-01 16:54:14 PDT 
Created attachment 166573 [details]
Patch
Comment 10 Anders Carlsson 2012-10-01 16:58:55 PDT 
Created attachment 166574 [details]
Patch
Comment 11 Build Bot 2012-10-01 18:54:08 PDT 
Comment on attachment 166574 [details]
Patch

Attachment 166574 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14118298

New failing tests:
http/tests/workers/terminate-during-sync-operation.html
Comment 12 Anders Carlsson 2012-10-02 10:36:50 PDT 
(In reply to comment #2)
> Can I ask what this is for?

Sure, it's to make it easier for ports to change the type of layers created on a per page basis. In WebKit2 it can be used to get rid of the setGraphicsLayerFactory code that the Qt port uses. We can also use it for experimenting with UI process side compositing.
Comment 13 Andreas Kling 2012-10-02 10:47:24 PDT 
Comment on attachment 166574 [details]
Patch

r=me with non-herpderp ChangeLog.
Comment 14 Anders Carlsson 2012-10-02 10:50:44 PDT 
Committed r130185: <http://trac.webkit.org/changeset/130185>