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

Side by Side 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, 6 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 import 'layout2.dart'; 5 import 'layout2.dart';
6 import 'dart:sky' as sky; 6 import 'dart:sky' as sky;
7 7
8 class AppView { 8 class AppView {
9 9
10 AppView(RenderBox root) { 10 AppView(RenderBox root) {
11 sky.view.setEventCallback(_handleEvent); 11 sky.view.setEventCallback(_handleEvent);
12 sky.view.setBeginFrameCallback(_beginFrame); 12 sky.view.setBeginFrameCallback(_beginFrame);
13 13
14 _renderView = new RenderView(child: root); 14 _renderView = new RenderView(child: root);
15 _renderView.attach(); 15 _renderView.attach();
16 _renderView.layout(newWidth: sky.view.width, newHeight: sky.view.height); 16 _renderView.layout(newWidth: sky.view.width, newHeight: sky.view.height);
17 17
18 sky.view.scheduleFrame(); 18 sky.view.scheduleFrame();
19 } 19 }
20 20
21 RenderView _renderView; 21 RenderView _renderView;
22 22
23 Map<int, HitTestResult> _hitTestResultForPointer = new Map<int, HitTestResult> ();
24
23 RenderBox get root => _renderView.child; 25 RenderBox get root => _renderView.child;
24 void set root(RenderBox value) { 26 void set root(RenderBox value) {
25 _renderView.child = value; 27 _renderView.child = value;
26 } 28 }
27
28 void _beginFrame(double timeStamp) { 29 void _beginFrame(double timeStamp) {
29 RenderNode.flushLayout(); 30 RenderNode.flushLayout();
30 _renderView.paintFrame(); 31 _renderView.paintFrame();
31 } 32 }
32 33
33 void _handleEvent(sky.Event event) { 34 void _handleEvent(sky.Event event) {
34 if (event is! sky.PointerEvent) 35 if (event is sky.PointerEvent)
35 return; 36 _handlePointerEvent(event);
36 HitTestResult result = new HitTestResult(); 37 }
37 _renderView.hitTest(result, x: event.x, y: event.y); 38
39 void _handlePointerEvent(sky.PointerEvent event) {
40 HitTestResult result;
41 switch(event.type) {
42 case 'pointerdown':
43 result = new HitTestResult();
44 _renderView.hitTest(result, x: event.x, y: event.y);
45 _hitTestResultForPointer[event.pointer] = result;
46 break;
47 case 'pointerup':
48 case 'pointercancel':
49 result = _hitTestResultForPointer[event.pointer];
50 _hitTestResultForPointer.remove(event.pointer);
51 break;
52 case 'pointermove':
53 result = _hitTestResultForPointer[event.pointer];
54 break;
55 }
56 assert(result != null);
38 result.path.reversed.forEach((RenderNode node) { 57 result.path.reversed.forEach((RenderNode node) {
39 node.handlePointer(event); 58 node.handlePointer(event);
40 }); 59 });
41 } 60 }
42 61
43 } 62 }
OLDNEW
« 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