Another attempt at landing this patch. ...

Another attempt at landing this patch. 

The reliability tests regressions caused by this patch have been addressed by the upstream
bug fix https://bugs.webkit.org/show_bug.cgi?id=27769

The IPCs for carrying data requested by plugins have been changed from
synchronous IPCs to asynchronous IPCs. 

This fixes bug http://code.google.com/p/chromium/issues/detail?id=14323, where
the Flash plugin would not render
content on the page if these IPCs were processed while the plugin waited for
sync calls like NPN_Evaluate to
return.

Test=covered by UI tests.

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

[huanr, 2009-07-29 18:12:46]

Is this just the revert of revert? If so rubber stamp LGTM

Huan

On Wed, Jul 29, 2009 at 11:04 AM, <ananta@chromium.org> wrote:
> Reviewers: huanr,
>
> Description:
> Another attempt at landing this patch.
>
> The reliability tests regressions caused by this patch have been
> addressed by the upstream
> bug fix https://bugs.webkit.org/show_bug.cgi?id=3D27769
>
> The IPCs for carrying data requested by plugins have been changed from
> synchronous IPCs to asynchronous IPCs.
>
> This fixes bug http://code.google.com/p/chromium/issues/detail?id=3D14323=
,
> where
> the Flash plugin would not render
> content on the page if these IPCs were processed while the plugin waited
> for
> sync calls like NPN_Evaluate to
> return.
>
> Test=3Dcovered by UI tests.
>
> Bug=3D14323
>
>
> Please review this at http://codereview.chromium.org/160338
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
> =A0M =A0 =A0 chrome/common/plugin_messages_internal.h
> =A0M =A0 =A0 chrome/plugin/webplugin_delegate_stub.h
> =A0M =A0 =A0 chrome/plugin/webplugin_delegate_stub.cc
> =A0M =A0 =A0 chrome/plugin/webplugin_proxy.cc
> =A0M =A0 =A0 chrome/renderer/webplugin_delegate_proxy.cc
> =A0M =A0 =A0 webkit/glue/plugins/plugin_instance.cc
> =A0M =A0 =A0 webkit/glue/plugins/plugin_stream_url.h
> =A0M =A0 =A0 webkit/glue/plugins/plugin_stream_url.cc
> =A0M =A0 =A0 webkit/glue/plugins/test/plugin_get_javascript_url_test.h
> =A0M =A0 =A0 webkit/glue/plugins/test/plugin_get_javascript_url_test.cc
> =A0M =A0 =A0 webkit/glue/webplugin.h
> =A0M =A0 =A0 webkit/glue/webplugin_impl.cc
>
>
>