Make compositor_frame.h NOT include base/trace_event/trace_event.h

ui/events/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_EVENTS_LATENCY_INFO_H_
 #define UI_EVENTS_LATENCY_INFO_H_
 
 #include <stdint.h>
 
 #include <memory>
+#include <queue>

[sadrul, 2016/08/25 19:45:14]

The code in here doesn't seem to actually use queue? Do we need this here?

 #include <string>
 #include <utility>
 #include <vector>
 
+#include "base/bind.h"
 #include "base/containers/small_map.h"
+#include "base/pickle.h"
+#include "base/strings/string16.h"

[sadrul, 2016/08/25 19:45:14]

It doesn't look like any of these headers need to be in this file?

[xlai (Olivia), 2016/08/25 20:11:16]

I've deleted all these #include files. As a result, these files are added to the
included list in some other files that include latency_info.h and implicitly use
these files.

 #include "base/time/time.h"
-#include "base/trace_event/trace_event.h"
 #include "ui/events/events_base_export.h"
 #include "ui/gfx/geometry/point_f.h"
 
 #if !defined(OS_IOS)
 #include "ipc/ipc_param_traits.h"  // nogncheck
 #include "mojo/public/cpp/bindings/struct_traits.h"  // nogncheck
 #endif
 
+namespace base {
+namespace trace_event {
+class ConvertableToTraceFormat;
+}
+}
+
 namespace ui {
 
 #if !defined(OS_IOS)
 namespace mojom {
 class LatencyInfoDataView;
 }
 #endif
 
 // When adding new components, or new metrics based on LatencyInfo,
 // please update latency_info.dot.
@@ skipping 203 matching lines @@
 #if !defined(OS_IOS)
   friend struct IPC::ParamTraits<ui::LatencyInfo>;
   friend struct mojo::StructTraits<ui::mojom::LatencyInfoDataView,
                                    ui::LatencyInfo>;
 #endif
 };
 
 }  // namespace ui
 
 #endif  // UI_EVENTS_LATENCY_INFO_H_