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

ui/latency/latency_info.h

 // Copyright 2013 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.
 
 #ifndef UI_LATENCY_LATENCY_INFO_H_
 #define UI_LATENCY_LATENCY_INFO_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 99 matching lines @@
   WHEEL,
   TOUCH,
   KEY_PRESS,
   OTHER,
   SOURCE_EVENT_TYPE_LAST = OTHER,
 };
 
 class LatencyInfo {
  public:
   struct LatencyComponent {
-    // Nondecreasing number that can be used to determine what events happened
-    // in the component at the time this struct was sent on to the next
-    // component.
-    int64_t sequence_number;
     // Average time of events that happened in this component.
     base::TimeTicks event_time;
     // Count of events that happened in this component
     uint32_t event_count;
     // Time of the oldest event that happened in this component.
     base::TimeTicks first_event_time;
     // Time of the most recent event that happened in this component.
     base::TimeTicks last_event_time;
   };
 
-  enum : size_t { kMaxInputCoordinates = 2 };
-
   // Map a Latency Component (with a component-specific int64_t id) to a
   // component info.
   using LatencyMap = base::flat_map<std::pair<LatencyComponentType, int64_t>,
                                     LatencyComponent>;
 
   LatencyInfo();
   LatencyInfo(const LatencyInfo& other);
   LatencyInfo(SourceEventType type);
   ~LatencyInfo();
 
@@ skipping 11 matching lines @@
                      const char* referring_msg);
 
   // Copy LatencyComponents with type |type| from |other| into |this|.
   void CopyLatencyFrom(const LatencyInfo& other, LatencyComponentType type);
 
   // Add LatencyComponents that are in |other| but not in |this|.
   void AddNewLatencyFrom(const LatencyInfo& other);
 
   // Modifies the current sequence number for a component, and adds a new
   // sequence number with the current timestamp.
-  void AddLatencyNumber(LatencyComponentType component,
-                        int64_t id,
-                        int64_t component_sequence_number);
+  void AddLatencyNumber(LatencyComponentType component, int64_t id);
 
   // Similar to |AddLatencyNumber|, and also appends |trace_name_str| to
   // the trace event's name.
   // This function should only be called when adding a BEGIN component.
   void AddLatencyNumberWithTraceName(LatencyComponentType component,
                                      int64_t id,
-                                     int64_t component_sequence_number,
                                      const char* trace_name_str);
 
   // Modifies the current sequence number and adds a certain number of events
   // for a specific component.
   void AddLatencyNumberWithTimestamp(LatencyComponentType component,
                                      int64_t id,
-                                     int64_t component_sequence_number,
                                      base::TimeTicks time,
                                      uint32_t event_count);
 
   // Returns true if the a component with |type| and |id| is found in
   // the latency_components and the component is stored to |output| if
   // |output| is not NULL. Returns false if no such component is found.
   bool FindLatency(LatencyComponentType type,
                    int64_t id,
                    LatencyComponent* output) const;
 
@@ skipping 26 matching lines @@
   bool began() const { return began_; }
   bool terminated() const { return terminated_; }
   void set_coalesced() { coalesced_ = true; }
   bool coalesced() const { return coalesced_; }
   int64_t trace_id() const { return trace_id_; }
   void set_trace_id(int64_t trace_id) { trace_id_ = trace_id; }
 
  private:
   void AddLatencyNumberWithTimestampImpl(LatencyComponentType component,
                                          int64_t id,
-                                         int64_t component_sequence_number,
                                          base::TimeTicks time,
                                          uint32_t event_count,
                                          const char* trace_name_str);
 
   // Converts latencyinfo into format that can be dumped into trace buffer.
   std::unique_ptr<base::trace_event::ConvertableToTraceFormat>
   AsTraceableData();
 
   // Shown as part of the name of the trace event for this LatencyInfo.
   // String is empty if no tracing is enabled.
   std::string trace_name_;
 
   LatencyMap latency_components_;
 
-  // The unique id for matching the ASYNC_BEGIN/END trace event.
+  // The unique id for matching the ASYNC_BEGIN/END trace event. -1 until a
+  // begin component has been encountered.
   int64_t trace_id_;
   // Whether this event has been coalesced into another event.
   bool coalesced_;
   // Whether a begin component has been added.
   bool began_;
   // Whether a terminal component has been added.
   bool terminated_;
   // Stores the type of the first source event.
   SourceEventType source_event_type_;
   // The expected queueing time on the main thread when this event was
   // dispatched.
   base::TimeDelta expected_queueing_time_on_dispatch_;
 
 #if !defined(OS_IOS)
   friend struct IPC::ParamTraits<ui::LatencyInfo>;
   friend struct mojo::StructTraits<ui::mojom::LatencyInfoDataView,
                                    ui::LatencyInfo>;
 #endif
 };
 
 // This is declared here for use in gtest-based unit tests, but is defined in
 // //ui/latency:test_support target.
 // Without this the default PrintTo template in gtest tries to pass LatencyInfo
 // by value, which leads to an alignment compile error on Windows.
 void PrintTo(const LatencyInfo& latency, ::std::ostream* os);
 
 }  // namespace ui
 
 #endif  // UI_LATENCY_LATENCY_INFO_H_