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

Unified Diff: ui/android/context_provider_factory.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
« no previous file with comments | « content/test/test_render_view_host.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/android/context_provider_factory.h
diff --git a/ui/android/context_provider_factory.h b/ui/android/context_provider_factory.h
index 8ad9d6f4a0c3eacef8a1e0982a974b337ceb4684..d0505894530f80132d240b996409eef5b001ec85 100644
--- a/ui/android/context_provider_factory.h
+++ b/ui/android/context_provider_factory.h
@@ -23,6 +23,7 @@ struct ContextCreationAttribHelper;
} // namespace gles
struct SharedMemoryLimits;
+class GpuChannelHost;
class GpuMemoryBufferManager;
} // namespace gpu
@@ -31,8 +32,20 @@ namespace ui {
// This class is not thread-safe and should only be accessed from the UI thread.
class UI_ANDROID_EXPORT ContextProviderFactory {
public:
- using ContextProviderCallback =
- base::Callback<void(const scoped_refptr<cc::ContextProvider>&)>;
+ enum class GpuChannelHostResult {
+ FAILURE_GPU_PROCESS_INITIALIZATION_FAILED,
+
+ // Used when the factory is shutting down. No more requests should be made
+ // to the factory after this response is dispatched.
+ FAILURE_FACTORY_SHUTDOWN,
+
+ // Set if the Context creation was successful.
+ SUCCESS,
+ };
+
+ using GpuChannelHostCallback =
+ base::Callback<void(scoped_refptr<gpu::GpuChannelHost>,
+ GpuChannelHostResult)>;
enum class ContextType {
BLIMP_RENDER_COMPOSITOR_CONTEXT,
@@ -50,18 +63,20 @@ class UI_ANDROID_EXPORT ContextProviderFactory {
virtual scoped_refptr<cc::VulkanContextProvider>
GetSharedVulkanContextProvider() = 0;
+ // The callback may be triggered synchronously if possible. If the creation
+ // fails, a null host is passed with the specified reason.
+ virtual void RequestGpuChannelHost(GpuChannelHostCallback callback) = 0;
+
// Creates an offscreen ContextProvider for the compositor. Any shared
// contexts passed here *must* have been created using this factory.
- // The callback may be triggered synchronously if possible, and will always
- // have the context provider.
- virtual void CreateOffscreenContextProvider(
+ virtual scoped_refptr<cc::ContextProvider> CreateOffscreenContextProvider(
ContextType context_type,
gpu::SharedMemoryLimits shared_memory_limits,
gpu::gles2::ContextCreationAttribHelper attributes,
bool support_locking,
bool automatic_flushes,
cc::ContextProvider* shared_context_provider,
- ContextProviderCallback result_callback) = 0;
+ scoped_refptr<gpu::GpuChannelHost> gpu_channel_host) = 0;
virtual cc::SurfaceManager* GetSurfaceManager() = 0;
« no previous file with comments | « content/test/test_render_view_host.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698