Bug 80171

Summary: [Chromium] Allow plugins to set opaque() on their platform layer
Product: WebKit Reporter: Dana Jansens <danakj>
Component: New BugsAssignee: Dana Jansens <danakj>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, backer, fishd, jamesr, jbauman, piman, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Dana Jansens 2012-03-02 10:33:53 PST
[Chromium] Allow plugins to set opaque() on their platform layer
Comment 1 Dana Jansens 2012-03-02 10:34:56 PST
Created attachment 129921 [details]
Patch
Comment 2 Dana Jansens 2012-03-02 10:36:21 PST
This matches the interface on WebMediaPlayerClient.
Comment 3 WebKit Review Bot 2012-03-02 10:36:29 PST
Please wait for approval from fishd@chromium.org before submitting because this patch contains changes to the Chromium public API.
Comment 4 Dana Jansens 2012-03-02 10:39:56 PST
Created attachment 129922 [details]
Patch

grammar fix.
Comment 5 James Robinson 2012-03-02 10:44:06 PST
Comment on attachment 129922 [details]
Patch

Do you want this for WebExternalTextureLayer as well?
Comment 6 Dana Jansens 2012-03-02 10:49:06 PST
(In reply to comment #5)
> (From update of attachment 129922 [details])
> Do you want this for WebExternalTextureLayer as well?

I think that's covered by its superclass WebLayer?
Comment 7 James Robinson 2012-03-02 11:03:04 PST
Comment on attachment 129922 [details]
Patch

Right, right.  R=me
Comment 8 WebKit Review Bot 2012-03-02 11:45:10 PST
Comment on attachment 129922 [details]
Patch

Clearing flags on attachment: 129922

Committed r109590: <http://trac.webkit.org/changeset/109590>
Comment 9 WebKit Review Bot 2012-03-02 11:45:15 PST
All reviewed patches have been landed.  Closing bug.