cc: Add image decode queue functionality to image manager.

cc/layers/layer_unittest.cc

 // Copyright 2011 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.
 
 #include "cc/layers/layer.h"
 
 #include <stddef.h>
 
 #include "base/threading/thread_task_runner_handle.h"
 #include "cc/animation/animation_host.h"
@@ skipping 115 matching lines @@
 
  protected:
   void SetUp() override {
     animation_host_ = AnimationHost::CreateForTesting(ThreadInstance::MAIN);
 
     LayerTreeHostInProcess::InitParams params;
     params.client = &fake_client_;
     params.settings = &settings_;
     params.task_graph_runner = &task_graph_runner_;
     params.mutator_host = animation_host_.get();
+    params.image_worker_task_runner = nullptr;

[enne (OOO), 2017/01/03 21:15:25]

Doesn't this default to nullptr?

[vmpstr, 2017/01/03 21:40:34]

Yes. Uh... Well, this is a relic of the past. As in, I went through a lot of
iterations some of which required this. In the end, I decided to leave it here
as an explicit "I know there's a param here called image_worker_task_runner, and
I'm OK with it being nullptr".

I can remove references to this if you prefer.

[enne (OOO), 2017/01/03 21:49:42]

Yah, I think I'd prefer them gone if they're not needed.  :)

[vmpstr, 2017/01/06 01:19:03]

Done.

 
     layer_tree_host_.reset(
         new StrictMock<MockLayerTreeHost>(&single_thread_client_, &params));
     layer_tree_ = static_cast<StrictMock<MockLayerTree>*>(
         layer_tree_host_->GetLayerTree());
   }
 
   void TearDown() override {
     Mock::VerifyAndClearExpectations(layer_tree_host_.get());
     Mock::VerifyAndClearExpectations(layer_tree_);
@@ skipping 891 matching lines @@
   }
 
   std::unique_ptr<LayerTreeHost> Create(LayerTreeSettings settings,
                                         MutatorHost* mutator_host) {
     LayerTreeHostInProcess::InitParams params;
     params.client = &client_;
     params.task_graph_runner = &task_graph_runner_;
     params.settings = &settings;
     params.main_task_runner = base::ThreadTaskRunnerHandle::Get();
     params.mutator_host = mutator_host;
+    params.image_worker_task_runner = nullptr;
 
     return LayerTreeHostInProcess::CreateSingleThreaded(&single_thread_client_,
                                                         &params);
   }
 
  private:
   FakeLayerTreeHostClient client_;
   StubLayerTreeHostSingleThreadClient single_thread_client_;
   TestTaskGraphRunner task_graph_runner_;
 };
@@ skipping 368 matching lines @@
   EXPECT_EQ(MutableProperty::kNone, impl_layer->mutable_properties());
 
   test_layer->PushPropertiesTo(impl_layer.get());
 
   EXPECT_EQ(ElementId(2, 0), impl_layer->element_id());
   EXPECT_EQ(MutableProperty::kTransform, impl_layer->mutable_properties());
 }
 
 }  // namespace
 }  // namespace cc