Only terminate latency components when no rendering is scheduled

content/browser/renderer_host/render_widget_host_latency_tracker.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 "content/browser/renderer_host/render_widget_host_latency_tracker.h"
 
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 
@@ skipping 202 matching lines @@
 void RenderWidgetHostLatencyTracker::OnInputEventAck(
     const blink::WebInputEvent& event,
     LatencyInfo* latency) {
   DCHECK(latency);
 
   // Latency ends when it is acked but does not cause render scheduling.
   bool rendering_scheduled = latency->FindLatency(
       ui::INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_COMPONENT, 0, NULL);
 
   if (WebInputEvent::isGestureEventType(event.type)) {
-    if (rendering_scheduled) {
+    if (!rendering_scheduled) {
       latency->AddLatencyNumber(
           ui::INPUT_EVENT_LATENCY_TERMINATED_GESTURE_COMPONENT, 0, 0);
       // TODO(jdduke): Consider exposing histograms for gesture event types.
     }
     return;
   }
 
   if (WebInputEvent::isTouchEventType(event.type)) {
     latency->AddLatencyNumber(ui::INPUT_EVENT_LATENCY_ACK_RWH_COMPONENT, 0, 0);
-    if (rendering_scheduled) {
+    if (!rendering_scheduled) {
       latency->AddLatencyNumber(
           ui::INPUT_EVENT_LATENCY_TERMINATED_TOUCH_COMPONENT, 0, 0);
-      ComputeInputLatencyHistograms(WebInputEvent::TouchTypeFirst,
-                                    latency_component_id_, *latency);
     }
+    ComputeInputLatencyHistograms(WebInputEvent::TouchTypeFirst,
+                                  latency_component_id_, *latency);
     return;
   }
 
   if (event.type == WebInputEvent::MouseWheel) {
     latency->AddLatencyNumber(ui::INPUT_EVENT_LATENCY_ACK_RWH_COMPONENT, 0, 0);
-    if (rendering_scheduled) {
+    if (!rendering_scheduled) {
       latency->AddLatencyNumber(
           ui::INPUT_EVENT_LATENCY_TERMINATED_MOUSE_COMPONENT, 0, 0);
-      ComputeInputLatencyHistograms(WebInputEvent::MouseWheel,
-                                    latency_component_id_, *latency);
     }
+    ComputeInputLatencyHistograms(WebInputEvent::MouseWheel,
+                                  latency_component_id_, *latency);
     return;
   }
 
   // TODO(jdduke): Determine if mouse and keyboard events are worth hooking
   // into LatencyInfo.
 }
 
 void RenderWidgetHostLatencyTracker::OnSwapCompositorFrame(
     std::vector<LatencyInfo>* latencies) {
   DCHECK(latencies);
@@ skipping 39 matching lines @@
           (swap_component.event_time - original_component.event_time)
               .InMicroseconds(),
           1,
           1000000,
           100);
     }
   }
 }
 
 }  // namespace content