Plumbing input event latency reporting through Mus GPU.

services/ui/surfaces/display_output_surface.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 "services/ui/surfaces/display_output_surface.h"
 
 #include <stdint.h>
 
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "cc/output/context_provider.h"
 #include "cc/output/output_surface_client.h"
 #include "cc/output/output_surface_frame.h"
 #include "cc/scheduler/begin_frame_source.h"
 #include "gpu/command_buffer/client/context_support.h"
 #include "gpu/command_buffer/client/gles2_interface.h"
+#include "ui/latency/latency_tracker.h"
 
 namespace ui {
 
 DisplayOutputSurface::DisplayOutputSurface(
     scoped_refptr<cc::InProcessContextProvider> context_provider,
     cc::SyntheticBeginFrameSource* synthetic_begin_frame_source)
     : cc::OutputSurface(context_provider),
       synthetic_begin_frame_source_(synthetic_begin_frame_source),
+      latency_tracker_(new LatencyTracker),

[Fady Samuel, 2017/04/13 03:32:17]

nit: base::MakeUnique<LatencyTracker>()  or better yet, why is this allocated on
the heap? Why not keep it inline?

[mfomitchev, 2017/04/13 17:15:29]

Done.

       weak_ptr_factory_(this) {
   capabilities_.flipped_output_surface =
       context_provider->ContextCapabilities().flips_vertically;
   capabilities_.supports_stencil =
       context_provider->ContextCapabilities().num_stencil_bits > 0;
   context_provider->SetSwapBuffersCompletionCallback(
       base::Bind(&DisplayOutputSurface::OnGpuSwapBuffersCompleted,
                  weak_ptr_factory_.GetWeakPtr()));
   context_provider->SetUpdateVSyncParametersCallback(
       base::Bind(&DisplayOutputSurface::OnVSyncParametersUpdated,
@@ skipping 36 matching lines @@
                                    bool has_alpha,
                                    bool use_stencil) {
   size_ = size;
   has_set_draw_rectangle_since_last_resize_ = false;
   context_provider()->ContextGL()->ResizeCHROMIUM(
       size.width(), size.height(), device_scale_factor, has_alpha);
 }
 
 void DisplayOutputSurface::SwapBuffers(cc::OutputSurfaceFrame frame) {
   DCHECK(context_provider_);
+
+  if (frame.latency_info.size() > 0)
+    context_provider_->ContextSupport()->AddLatencyInfo(frame.latency_info);
+
   set_draw_rectangle_for_frame_ = false;
   if (frame.sub_buffer_rect) {
     context_provider_->ContextSupport()->PartialSwapBuffers(
         *frame.sub_buffer_rect);
   } else {
     context_provider_->ContextSupport()->Swap();
   }
 }
 
 uint32_t DisplayOutputSurface::GetFramebufferCopyTextureFormat() {
@@ skipping 24 matching lines @@
   return false;
 }
 
 void DisplayOutputSurface::ApplyExternalStencil() {}
 
 void DisplayOutputSurface::DidReceiveSwapBuffersAck(gfx::SwapResult result) {
   client_->DidReceiveSwapBuffersAck();
 }
 
 void DisplayOutputSurface::OnGpuSwapBuffersCompleted(
-    const std::vector<ui::LatencyInfo>& latency_info,
+    const std::vector<LatencyInfo>& latency_info,
     gfx::SwapResult result,
     const gpu::GpuProcessHostedCALayerTreeParamsMac* params_mac) {
+  for (const auto& latency : latency_info) {
+    if (latency.latency_components().size() > 0)
+      latency_tracker_->OnGpuSwapBuffersCompleted(latency);
+  }
   DidReceiveSwapBuffersAck(result);
 }
 
 void DisplayOutputSurface::OnVSyncParametersUpdated(base::TimeTicks timebase,
                                                     base::TimeDelta interval) {
   // TODO(brianderson): We should not be receiving 0 intervals.
   synthetic_begin_frame_source_->OnUpdateVSyncParameters(
       timebase,
       interval.is_zero() ? cc::BeginFrameArgs::DefaultInterval() : interval);
 }
 
 }  // namespace ui