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

ui/events/latency_info.h

Index: ui/events/latency_info.h
diff --git a/ui/events/latency_info.h b/ui/events/latency_info.h
index c69d880193d60b77b6f6786ab8431cb38959a53a..fb98069e9670bca8f94cd640f895516fb9566673 100644
--- a/ui/events/latency_info.h
+++ b/ui/events/latency_info.h
@@ -8,13 +8,16 @@
 #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"
 
@@ -23,6 +26,12 @@
 #include "mojo/public/cpp/bindings/struct_traits.h"  // nogncheck
 #endif
 
+namespace base {
+namespace trace_event {
+class ConvertableToTraceFormat;
+}
+}
+
 namespace ui {
 
 #if !defined(OS_IOS)