Provide TaskRunner to ContextCacheController

ui/compositor/test/in_process_context_provider.cc

Index: ui/compositor/test/in_process_context_provider.cc
diff --git a/ui/compositor/test/in_process_context_provider.cc b/ui/compositor/test/in_process_context_provider.cc
index ba3dde4fb1f03a36b074770ea8189b1daf695032..59998ca00fb1580bfdade7504b24bdd164ed24ab 100644
--- a/ui/compositor/test/in_process_context_provider.cc
+++ b/ui/compositor/test/in_process_context_provider.cc
@@ -90,13 +90,13 @@ bool InProcessContextProvider::BindToCurrentThread() {
         !window_, /* is_offscreen */
         window_, (shared_context_ ? shared_context_->context_.get() : nullptr),
         attribs_, gpu::SharedMemoryLimits(), gpu_memory_buffer_manager_,
-        image_factory_));
+        image_factory_, nullptr /* task_runner */));

[danakj, 2016/09/19 18:59:17]

I think you should pass non-null cuz otherwise if a test does lose context in
future it'll just crash. Null was meant to be for the |surface| isn't null case
right?

[ericrk, 2016/09/19 20:00:27]

Yeah - fair enough - I was thinking this was fine as long as tests don't use it,
but you're right, it's quite brittle.

 
     if (!context_)
       return false;
 
-    cache_controller_.reset(
-        new cc::ContextCacheController(context_->GetImplementation()));
+    cache_controller_.reset(new cc::ContextCacheController(
+        context_->GetImplementation(), nullptr /* task_runner */));
   }
 
   std::string unique_context_name =