Summary: | [chromium] Expose WebVideoLayer to Platform API and port WebMediaPlayerClientImpl to using it | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | James Robinson <jamesr> | ||||
Component: | New Bugs | Assignee: | James Robinson <jamesr> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, cc-bugs, dglazkov, enne, eric.carlson, feature-media-reviews, fischman, fishd, scherkus, sievers, tkent+wkapi, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
James Robinson
2012-04-13 17:16:41 PDT
Created attachment 137186 [details]
Patch
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI. Daniel or Ami - would one of you mind taking a careful read of the threading comments I added in WebVideoFrameProvider.h and making sure they are correct and sufficient for somebody to safely use the interface without needing to look too much at the implementation? (In reply to comment #3) > Daniel or Ami - would one of you mind taking a careful read of the threading comments I added in WebVideoFrameProvider.h and making sure they are correct and sufficient for somebody to safely use the interface without needing to look too much at the implementation? The comments sound good to me. Comment on attachment 137186 [details]
Patch
R=me. Go, go, gtfo.
Committed r114335: <http://trac.webkit.org/changeset/114335> |