[Webkit-unassigned] [Bug 109658] New: Unify GraphicsLayer::setContentsToMedia and setContentsToCanvas

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=109658

           Summary: Unify GraphicsLayer::setContentsToMedia and
                    setContentsToCanvas
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: trchen at chromium.org


Per discussion in https://bugs.webkit.org/show_bug.cgi?id=109560
trchen: Per off-line discussion with aelias@, we think we should make a GraphicsLayerChromium::setContentsToCoordinatedScrollbar extension. Does that sound cleaner?
smfr:  You could do that, but it might be better to make a more general setContentsToCustomSomething() function that can live on GraphicsLayer.
jamesr: As far as I can see, implementations of GraphicsLayer do not do anything different for setContentsToMedia(PlatformLayer*) and setContentsToCanvas(PlatformLayer*). GraphicsLayerCA and GraphicsLayerChromium set the contentsLayerPurpose, but this doesn't appear to be used for anything.  I think we should just fold them together into one function that accepts any sort of PlatformLayer.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.