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

Unified Diff: ui/ozone/platform/drm/gpu/gbm_buffer_base.cc

Issue 2705733003: ozone: Don't assume AddFB succeeds when importing buffers. (Closed)
Patch Set: Created 3 years, 10 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/drm/gpu/gbm_buffer.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/ozone/platform/drm/gpu/gbm_buffer_base.cc
diff --git a/ui/ozone/platform/drm/gpu/gbm_buffer_base.cc b/ui/ozone/platform/drm/gpu/gbm_buffer_base.cc
index 934b4741b42bcf3b44a6208f26539d76f385bcb7..eb0b9009349fef71bfba41624f5deacaa660bd94 100644
--- a/ui/ozone/platform/drm/gpu/gbm_buffer_base.cc
+++ b/ui/ozone/platform/drm/gpu/gbm_buffer_base.cc
@@ -40,13 +40,9 @@ GbmBufferBase::GbmBufferBase(const scoped_refptr<GbmDevice>& drm,
// DRM_MODE_FB_MODIFIERS set. We only set that when we've created
// a bo with modifiers, otherwise, we rely on the "no modifiers"
// behavior doing the right thing.
- if (!drm_->AddFramebuffer2(gbm_bo_get_width(bo), gbm_bo_get_height(bo),
- framebuffer_pixel_format_, handles, strides,
- offsets, modifiers, &framebuffer_,
- addfb_flags)) {
- PLOG(ERROR) << "Failed to register buffer";
- return;
- }
+ drm_->AddFramebuffer2(gbm_bo_get_width(bo), gbm_bo_get_height(bo),
+ framebuffer_pixel_format_, handles, strides, offsets,
+ modifiers, &framebuffer_, addfb_flags);
}
}
« no previous file with comments | « ui/ozone/platform/drm/gpu/gbm_buffer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698