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

Unified Diff: sky/sdk/lib/framework/app.dart

Issue 1158933004: In Sky framework AppView, cache the hit test result for pointer events and continue using that resu… (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: fix merge conflict Created 5 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/sdk/lib/framework/app.dart
diff --git a/sky/sdk/lib/framework/app.dart b/sky/sdk/lib/framework/app.dart
index 4b245089421f632b82cdad0504aae9dd94ad50ec..6250fe40d2b9a65b866ae9f2238783a552da1331 100644
--- a/sky/sdk/lib/framework/app.dart
+++ b/sky/sdk/lib/framework/app.dart
@@ -20,21 +20,40 @@ class AppView {
RenderView _renderView;
+ Map<int, HitTestResult> _hitTestResultForPointer = new Map<int, HitTestResult>();
+
RenderBox get root => _renderView.child;
void set root(RenderBox value) {
_renderView.child = value;
}
-
void _beginFrame(double timeStamp) {
RenderNode.flushLayout();
_renderView.paintFrame();
}
void _handleEvent(sky.Event event) {
- if (event is! sky.PointerEvent)
- return;
- HitTestResult result = new HitTestResult();
- _renderView.hitTest(result, x: event.x, y: event.y);
+ if (event is sky.PointerEvent)
+ _handlePointerEvent(event);
+ }
+
+ void _handlePointerEvent(sky.PointerEvent event) {
+ HitTestResult result;
+ switch(event.type) {
+ case 'pointerdown':
+ result = new HitTestResult();
+ _renderView.hitTest(result, x: event.x, y: event.y);
+ _hitTestResultForPointer[event.pointer] = result;
+ break;
+ case 'pointerup':
+ case 'pointercancel':
+ result = _hitTestResultForPointer[event.pointer];
+ _hitTestResultForPointer.remove(event.pointer);
+ break;
+ case 'pointermove':
+ result = _hitTestResultForPointer[event.pointer];
+ break;
+ }
+ assert(result != null);
result.path.reversed.forEach((RenderNode node) {
node.handlePointer(event);
});
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698