Refactor client visibility handling

services/ui/surfaces/surfaces_context_provider.h

 // Copyright 2014 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.
 
 #ifndef SERVICES_UI_SURFACES_SURFACES_CONTEXT_PROVIDER_H_
 #define SERVICES_UI_SURFACES_SURFACES_CONTEXT_PROVIDER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 32 matching lines @@
  public:
   explicit SurfacesContextProvider(gfx::AcceleratedWidget widget);
 
   void SetDelegate(SurfacesContextProviderDelegate* delegate);
 
   // cc::ContextProvider implementation.
   bool BindToCurrentThread() override;
   gpu::gles2::GLES2Interface* ContextGL() override;
   gpu::ContextSupport* ContextSupport() override;
   class GrContext* GrContext() override;
+  cc::ContextCacheController* CacheController() override;
   void InvalidateGrContext(uint32_t state) override;
   gpu::Capabilities ContextCapabilities() override;
-  void DeleteCachedResources() override {}
   void SetLostContextCallback(
       const LostContextCallback& lost_context_callback) override;
   base::Lock* GetLock() override;
 
   // SurfacesContextProvider API.
   void SetSwapBuffersCompletionCallback(
       gl::GLSurface::SwapCompletionCallback callback);
 
  protected:
   friend class base::RefCountedThreadSafe<SurfacesContextProvider>;
   ~SurfacesContextProvider() override;
 
  private:
   // Callbacks for CommandBufferProxyImpl:
   void OnGpuSwapBuffersCompleted(
       const std::vector<ui::LatencyInfo>& latency_info,
       gfx::SwapResult result,
       const gpu::GpuProcessHostedCALayerTreeParamsMac* params_mac);
   void OnUpdateVSyncParameters(base::TimeTicks timebase,
                                base::TimeDelta interval);
 
   // From GLES2Context:
   // Initialized in BindToCurrentThread.
   std::unique_ptr<gpu::gles2::GLES2CmdHelper> gles2_helper_;
   std::unique_ptr<gpu::TransferBuffer> transfer_buffer_;
   std::unique_ptr<gpu::gles2::GLES2Implementation> implementation_;
 

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

nit: no whitespace

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

Done.

+  std::unique_ptr<cc::ContextCacheController> cache_controller_;
+
   gpu::Capabilities capabilities_;
   LostContextCallback lost_context_callback_;
 
   SurfacesContextProviderDelegate* delegate_;
   gfx::AcceleratedWidget widget_;
   std::unique_ptr<gpu::CommandBufferProxyImpl> command_buffer_proxy_impl_;
   gl::GLSurface::SwapCompletionCallback swap_buffers_completion_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(SurfacesContextProvider);
 };
 
 }  // namespace ui
 
 #endif  // SERVICES_UI_SURFACES_SURFACES_CONTEXT_PROVIDER_H_