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

Unified Diff: ui/android/context_provider_factory.h

Issue 2190033002: content: Add ContextProviderFactory to create a render ContextProvider. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: .. Created 4 years, 5 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: ui/android/context_provider_factory.h
diff --git a/ui/android/context_provider_factory.h b/ui/android/context_provider_factory.h
new file mode 100644
index 0000000000000000000000000000000000000000..c2db31eacb45b938b829038b87b684b89d6e13a5
--- /dev/null
+++ b/ui/android/context_provider_factory.h
@@ -0,0 +1,77 @@
+// 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 UI_ANDROID_CONTEXT_PROVIDER_FACTORY_H_
+#define UI_ANDROID_CONTEXT_PROVIDER_FACTORY_H_
+
+#include "base/callback.h"
+#include "base/memory/ref_counted.h"
+#include "ui/android/ui_android_export.h"
+#include "ui/gfx/native_widget_types.h"
+
+namespace cc {
+class ContextProvider;
+class GpuMemoryBufferManager;
+class VulkanContextProvider;
+class SharedBitmapManager;
+class SurfaceManager;
+}
+
+namespace gpu {
+namespace gles2 {
+struct ContextCreationAttribHelper;
+} // namespace gles
+
+struct SharedMemoryLimits;
+class GpuMemoryBufferManager;
+} // namespace gpu
+
+namespace ui {
+
+// This class is not thread-safe and should only be accessed from the UI thread.
+class UI_ANDROID_EXPORT ContextProviderFactory {
+ public:
+ struct ContextProviders {
+ ContextProviders();
+ ~ContextProviders();
+ scoped_refptr<cc::ContextProvider> compositor_context_provider;
+ scoped_refptr<cc::ContextProvider> worker_context_provider;
+ };
+
+ using ContextProviderCallback = base::Callback<void(const ContextProviders&)>;
+
+ static ContextProviderFactory* GetInstance();
+
+ // This should only be called once, on startup. Ownership remains with the
+ // caller.
+ static void SetInstance(ContextProviderFactory* context_provider_factory);
+
+ virtual ~ContextProviderFactory(){};
no sievers 2016/08/02 20:44:34 nit: space missing in between () and {}
Khushal 2016/08/02 22:22:16 Done.
+
+ virtual scoped_refptr<cc::VulkanContextProvider>
+ GetSharedVulkanContextProvider() = 0;
+
+ // Creates a ContextProvider for the compositor. If the |widget| is
+ // set, then an onscreen context for display is created, otherwise this will
+ // return an offscreen context.
+ virtual void RequestContextProviders(
no sievers 2016/08/02 20:44:34 nit: I think 'Create' is more precise than 'Reques
Khushal 2016/08/02 22:22:16 Done.
+ gfx::AcceleratedWidget widget,
+ gpu::SharedMemoryLimits shared_memory_limits,
+ gpu::gles2::ContextCreationAttribHelper attributes,
+ bool support_locking,
+ bool automatic_flushes,
+ ContextProviderCallback result_callback) = 0;
+
+ virtual cc::SurfaceManager* GetSurfaceManager() = 0;
+
+ virtual uint32_t AllocateSurfaceClientId() = 0;
+
+ virtual cc::SharedBitmapManager* GetSharedBitmapManager() = 0;
Khushal 2016/08/01 21:29:10 Added these too, we'll need them for Blimp anyway.
+
+ virtual gpu::GpuMemoryBufferManager* GetGpuMemoryBufferManager() = 0;
+};
+
+} // namespace ui
+
+#endif // UI_ANDROID_CONTEXT_PROVIDER_FACTORY_H_

Powered by Google App Engine
This is Rietveld 408576698