cc: Add checker-imaging support to TileManager.

cc/tiles/image_controller_unittest.cc

 // Copyright 2016 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 "base/bind.h"
 #include "base/optional.h"
 #include "base/run_loop.h"
 #include "base/test/test_simple_task_runner.h"
 #include "base/threading/sequenced_task_runner_handle.h"
 #include "cc/tiles/image_controller.h"
@@ skipping 429 matching lines @@
                      run_loop1.QuitClosure()));
   RunOrTimeout(&run_loop1);
   EXPECT_EQ(expected_id1, decode_client1.id());
   EXPECT_TRUE(task->has_run());
   EXPECT_EQ(1, cache()->number_of_refs());
 
   controller()->SetImageDecodeCache(nullptr);
   EXPECT_EQ(0, cache()->number_of_refs());
 }
 
+TEST_F(ImageControllerTest, DispatchesDecodeCallbacksAfterCacheChanged) {

[Khushal, 2017/02/17 19:05:38]

I added the test. Since I stumbled onto the multiple completion on TileTasks
while doing this, fixed that too. Do you want to take another look?

+  scoped_refptr<SimpleTask> task(new SimpleTask);
+  cache()->SetTaskToUse(task);
+
+  base::RunLoop run_loop1;
+  DecodeClient decode_client1;
+  base::RunLoop run_loop2;
+  DecodeClient decode_client2;
+
+  controller()->QueueImageDecode(
+      image(),
+      base::Bind(&DecodeClient::Callback, base::Unretained(&decode_client1),
+                 run_loop1.QuitClosure()));
+  controller()->QueueImageDecode(
+      image(),
+      base::Bind(&DecodeClient::Callback, base::Unretained(&decode_client2),
+                 run_loop2.QuitClosure()));
+
+  // Now reset the image cache before decode completed callbacks are posted to
+  // the compositor thread. Ensure that the completion callbacks for the decode
+  // is still run.
+  TestableCache new_cache;
+  controller()->SetImageDecodeCache(&new_cache);
+  RunOrTimeout(&run_loop1);
+  RunOrTimeout(&run_loop2);
+}
+
 }  // namespace
 }  // namespace cc