Remove threading from RendererGpuVideoAcceleratorFactories

media/base/bind_to_loop.h.pump

 $$ This is a pump file for generating file templates.  Pump is a python
 $$ script that is part of the Google Test suite of utilities.  Description
 $$ can be found here:
 $$
 $$ http://code.google.com/p/googletest/wiki/PumpManual
 $$
 
 $$ See comment for MAX_ARITY in base/bind.h.pump.
 $var MAX_ARITY = 7
 
 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_BIND_TO_LOOP_H_
 #define MEDIA_BASE_BIND_TO_LOOP_H_
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/message_loop/message_loop_proxy.h"
+#include "base/synchronization/waitable_event.h"
 
 // This is a helper utility for base::Bind()ing callbacks on to particular
 // MessageLoops.  A typical use is when |a| (of class |A|) wants to hand a
 // callback such as base::Bind(&A::AMethod, a) to |b|, but needs to ensure that
 // when |b| executes the callback, it does so on a particular MessageLoop.
 //
 // Typical usage: request to be called back on the current thread:
 // other->StartAsyncProcessAndCallMeBack(
 //    media::BindToLoop(MessageLoopProxy::current(),
 //                      base::Bind(&MyClass::MyMethod, this)));
@@ skipping 17 matching lines @@
 template <typename T, typename R>
 base::internal::PassedWrapper<scoped_ptr_malloc<T, R> > TrampolineForward(
     scoped_ptr_malloc<T, R>& p) { return base::Passed(&p); }
 
 template <typename T>
 base::internal::PassedWrapper<ScopedVector<T> > TrampolineForward(
     ScopedVector<T>& p) { return base::Passed(&p); }
 
 template <typename T> struct TrampolineHelper;
 
+// Caller helper to call a base::Closure synchronously
+void TrampolineSyncCaller(const base::Closure& closure,
+                          base::WaitableEvent* waiter);
+
 $range ARITY 0..MAX_ARITY
 $for ARITY [[
 $range ARG 1..ARITY
 
 template <$for ARG , [[typename A$(ARG)]]>
 struct TrampolineHelper<void($for ARG , [[A$(ARG)]])> {
   static void Run(
       const scoped_refptr<base::MessageLoopProxy>& loop,
       const base::Callback<void($for ARG , [[A$(ARG)]])>& cb
 $if ARITY != 0 [[, ]]
 $for ARG , [[A$(ARG) a$(ARG)]]
 ) {
     loop->PostTask(FROM_HERE, base::Bind(cb
 $if ARITY != 0 [[, ]]
 $for ARG , [[internal::TrampolineForward(a$(ARG))]]));
   }
+  static void RunSync(
+      const scoped_refptr<base::MessageLoopProxy>& loop,
+      const base::Callback<void($for ARG , [[A$(ARG)]])>& cb
+$if ARITY != 0 [[, ]]
+$for ARG , [[A$(ARG) a$(ARG)]]
+) {
+    base::WaitableEvent waiter(false, false);
+    loop->PostTask(
+        FROM_HERE,
+        base::Bind(
+            &TrampolineSyncCaller,
+            base::Bind(cb
+$if ARITY != 0 [[, ]]
+$for ARG , [[internal::TrampolineForward(a$(ARG))]]),
+            &waiter));
+    waiter.Wait();
+  }
 };
 
 
 ]]  $$ for ARITY
 
 }  // namespace internal
 
 template<typename T>
 static base::Callback<T> BindToLoop(
     const scoped_refptr<base::MessageLoopProxy>& loop,
     const base::Callback<T>& cb) {
   return base::Bind(&internal::TrampolineHelper<T>::Run, loop, cb);
 }
 
 template<typename T>
+static base::Callback<T> BindToLoopSync(
+    const scoped_refptr<base::MessageLoopProxy>& loop,
+    const base::Callback<T>& cb) {
+  DCHECK(!loop->BelongsToCurrentThread());

[Ami GONE FROM CHROMIUM, 2013/10/22 17:21:19]

This wants to live inside RunSync, not here, right?  
(it should be fine to bind a closure to the current loop, for synchronous
execution, where the outermost Run() (and thus Wait()) is happening on another
thread)

[sheu, 2013/10/24 01:16:39]

That's true.  Done.

+  return base::Bind(&internal::TrampolineHelper<T>::RunSync, loop, cb);
+}
+
+template<typename T>
 static base::Callback<T> BindToCurrentLoop(

[Ami GONE FROM CHROMIUM, 2013/10/22 17:21:19]

...which gives rise to wondering whether there's going to be a need for a
BindToCurrentLoopSync someday.  But apparently that day is not today, so no need
to add this :)

     const base::Callback<T>& cb) {
   return BindToLoop(base::MessageLoopProxy::current(), cb);
 }
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_BIND_TO_LOOP_H_