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

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

Issue 1084173004: Adding status to swap complete (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase, separate out use of the result, add enum Created 5 years, 8 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/platform/drm/gpu/drm_window.cc
diff --git a/ui/ozone/platform/drm/gpu/drm_window.cc b/ui/ozone/platform/drm/gpu/drm_window.cc
index ec6a0eafdcd85c3a93b65673193c1b9ca695d669..843a3e1ee126621f0b43ef4297d9c1ebeb6eb68a 100644
--- a/ui/ozone/platform/drm/gpu/drm_window.cc
+++ b/ui/ozone/platform/drm/gpu/drm_window.cc
@@ -121,7 +121,9 @@ void DrmWindow::QueueOverlayPlane(const OverlayPlane& plane) {
pending_planes_.push_back(plane);
}
-bool DrmWindow::SchedulePageFlip(bool is_sync, const base::Closure& callback) {
+bool DrmWindow::SchedulePageFlip(
+ bool is_sync,
+ const base::Callback<void(gfx::SwapResult)>& callback) {
alexst (slow to review) 2015/04/29 17:53:37 We should typdef this as well somewhere in the pla
achaulk 2015/05/06 18:29:48 Done.
last_submitted_planes_.clear();
last_submitted_planes_.swap(pending_planes_);
last_swap_sync_ = is_sync;
@@ -131,7 +133,7 @@ bool DrmWindow::SchedulePageFlip(bool is_sync, const base::Closure& callback) {
callback);
}
- callback.Run();
+ callback.Run(gfx::SwapAck);
return true;
}

Powered by Google App Engine
This is Rietveld 408576698