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

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

 // 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.
 
 #ifndef UI_OZONE_PLATFORM_DRM_GPU_HARDWARE_DISPLAY_CONTROLLER_H_
 #define UI_OZONE_PLATFORM_DRM_GPU_HARDWARE_DISPLAY_CONTROLLER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 #include <xf86drmMode.h>
@@ skipping 100 matching lines @@
   // be presented to the callback. We use that argument to pass in the HDCO
   // object the event belongs to.
   //
   // Between this call and the callback, the framebuffers used in this call
   // should not be modified in any way as it would cause screen tearing if the
   // hardware performed the flip. Note that the frontbuffer should also not
   // be modified as it could still be displayed.
   //
   // Note that this function does not block. Also, this function should not be
   // called again before the page flip occurrs.
-  //
-  // Returns true if the page flip was successfully registered, false otherwise.
-  //
-  // When called with |test_only| true, this performs the page flip without
-  // changing any state, reporting if this page flip would be allowed to occur.
-  bool SchedulePageFlip(const OverlayPlaneList& plane_list,
-                        bool test_only,
+  void SchedulePageFlip(const OverlayPlaneList& plane_list,
                         const PageFlipCallback& callback);
 
+  // Returns true if the page flip with the |plane_list| would succeed. This
+  // doesn't change any state.
+  bool TestPageFlip(const OverlayPlaneList& plane_list);
+
   std::vector<uint32_t> GetCompatibleHardwarePlaneIds(
       const OverlayPlane& plane) const;
 
   // Set the hardware cursor to show the contents of |surface|.
   bool SetCursor(const scoped_refptr<ScanoutBuffer>& buffer);
 
   bool UnsetCursor();
 
   // Moves the hardware cursor to |location|.
   bool MoveCursor(const gfx::Point& location);
@@ skipping 11 matching lines @@
 
   uint64_t GetTimeOfLastFlip() const;
 
   const std::vector<CrtcController*>& crtc_controllers() const {
     return crtc_controllers_.get();
   }
 
   scoped_refptr<DrmDevice> GetAllocationDrmDevice() const;
 
  private:
+  bool ActualSchedulePageFlip(const OverlayPlaneList& plane_list,
+                              bool test_only,
+                              const PageFlipCallback& callback);
+
   base::ScopedPtrHashMap<DrmDevice*, scoped_ptr<HardwareDisplayPlaneList>>
       owned_hardware_planes_;
 
   // Stores the CRTC configuration. This is used to identify monitors and
   // configure them.
   ScopedVector<CrtcController> crtc_controllers_;
 
   // Location of the controller on the screen.
   gfx::Point origin_;
 
   bool is_disabled_;
 
   DISALLOW_COPY_AND_ASSIGN(HardwareDisplayController);
 };
 
 }  // namespace ui
 
 #endif  // UI_OZONE_PLATFORM_DRM_GPU_HARDWARE_DISPLAY_CONTROLLER_H_