Removed GPU plugin....

Removed GPU plugin.

Pepper 3D v2 does not use the GPU plugin. It is integrated with the accelerated compositor.

TEST=PPAPI 3D v2 still works, trybots
BUG=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=76840

[piman, 2011-03-01 22:16:00]

LGTM. I don't think we use the WebPluginDelegatePepper any more at all, so maybe
all of it could go ?

[jam, 2011-03-04 05:33:40]

On 2011/03/01 22:16:00, piman wrote:
> LGTM. I don't think we use the WebPluginDelegatePepper any more at all, so
maybe
> all of it could go ?

ping?  if it's not used, can we take it out?

[apatrick, 2011-03-04 17:49:17]

Sure, if it's not used anymore.

[apatrick_chromium, 2011-03-09 19:25:34]

On 2011/03/04 05:33:40, John Abd-El-Malek wrote:
> On 2011/03/01 22:16:00, piman wrote:
> > LGTM. I don't think we use the WebPluginDelegatePepper any more at all, so
> maybe
> > all of it could go ?
> 
> ping?  if it's not used, can we take it out?

I tried removing WebPluginDelegatePepper. Unfortunately it's still used by the
PDF plugin. See change to render_view.cc here:

http://codereview.chromium.org/6646025/

[jam, 2011-03-09 20:23:40]

On Wed, Mar 9, 2011 at 11:25 AM, <apatrick@chromium.org> wrote:

> Reviewers: piman, John Abd-El-Malek, apatrick,
>
> Message:
>
> On 2011/03/04 05:33:40, John Abd-El-Malek wrote:
>
>> On 2011/03/01 22:16:00, piman wrote:
>> > LGTM. I don't think we use the WebPluginDelegatePepper any more at all,
>> so
>> maybe
>> > all of it could go ?
>>
>
>  ping?  if it's not used, can we take it out?
>>
>
> I tried removing WebPluginDelegatePepper. Unfortunately it's still used by
> the
> PDF plugin. See change to render_view.cc here:
>
> http://codereview.chromium.org/6646025/


This is left over code, not used anymore.  PDFs now don't create a
WebPluginDelegate (which is used by WebPluginImpl).  They use another
WebPlugin object directly, i.e.
CreatePepperPlugin in RenderView::createPlugin


>
>
>
> Description:
> Removed GPU plugin.
>
> Pepper 3D v2 does not use the GPU plugin. It is integrated with the
> accelerated
> compositor.
>
> TEST=PPAPI 3D v2 still works, trybots
> BUG=none
>
>
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=76840
>
>
> Please review this at http://codereview.chromium.org/6588090/
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  M     chrome/app/chrome_main.cc
>  M     chrome/chrome.gyp
>  M     chrome/chrome_common.gypi
>  M     chrome/chrome_tests.gypi
>  M     chrome/common/chrome_switches.h
>  M     chrome/common/chrome_switches.cc
>  D     chrome/common/gpu_plugin.h
>  D     chrome/common/gpu_plugin.cc
>  M     chrome/common/plugin_messages_internal.h
>  D     chrome/plugin/command_buffer_stub.h
>  D     chrome/plugin/command_buffer_stub.cc
>  D     chrome/plugin/command_buffer_stub_win.cc
>  M     chrome/plugin/plugin_main.cc
>  M     chrome/plugin/webplugin_delegate_stub.h
>  M     chrome/plugin/webplugin_delegate_stub.cc
>  M     chrome/renderer/webplugin_delegate_pepper.h
>  M     chrome/renderer/webplugin_delegate_pepper.cc
>  M     chrome/renderer/webplugin_delegate_proxy.h
>  M     chrome/renderer/webplugin_delegate_proxy.cc
>  D     chrome/test/ui/pepper_uitest.cc
>  M     content/browser/plugin_process_host.cc
>  M     content/browser/plugin_service.cc
>  M     gpu/gpu.gyp
>  D     gpu/gpu_plugin/gpu_plugin.h
>  D     gpu/gpu_plugin/gpu_plugin.cc
>  M     webkit/glue/webkit_glue.gypi
>
>
>

[apatrick_chromium, 2011-03-09 21:52:00]

Awesome, I'll keep deleting code then :)

[jam, 2011-03-09 21:53:42]

great, i really like the cl below!

On Wed, Mar 9, 2011 at 1:52 PM, <apatrick@chromium.org> wrote:

> Awesome, I'll keep deleting code then :)
>
>
> http://codereview.chromium.org/6588090/
>