Commit 2bb02293 authored by Fady Samuel's avatar Fady Samuel Committed by Commit Bot

Surface synchronization: Re-enable DCHECK in RenderWidget

Now that OOPIFs work in Mus+Ash, we can re-enable a surface synchronization
DCHECK in RenderWidget.

Bug: 672962
Change-Id: Ie9918085a103bf31c98f292667e312b78feedf96
Reviewed-on: https://chromium-review.googlesource.com/721631
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Reviewed-by: default avatarAntoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509609}
parent 453656aa
...@@ -1279,10 +1279,8 @@ void RenderWidget::Resize(const ResizeParams& params) { ...@@ -1279,10 +1279,8 @@ void RenderWidget::Resize(const ResizeParams& params) {
// If the ID is not valid, then the compositor will defer commits until // If the ID is not valid, then the compositor will defer commits until
// it receives a valid surface ID. This is a no-op if surface // it receives a valid surface ID. This is a no-op if surface
// synchronization is disabled. // synchronization is disabled.
// TODO(crbug.com/758387): Re-enable this DCHECK once the mash login screen DCHECK(!compositor_->IsSurfaceSynchronizationEnabled() ||
// is fixed. !params.needs_resize_ack || local_surface_id_.is_valid());
// DCHECK(!compositor_->IsSurfaceSynchronizationEnabled() ||
// local_surface_id_.is_valid());
compositor_->SetViewportSize(params.physical_backing_size, compositor_->SetViewportSize(params.physical_backing_size,
local_surface_id_); local_surface_id_);
compositor_->SetBrowserControlsHeight( compositor_->SetBrowserControlsHeight(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment