Fix PrepareTextureCopyOutputResult to avoid ScopedCallbackRunner.

base/callback_helpers.h

Index: base/callback_helpers.h
diff --git a/base/callback_helpers.h b/base/callback_helpers.h
index c4b0d867634751fbc3e0dfa3a4b0b7177def79e8..5dc9311de0a47210e23bb9a4b090996e09e99f43 100644
--- a/base/callback_helpers.h
+++ b/base/callback_helpers.h
@@ -37,15 +37,15 @@ class BASE_EXPORT ScopedClosureRunner {
 
   ScopedClosureRunner(ScopedClosureRunner&& other);
 
-  // Calls the current closure if it's set and replaces it with the closure from
-  // |other|.
+  // Releases the current closure if it's set and replaces it with the closure

[danakj, 2016/06/27 22:49:57]

It dchecks there's no closure, so this should say it can only be called on an
empty ScopedClosureRunner. However, I think it'd also be okay to make it do the
same thing as ReplaceClosure and drop the DCHECK.

[Sergey Ulanov, 2016/06/28 19:15:57]

Done.

+  // from |other|.
   ScopedClosureRunner& operator=(ScopedClosureRunner&& other);
 
   // Calls the current closure and resets it, so it wont be called again.
-  void Reset();
+  void RunNow();
 
-  // Calls the current closure and replaces it with the new one.
-  void Reset(const Closure& closure);
+  // Replaces closure with the new one releasing the old one without calling it.

[danakj, 2016/06/27 22:49:57]

The comment says it won't run it but the code does run it still.

[Sergey Ulanov, 2016/06/28 19:15:57]

Duh, yes, sorry

+  void ReplaceClosure(const Closure& closure);
 
   // Releases the Closure without calling.
   Closure Release() WARN_UNUSED_RESULT;