Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(35)

Unified Diff: chrome/browser/android/blimp/chrome_compositor_dependencies.h

Issue 2297933002: blimp: Set up the CompositorDependencies for blimp in Chrome. (Closed)
Patch Set: retry gpu process failures after crbug.com/643282 Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/blimp/chrome_compositor_dependencies.h
diff --git a/chrome/browser/android/blimp/chrome_compositor_dependencies.h b/chrome/browser/android/blimp/chrome_compositor_dependencies.h
new file mode 100644
index 0000000000000000000000000000000000000000..fcf5903f64727204bc8d736d22476acc9fc9041f
--- /dev/null
+++ b/chrome/browser/android/blimp/chrome_compositor_dependencies.h
@@ -0,0 +1,47 @@
+// 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.
+
+#ifndef CHROME_BROWSER_ANDROID_BLIMP_CHROME_COMPOSITOR_DEPENDENCIES_H_
+#define CHROME_BROWSER_ANDROID_BLIMP_CHROME_COMPOSITOR_DEPENDENCIES_H_
+
+#include "base/macros.h"
+#include "base/memory/weak_ptr.h"
+#include "blimp/client/public/compositor/compositor_dependencies.h"
+#include "ui/android/context_provider_factory.h"
+
+// A wrapper for the Blimp compositor dependencies that passes through to the
+// ui::ContextProviderFactory. The ContextProviderFactory must outlive this
+// class.
+class ChromeCompositorDependencies
+ : public blimp::client::CompositorDependencies {
+ public:
+ ChromeCompositorDependencies(
+ ui::ContextProviderFactory* context_provider_factory);
+ ~ChromeCompositorDependencies() override;
+
+ // CompositorDependencies implementation.
+ gpu::GpuMemoryBufferManager* GetGpuMemoryBufferManager() override;
+ cc::SurfaceManager* GetSurfaceManager() override;
+ uint32_t AllocateSurfaceClientId() override;
+ void GetContextProviders(const ContextProviderCallback& callback) override;
+
+ private:
+ void OnGpuChannelEstablished(
+ const ContextProviderCallback& callback,
+ scoped_refptr<gpu::GpuChannelHost> gpu_channel_host,
+ ui::ContextProviderFactory::GpuChannelHostResult result);
+
+ void HandlePendingRequests(gpu::GpuChannelHost* gpu_channel_host);
+
+ ui::ContextProviderFactory* context_provider_factory_;
+
+ // Worker context shared across all Blimp Compositors.
+ scoped_refptr<cc::ContextProvider> shared_main_thread_worker_context_;
+
+ base::WeakPtrFactory<ChromeCompositorDependencies> weak_factory_;
+
+ DISALLOW_COPY_AND_ASSIGN(ChromeCompositorDependencies);
+};
+
+#endif // CHROME_BROWSER_ANDROID_BLIMP_CHROME_COMPOSITOR_DEPENDENCIES_H_

Powered by Google App Engine
This is Rietveld 408576698