Surface synchronization: Add a diagnostic CHECK in ClientLayerTreeFrameSink

There's a race in cc somewhere from commit to draw. I don't know exactly
how to repro the race yet even though I have a high probably fix. This
CL is intended to catch the race in the wild and produce a stack trace
in a crash log. Hopefully from there we can acertain repro steps.

Once repo steps are produced locally (either via user interaction or
unit test) then this CL will be reverted.

Bug: 672962
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: I3bce066591e15d5c3aac7d4c8685dce411e284b7
Reviewed-on: https://chromium-review.googlesource.com/973987Reviewed-by: Saman Sami <samans@chromium.org>
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544834}

components/viz/client/client_layer_tree_frame_sink.cc

@@ -126,6 +126,10 @@ void ClientLayerTreeFrameSink::SubmitCompositorFrame(CompositorFrame frame) {
   if (!enable_surface_synchronization_) {
     local_surface_id_ =
         local_surface_id_provider_->GetLocalSurfaceIdForFrame(frame);
+  } else {
+    CHECK(local_surface_id_ != last_submitted_local_surface_id_ ||
+          (last_submitted_device_scale_factor_ == frame.device_scale_factor() &&
+           last_submitted_size_in_pixels_ == frame.size_in_pixels()));
   }
 
   TRACE_EVENT_FLOW_BEGIN0(TRACE_DISABLED_BY_DEFAULT("cc.debug.ipc"),
@@ -138,6 +142,10 @@ void ClientLayerTreeFrameSink::SubmitCompositorFrame(CompositorFrame frame) {
   if (hit_test_data_provider_)
     hit_test_region_list = hit_test_data_provider_->GetHitTestData(frame);
 
+  last_submitted_local_surface_id_ = local_surface_id_;
+  last_submitted_device_scale_factor_ = frame.device_scale_factor();
+  last_submitted_size_in_pixels_ = frame.size_in_pixels();
+
   compositor_frame_sink_ptr_->SubmitCompositorFrame(
       local_surface_id_, std::move(frame), std::move(hit_test_region_list),
       tracing_enabled ? base::TimeTicks::Now().since_origin().InMicroseconds()

components/viz/client/client_layer_tree_frame_sink.h

@@ -127,6 +127,10 @@ class VIZ_CLIENT_EXPORT ClientLayerTreeFrameSink
   const bool enable_surface_synchronization_;
   const bool wants_animate_only_begin_frames_;
 
+  LocalSurfaceId last_submitted_local_surface_id_;
+  float last_submitted_device_scale_factor_ = 1.f;
+  gfx::Size last_submitted_size_in_pixels_;
+
   base::WeakPtrFactory<ClientLayerTreeFrameSink> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientLayerTreeFrameSink);