android_webview: Remove dependency on webkit's ContextProviderInProcess.

android_webview/browser/context_provider_in_process.h

-// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Copyright 2015 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 WEBKIT_COMMON_GPU_CONTEXT_PROVIDER_IN_PROCESS_H_
-#define WEBKIT_COMMON_GPU_CONTEXT_PROVIDER_IN_PROCESS_H_
+#ifndef ANDROID_WEBVIEW_BROWSER_CONTEXT_PROVIDER_IN_PROCESS_H_
+#define ANDROID_WEBVIEW_BROWSER_CONTEXT_PROVIDER_IN_PROCESS_H_
 
 #include <string>
 
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/synchronization/lock.h"
 #include "base/threading/thread_checker.h"
 #include "cc/output/context_provider.h"
-#include "gpu/command_buffer/common/gles2_cmd_utils.h"
+#include "gpu/command_buffer/service/in_process_command_buffer.h"
 #include "skia/ext/refptr.h"
-#include "ui/gfx/native_widget_types.h"
+
+namespace gfx {
+class GLSurface;
+}
 
 namespace gpu {
 class GLInProcessContext;
 }
 
-namespace ui {
+namespace android_webview {
 
-class InProcessContextProvider : public cc::ContextProvider {
+class ContextProviderInProcess : public cc::ContextProvider {

[boliu, 2015/04/20 10:06:05]

Call this a different name to avoid confusion, maybe
AwRenderThreadContextProvider

[tfarina, 2015/04/22 02:09:04]

Done.

  public:
-  static scoped_refptr<InProcessContextProvider> Create(
-      const gpu::gles2::ContextCreationAttribHelper& attribs,
-      bool lose_context_when_out_of_memory,
-      gfx::AcceleratedWidget window,
+  static scoped_refptr<ContextProviderInProcess> Create(
+      scoped_refptr<gfx::GLSurface> surface,
+      scoped_refptr<gpu::InProcessCommandBuffer::Service> service,
       const std::string& debug_name);
 
-  // Uses default attributes for creating an offscreen context.
-  static scoped_refptr<InProcessContextProvider> CreateOffscreen(
-      bool lose_context_when_out_of_memory);
-
  private:
-  InProcessContextProvider(
-      const gpu::gles2::ContextCreationAttribHelper& attribs,
-      bool lose_context_when_out_of_memory,
-      gfx::AcceleratedWidget window,
+  ContextProviderInProcess(
+      scoped_refptr<gfx::GLSurface> surface,
+      scoped_refptr<gpu::InProcessCommandBuffer::Service> service,
       const std::string& debug_name);
-  ~InProcessContextProvider() override;
+  ~ContextProviderInProcess() override;
 
   // cc::ContextProvider:
   bool BindToCurrentThread() override;
   Capabilities ContextCapabilities() override;
   gpu::gles2::GLES2Interface* ContextGL() override;
   gpu::ContextSupport* ContextSupport() override;
   class GrContext* GrContext() override;
   void SetupLock() override;
   base::Lock* GetLock() override;
   bool IsContextLost() override;
   void VerifyContexts() override;
   void DeleteCachedResources() override;
   bool DestroyedOnMainThread() override;
   void SetLostContextCallback(
       const LostContextCallback& lost_context_callback) override;
   void SetMemoryPolicyChangedCallback(
       const MemoryPolicyChangedCallback& memory_policy_changed_callback)
       override;
 
   void OnLostContext();
 
   base::ThreadChecker main_thread_checker_;
   base::ThreadChecker context_thread_checker_;

[boliu, 2015/04/20 10:06:05]

We can assume main_thread and context_thread is always the same. So only need
one thread_checker, and don't need destroyed_lock_.

[tfarina, 2015/04/22 02:09:04]

Done.

 
+  scoped_refptr<gfx::GLSurface> surface_;
+  scoped_refptr<gpu::InProcessCommandBuffer::Service> service_;

[boliu, 2015/04/20 10:06:05]

You can create GLInProcessContext in the constructor, so no need to hold these
variables.

[tfarina, 2015/04/22 02:09:04]

Done.

   scoped_ptr<gpu::GLInProcessContext> context_;
   skia::RefPtr<class GrContext> gr_context_;
 
-  gpu::gles2::ContextCreationAttribHelper attribs_;
-  bool lose_context_when_out_of_memory_;
-  gfx::AcceleratedWidget window_;
   std::string debug_name_;
   cc::ContextProvider::Capabilities capabilities_;
 
   LostContextCallback lost_context_callback_;
 
   base::Lock destroyed_lock_;
   bool destroyed_;
 
   base::Lock context_lock_;
 
-  DISALLOW_COPY_AND_ASSIGN(InProcessContextProvider);
+  DISALLOW_COPY_AND_ASSIGN(ContextProviderInProcess);
 };
 
-}  // namespace ui
+}  // namespace android_webview
 
-#endif  // WEBKIT_COMMON_GPU_CONTEXT_PROVIDER_IN_PROCESS_H_
+#endif  // ANDROID_WEBVIEW_BROWSER_CONTEXT_PROVIDER_IN_PROCESS_H_