Remove LatencyInfo::sequence_number. (May break metrics).

cc/trees/latency_info_swap_promise_monitor.cc

Index: cc/trees/latency_info_swap_promise_monitor.cc
diff --git a/cc/trees/latency_info_swap_promise_monitor.cc b/cc/trees/latency_info_swap_promise_monitor.cc
index cb0af9ee9a286440c58d85b62332d96ffae723e2..bcbe669e73958f730ee3d88d1bda578877d30cd9 100644
--- a/cc/trees/latency_info_swap_promise_monitor.cc
+++ b/cc/trees/latency_info_swap_promise_monitor.cc
@@ -21,7 +21,7 @@ bool AddRenderingScheduledComponent(ui::LatencyInfo* latency_info,
               : ui::INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_IMPL_COMPONENT;
   if (latency_info->FindLatency(type, 0, nullptr))
     return false;
-  latency_info->AddLatencyNumber(type, 0, 0);
+  latency_info->AddLatencyNumber(type, 0);
   return true;
 }
 
@@ -31,7 +31,7 @@ bool AddForwardingScrollUpdateToMainComponent(ui::LatencyInfo* latency_info) {
           nullptr))
     return false;
   latency_info->AddLatencyNumber(
-      ui::INPUT_EVENT_LATENCY_FORWARD_SCROLL_UPDATE_TO_MAIN_COMPONENT, 0,
+      ui::INPUT_EVENT_LATENCY_FORWARD_SCROLL_UPDATE_TO_MAIN_COMPONENT,
       latency_info->trace_id());
   return true;
 }
@@ -73,26 +73,10 @@ void LatencyInfoSwapPromiseMonitor::OnSetNeedsRedrawOnImpl() {
 
 void LatencyInfoSwapPromiseMonitor::OnForwardScrollUpdateToMainThreadOnImpl() {
   if (AddForwardingScrollUpdateToMainComponent(latency_)) {
-    int64_t new_sequence_number = 0;
-    for (ui::LatencyInfo::LatencyMap::const_iterator it =
-             latency_->latency_components().begin();
-         it != latency_->latency_components().end(); ++it) {
-      if (it->first.first == ui::INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT) {
-        new_sequence_number =
-            ((static_cast<int64_t>(base::PlatformThread::CurrentId()) << 32) ^
-             (reinterpret_cast<uint64_t>(this) << 32)) |
-            (it->second.sequence_number & 0xffffffff);
-        if (new_sequence_number == it->second.sequence_number)
-          return;
-        break;
-      }
-    }
-    if (!new_sequence_number)
-      return;

[tdresser, 2017/06/08 18:36:44]

This is a material change. 

I think what's happening here is that we're returning early if we've already
sent the event to this main thread.

I'm not sure when this could happen though...

     ui::LatencyInfo new_latency;
     new_latency.AddLatencyNumberWithTraceName(
         ui::LATENCY_BEGIN_SCROLL_LISTENER_UPDATE_MAIN_COMPONENT, 0,
-        new_sequence_number, "ScrollUpdate");
+        "ScrollUpdate");
     new_latency.CopyLatencyFrom(
         *latency_,
         ui::INPUT_EVENT_LATENCY_FORWARD_SCROLL_UPDATE_TO_MAIN_COMPONENT);