This CL ensures that plugins always peek in the context of outgoing sync calls.

I will be watching the reliability test runs closely for any crashes which creep in due to reentrancies into plugins caused by this CL. 

This fixes bug http://code.google.com/p/chromium/issues/detail?id=15985

It is a touch tricky to implement a test case for this. Will add one hopefully in a subsequent CL

Bug=15985
Test=Covered by UI test

Review URL: http://codereview.chromium.org/173211

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24266 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 10 insertions, 0 deletions

diff --git a/chrome/test/ui/npapi_uitest.cc b/chrome/test/ui/npapi_uitest.cc
index eb19042..a4fa299 100644
--- a/chrome/test/ui/npapi_uitest.cc
+++ b/chrome/test/ui/npapi_uitest.cc
@@ -298,3 +298,13 @@ TEST_F(NPAPIVisiblePluginTester, MultipleInstancesSyncCalls) {
                 kTestCompleteSuccess, kShortWaitTimeout);
 }
 
+TEST_F(NPAPIVisiblePluginTester, EnsurePluginsPumpInSyncCalls) {
+  if (UITest::in_process_renderer())
+    return;
+
+  GURL url = GetTestUrl(L"npapi",
+                        L"ensure_plugins_pump_messages_in_sync_calls.html");
+  NavigateToURL(url);
+  WaitForFinish("src_plugin_for_outgoing_sync_call", "1", url,
+                kTestCompleteCookie, kTestCompleteSuccess, kShortWaitTimeout);
+}