Refactor client visibility handling

cc/test/test_context_provider.h

Index: cc/test/test_context_provider.h
diff --git a/cc/test/test_context_provider.h b/cc/test/test_context_provider.h
index d7c5627f4781e5f13a47704a8c73cb90f37517a6..bd1f0476f8e9ee28c15c1dac11b6e360e5b19fcd 100644
--- a/cc/test/test_context_provider.h
+++ b/cc/test/test_context_provider.h
@@ -47,9 +47,9 @@ class TestContextProvider : public ContextProvider {
   gpu::gles2::GLES2Interface* ContextGL() override;
   gpu::ContextSupport* ContextSupport() override;
   class GrContext* GrContext() override;
+  ContextCacheController* CacheController() override;
   void InvalidateGrContext(uint32_t state) override;
   base::Lock* GetLock() override;
-  void DeleteCachedResources() override;
   void SetLostContextCallback(const LostContextCallback& cb) override;
 
   TestWebGraphicsContext3D* TestContext3d();
@@ -85,6 +85,8 @@ class TestContextProvider : public ContextProvider {
   LostContextCallback lost_context_callback_;
   sk_sp<class GrContext> gr_context_;
 
+  std::unique_ptr<ContextCacheController> cache_controller_;

[danakj, 2016/08/26 23:49:08]

Hm, kinda weird that gr_context_ and this are way at the bottom, can they be up
with context_gl_?

[ericrk, 2016/08/29 22:43:22]

Done.

+
   base::WeakPtrFactory<TestContextProvider> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TestContextProvider);