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

Unified Diff: ui/ozone/public/client_pixmap.h

Issue 1128113011: ozone: Introduce ClientPixmap and ClientPixmapFactory for non-GPU processes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Introduce ClientPixmap and rename to ClientPixmapManager Created 5 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/ozone/public/client_pixmap.h
diff --git a/ui/ozone/public/client_pixmap.h b/ui/ozone/public/client_pixmap.h
new file mode 100644
index 0000000000000000000000000000000000000000..9e288a83eeb5bdb5e86df23bc82d5436074874b9
--- /dev/null
+++ b/ui/ozone/public/client_pixmap.h
@@ -0,0 +1,26 @@
+// 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 UI_OZONE_PUBLIC_CLIENT_PIXMAP_H_
+#define UI_OZONE_PUBLIC_CLIENT_PIXMAP_H_
+
+#include "ui/ozone/ozone_export.h"
+
+namespace ui {
+
+// This represents a buffer that can be written to directly by regular CPU code
+// in non-GPU processes, but can also be read by the GPU.
reveman 2015/07/30 16:14:10 "non-GPU processes", do we need to be this specifi
dshwang 2015/07/30 16:50:15 That's true, though we don't have the use case yet
+// NativePixmap is its counterpart in GPU process.
+class OZONE_EXPORT ClientPixmap {
reveman 2015/07/30 16:14:10 Why the "Native" prefix in one case but not the ot
dshwang 2015/07/30 16:50:15 Ok, I remain this up to spang. spang, which do you
dshwang 2015/07/31 10:41:04 Ok, NativePixmap and ClientNativePixmap are chosen
+ public:
+ virtual ~ClientPixmap() {}
+
+ virtual void* Map() = 0;
+ virtual void Unmap() = 0;
+ virtual int GetStride() const = 0;
+};
+
+} // namespace ui
+
+#endif // UI_OZONE_PUBLIC_CLIENT_PIXMAP_H_

Powered by Google App Engine
This is Rietveld 408576698