ozone: Add sync flag for atomic commit

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

 // 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/drm/gpu/hardware_display_plane_manager_atomic.h"
 
 #include "base/bind.h"
 #include "ui/ozone/platform/drm/gpu/drm_device.h"
 #include "ui/ozone/platform/drm/gpu/hardware_display_plane_atomic.h"
 #include "ui/ozone/platform/drm/gpu/scanout_buffer.h"
@@ skipping 23 matching lines @@
       plane->set_in_use(false);
       HardwareDisplayPlaneAtomic* atomic_plane =
           static_cast<HardwareDisplayPlaneAtomic*>(plane);
       atomic_plane->SetPlaneData(plane_list->atomic_property_set.get(), 0, 0,
                                  gfx::Rect(), gfx::Rect());
     }
   }
 
   plane_list->plane_list.swap(plane_list->old_plane_list);
   plane_list->plane_list.clear();
-  if (!drm_->CommitProperties(plane_list->atomic_property_set.get(), 0,
+  if (!drm_->CommitProperties(plane_list->atomic_property_set.get(), 0, true,

[dnicoara, 2015/04/24 19:47:49]

Shouldn't we set this to 'false'? We'd rather not block the main thread.

[achaulk, 2015/04/24 19:50:41]

Actually, it should be is_sync which has helpfully already been propagated and I
missed it

[dnicoara, 2015/04/24 19:51:47]

Thanks!

                               base::Bind(&AtomicPageFlipCallback))) {
     PLOG(ERROR) << "Failed to commit properties";
     return false;
   }
   return true;
 }
 
 bool HardwareDisplayPlaneManagerAtomic::AssignOverlayPlanes(
     HardwareDisplayPlaneList* plane_list,
     const OverlayPlaneList& overlay_list,
@@ skipping 26 matching lines @@
 }
 
 scoped_ptr<HardwareDisplayPlane> HardwareDisplayPlaneManagerAtomic::CreatePlane(
     uint32_t plane_id,
     uint32_t possible_crtcs) {
   return scoped_ptr<HardwareDisplayPlane>(
       new HardwareDisplayPlaneAtomic(plane_id, possible_crtcs));
 }
 
 }  // namespace ui