Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(43)

Unified Diff: components/viz/hittest/hittest.h

Issue 2908783002: WIP Hittest Component.
Patch Set: format Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/viz/hittest/hittest.h
diff --git a/components/viz/hittest/hittest.h b/components/viz/hittest/hittest.h
new file mode 100644
index 0000000000000000000000000000000000000000..86b927e8ade49b9522b10020322e2a0985d19958
--- /dev/null
+++ b/components/viz/hittest/hittest.h
@@ -0,0 +1,74 @@
+// Copyright 2015 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 COMPONENTS_VIZ_HITTEST_HITTEST_H_
+#define COMPONENTS_VIZ_HITTEST_HITTEST_H_
+
+#include <stdint.h>
+
+#include <map>
+#include <memory>
+
+#include "cc/output/compositor_frame.h"
rjkroege 2017/05/29 15:58:02 not a correct dependency in my opinion.
+#include "cc/surfaces/surface_id.h"
+#include "components/viz/hittest/hittest_export.h"
+#include "components/viz/hittest/public/interfaces/hittest_data.mojom.h"
+#include "ui/gfx/geometry/quad_f.h"
+
+namespace viz {
+
+// Hittest
rjkroege 2017/05/29 15:58:03 you could merge this line with the one below
+// Maintains maping between display regions and associated surfaces
+// in shared memory to enable efficient hit testing across processes.
+//
+// This is intended to be created in the viz or GPU process. For mus+ash this
+// will be true after the mus process split.
+
rjkroege 2017/05/29 15:58:03 no space? git cl format?
+class HITTEST_EXPORT Hittest {
+ public:
+ static cc::SurfaceId SurfaceIdAtPoint(cc::SurfaceId root_surface_id,
rjkroege 2017/05/29 15:58:03 i'm not sure that this is right.
gklassen 2017/06/05 21:32:13 what would you change?
+ const gfx::Point& point,
+ gfx::Point* transformed_point) {
+ return GetInstance()->InternalGetSurfaceIdAtPoint(root_surface_id, point,
+ transformed_point);
+ }
+
+ static void DisplayWillDrawAndSwap(bool will_draw_and_swap,
rjkroege 2017/05/29 15:58:03 seems not right. hittest component should provide
+ const cc::RenderPassList& render_passes) {
+ GetInstance()->Aggregate();
+ }
+
+ private:
+ static Hittest* instance_;
+
+ static Hittest* GetInstance() {
rjkroege 2017/05/29 15:58:03 are you trying to create a singleton? I'd argue th
+ if (instance_ == NULL) {
+ instance_ = new Hittest();
+ }
+ return instance_;
+ }
+
+ Hittest();
+ ~Hittest();
+
+ std::map<cc::SurfaceId, viz::hittest::mojom::HittestData> pending_;
+
+ struct Region;
+ std::vector<Region> regions_[2];
rjkroege 2017/05/29 15:58:03 this is the shared memory? AFAIK: vector is not a
+ std::vector<Region>* current_regions_;
+
+ cc::SurfaceId InternalGetSurfaceIdAtPoint(cc::SurfaceId root_surface_id,
rjkroege 2017/05/29 15:58:02 these all need method comments?
+ const gfx::Point& point,
+ gfx::Point* transformed_point);
+
+ void SubmitHittestData(viz::hittest::mojom::HittestData& hittest_data);
+
+ void Aggregate();
+
+ void swap();
+};
+
+} // namespace viz
+
+#endif // COMPONENTS_VIZ_HITTEST_HITTEST_H_

Powered by Google App Engine
This is Rietveld 408576698