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

Unified Diff: ui/ozone/platform/dri/buffer_data.cc

Issue 325133003: Refactor BufferData class into a separate file to be later used by individual gbm_bo's in EGLImage. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nits Created 6 years, 6 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 | « ui/ozone/platform/dri/buffer_data.h ('k') | ui/ozone/platform/dri/gbm.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/ozone/platform/dri/buffer_data.cc
diff --git a/ui/ozone/platform/dri/buffer_data.cc b/ui/ozone/platform/dri/buffer_data.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a2ea1b38cc2397b3b42905e8726430321905aeb5
--- /dev/null
+++ b/ui/ozone/platform/dri/buffer_data.cc
@@ -0,0 +1,71 @@
+// Copyright 2014 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.
+
+#include "ui/ozone/platform/dri/buffer_data.h"
+
+#include <gbm.h>
+
+#include "base/logging.h"
+#include "ui/ozone/platform/dri/dri_wrapper.h"
+
+namespace ui {
+
+// Pixel configuration for the current buffer format.
+// TODO(dnicoara) These will need to change once we query the hardware for
+// supported configurations.
+const uint8_t kColorDepth = 24;
+const uint8_t kPixelDepth = 32;
+
+BufferData::BufferData(DriWrapper* dri, gbm_bo* buffer)
+ : dri_(dri),
+ handle_(gbm_bo_get_handle(buffer).u32),
+ framebuffer_(0) {
+ // Register the buffer with the controller. This will allow us to scan out the
+ // buffer once we're done drawing into it. If we can't register the buffer
+ // then there's no point in having BufferData associated with it.
+ if (!dri_->AddFramebuffer(gbm_bo_get_width(buffer),
+ gbm_bo_get_height(buffer),
+ kColorDepth,
+ kPixelDepth,
+ gbm_bo_get_stride(buffer),
+ handle_,
+ &framebuffer_)) {
+ LOG(ERROR) << "Failed to register buffer";
+ }
+}
+
+BufferData::~BufferData() {
+ if (framebuffer_)
+ dri_->RemoveFramebuffer(framebuffer_);
+}
+
+// static
+BufferData* BufferData::CreateData(DriWrapper* dri,
+ gbm_bo* buffer) {
+ BufferData* data = new BufferData(dri, buffer);
+ if (!data->framebuffer()) {
+ delete data;
+ return NULL;
+ }
+
+ // GBM can destroy the buffers at any time as long as they aren't locked. This
+ // sets a callback such that we can clean up all our state when GBM destroys
+ // the buffer.
+ gbm_bo_set_user_data(buffer, data, BufferData::Destroy);
+
+ return data;
+}
+
+// static
+void BufferData::Destroy(gbm_bo* buffer, void* data) {
+ BufferData* bd = static_cast<BufferData*>(data);
+ delete bd;
+}
+
+// static
+BufferData* BufferData::GetData(gbm_bo* buffer) {
+ return static_cast<BufferData*>(gbm_bo_get_user_data(buffer));
+}
+
+} // namespace ui
« no previous file with comments | « ui/ozone/platform/dri/buffer_data.h ('k') | ui/ozone/platform/dri/gbm.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698