PPAPI: Refactor ppapi test callbacks to ease testing blocking callbacks.

ppapi/proxy/ppb_network_monitor_private_proxy.cc

Index: ppapi/proxy/ppb_network_monitor_private_proxy.cc
diff --git a/ppapi/proxy/ppb_network_monitor_private_proxy.cc b/ppapi/proxy/ppb_network_monitor_private_proxy.cc
index 4fa2d6eec240fd16cd8c7ff12e8cbd5d5cd5a93c..e54f0ed4cdaea53ad39839ca8e56342ecc8d81af 100644
--- a/ppapi/proxy/ppb_network_monitor_private_proxy.cc
+++ b/ppapi/proxy/ppb_network_monitor_private_proxy.cc
@@ -98,9 +98,9 @@ PP_Resource PPB_NetworkMonitor_Private_Proxy::CreateProxyResource(
     // here.
     proxy->current_list_ = NULL;
   } else if (proxy->current_list_.get()) {
-    MessageLoop::current()->PostTask(FROM_HERE, base::Bind(
+    MessageLoop::current()->PostTask(FROM_HERE, RunWhileLocked(base::Bind(

[dmichael (off chromium), 2012/04/02 21:15:02]

somewhat unrelated, but causes problems running the tests locally for me, since
I'm building with enable_pepper_threading=1.

[bbudge, 2012/04/04 19:03:00]

Does it also cause problems in normal use? Is there a performance cost for
locking?

[dmichael (off chromium), 2012/04/04 19:50:47]

Yes, but locking is not enabled yet, so RunWhileLocked ultimately just adds a
little function-call overhead. We'll have to do some benchmarking or something
to make sure locking is fast enough, and Graphics3D is going to need some help
to avoid too much locking. In this case, the cost of locking is negligible,
since NetworkLists aren't going to change very frequently.

The real problem here is one of correctness, when locking is enabled. The design
is to always hold the lock while in proxy code. Note that in
OnNetworkListReceived, we unlock the mutex before calling out to the plugin. The
code before this patch means we enter the proxy when the task is run and invokes
OnNetworkListReceived, and we do not have the lock. This is bad because (a) we
probably will have concurrency problems, and (b) we try to _unlock_ the lock in
OnNetworkListReceived, which fails if we don't own the lock.

         &NetworkMonitor::OnNetworkListReceived,
-        result->AsWeakPtr(), proxy->current_list_));
+        result->AsWeakPtr(), proxy->current_list_)));
   }
 
   return result.release()->GetReference();