WIP Hittest Component.

components/viz/frame_sinks/gpu_root_compositor_frame_sink.h

Index: components/viz/frame_sinks/gpu_root_compositor_frame_sink.h
diff --git a/components/viz/frame_sinks/gpu_root_compositor_frame_sink.h b/components/viz/frame_sinks/gpu_root_compositor_frame_sink.h
index 18df306f331cde0a18d9b0da4444d9d464d1aa3f..38e15fa52f77d972ddd88e02e6fab4361aacc2d4 100644
--- a/components/viz/frame_sinks/gpu_root_compositor_frame_sink.h
+++ b/components/viz/frame_sinks/gpu_root_compositor_frame_sink.h
@@ -12,6 +12,7 @@
 #include "cc/surfaces/local_surface_id.h"
 #include "cc/surfaces/surface_id.h"
 #include "components/viz/frame_sinks/gpu_compositor_frame_sink_delegate.h"
+#include "components/viz/hittest/hittest_aggregator.h"
 #include "mojo/public/cpp/bindings/associated_binding.h"
 #include "mojo/public/cpp/bindings/binding.h"
 
@@ -102,6 +103,8 @@ class GpuRootCompositorFrameSink
       compositor_frame_sink_private_binding_;
   mojo::AssociatedBinding<cc::mojom::DisplayPrivate> display_private_binding_;
 
+  HittestAggregator hittest_aggregator_;

[rjkroege, 2017/06/07 20:22:56]

If you write it this way, you are using space even before the code is finished.
Consider unique_ptr<> and only creating if the feature is enabled. (Feature
needs to behind a flag before landing)

[gklassen, 2017/06/08 17:09:22]

Good call. Done.

+
   DISALLOW_COPY_AND_ASSIGN(GpuRootCompositorFrameSink);
 };