[Ozone-DRM] Remove bool return value for SchedulePageFlip call

ui/ozone/platform/drm/gpu/hardware_display_controller.cc

Index: ui/ozone/platform/drm/gpu/hardware_display_controller.cc
diff --git a/ui/ozone/platform/drm/gpu/hardware_display_controller.cc b/ui/ozone/platform/drm/gpu/hardware_display_controller.cc
index a377e8bafa46547689d804528413d9db7ff6a9a2..1b503c558f8bb440ca8b894bc44d00b643f778c0 100644
--- a/ui/ozone/platform/drm/gpu/hardware_display_controller.cc
+++ b/ui/ozone/platform/drm/gpu/hardware_display_controller.cc
@@ -23,6 +23,12 @@
 
 namespace ui {
 
+namespace {
+
+void EmptyFlipCallback(gfx::SwapResult) {}
+
+}  // namespace
+
 HardwareDisplayController::HardwareDisplayController(
     scoped_ptr<CrtcController> controller,
     const gfx::Point& origin)
@@ -72,7 +78,19 @@ void HardwareDisplayController::Disable() {
   is_disabled_ = true;
 }
 
-bool HardwareDisplayController::SchedulePageFlip(
+void HardwareDisplayController::SchedulePageFlip(
+    const OverlayPlaneList& plane_list,
+    const PageFlipCallback& callback) {
+  ActualSchedulePageFlip(plane_list, false /* test_only */, callback);
+}
+
+bool HardwareDisplayController::TestPageFlip(
+    const OverlayPlaneList& plane_list) {
+  return ActualSchedulePageFlip(plane_list, true /* test_only */,
+                                base::Bind(&EmptyFlipCallback));
+}
+
+bool HardwareDisplayController::ActualSchedulePageFlip(

[spang, 2015/10/29 20:59:20]

This only moves the problem to ActualSchedulePageFlip.

It should be solvable, CheckOverlayCapabilities at least takes a callback.

[dnicoara, 2015/10/30 14:59:31]

Yes, though DrmWindow::TestPageFlip() has some allocation optimizations that I
wanted to take another look at before committing to a solution. My plan is to
have a followup that refactors TestPageFlip().

     const OverlayPlaneList& plane_list,
     bool test_only,
     const PageFlipCallback& callback) {