[Webkit-unassigned] [Bug 48032] [chromium] Added PluginLayerChromium for hardware accelerated compositing of plugins

[bugzilla-daemon at webkit.org]

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





--- Comment #16 from Al <apatrick at chromium.org>  2010-10-27 16:02:21 PST ---
(In reply to comment #13)
> (From update of attachment 71809 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=71809&action=review
> 
> This looks good to me. I'm only marking it r- because of the copyright headers. Please update them and re-upload the patch.
> 
> > WebCore/platform/graphics/chromium/PluginLayerChromium.cpp:28
> > + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> 
> It turns out that this is the wrong form of license header for contributed code to WebKit. Even though other files in this directory are also incorrect, let's not propagate the error further. You can find the correct header under WebKit/LICENSE or a C++ formatted version with the Google copyright in e.g. WebKit/chromium/tests/PODIntervalTreeTest.cpp.
> 

Done

> > WebCore/platform/graphics/chromium/PluginLayerChromium.h:28
> > + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> 
> Fix copyright header here too.
>

Done

> > WebKit/chromium/public/WebPlugin.h:68
> > +    // If the plugin instance is backed by an OpenGL, return its ID in the
> 
> by an OpenGL -> by an OpenGL context (?)
> 

Done

> > WebKit/chromium/public/WebPluginContainer.h:57
> > +    // If the plugin instance is backed by an OpenGL, return its ID in the
> 
> Same as above.
> 

Done

> > WebKit/chromium/src/WebPluginContainerImpl.cpp:291
> > +#if USE(ACCELERATED_COMPOSITING)
> 
> If the intent is to make WebPluginContainer::getBackingTextureId() pure virtual then you need an alternate path when USE(ACCELERATED_COMPOSITING) is not enabled. Perhaps conditionalize the method bodies.
> 

As fishd advised, this will not be pure virtual.

> > WebKit/chromium/src/WebPluginContainerImpl.h:92
> > +#if USE(ACCELERATED_COMPOSITING)
> 
> Same comment as above about USE(ACCELERATED_COMPOSITING).

As above.

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