Surface synchronization: Remove check in LayerTreeImpl

We still get the occasional surface invariants violations in the wild and
we don't necessarily want to crash the renderer in this case. We will already
log things appropriately.

Bug: 672962, 828741
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I262850f2cf704cd59366da6dd4d154227aab5220
Reviewed-on: https://chromium-review.googlesource.com/1112491Reviewed-by: Saman Sami <samans@chromium.org>
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569781}

cc/trees/layer_tree_impl.cc

@@ -1003,12 +1003,6 @@ void LayerTreeImpl::SetDeviceScaleFactor(float device_scale_factor) {
 
 void LayerTreeImpl::SetLocalSurfaceIdFromParent(
     const viz::LocalSurfaceId& local_surface_id_from_parent) {
-  // If surface synchronization is on then we expect that if the viewport size
-  // has changed then either the |local_surface_id_from_parent| or there is a
-  // pending impl-side request for new viz::LocalSurfaceId allocation.
-  CHECK(!settings().enable_surface_synchronization || !viewport_size_invalid_ ||
-        local_surface_id_from_parent_ != local_surface_id_from_parent ||
-        new_local_surface_id_request_);
   local_surface_id_from_parent_ = local_surface_id_from_parent;
 }