Summary: | Allows plugins to participate in accelerated compositing | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> | ||||||
Component: | Layout and Rendering | Assignee: | Simon Fraser (smfr) <simon.fraser> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, eric | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Mac | ||||||||
OS: | OS X 10.5 | ||||||||
Attachments: |
|
Description
Simon Fraser (smfr)
2010-01-10 17:22:08 PST
Created attachment 46248 [details]
Part 1 patch: some preparatory renaming
Comment on attachment 46248 [details] Part 1 patch: some preparatory renaming > - videoGraphicsLayer->setContentsToVideo((PlatformLayer *)m_qtVideoLayer.get()); > + videoGraphicsLayer->setContentsToMedia((PlatformLayer *)m_qtVideoLayer.get()); > - videoGraphicsLayer->setContentsToVideo((PlatformLayer *)m_qtVideoLayer.get()); > + videoGraphicsLayer->setContentsToMedia((PlatformLayer *)m_qtVideoLayer.get()); C++ style cast instead? Comment on attachment 46248 [details] Part 1 patch: some preparatory renaming http://trac.webkit.org/changeset/53059 Created attachment 46312 [details]
Main patch
Maybe Adam knows why all the bubbles are red here? Never mind, looks like the patch just failed to apply, which is why it's red everywhere. I think Adam was going to change that to purple at some point. I think it doesn't like that my patch is missing the base64-encoded PNG images. |