Chromium Code Reviews Chromium Code Reviews
Issue 1142283004:
Implement a Hit Test Cache. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| OLD | NEW |
|---|---|
| 1 Count how many hit tests are required for various event scenarios. Hit tests can be expensive and it's often tempting to add more. These values should only ever be changed to go down, not up. | 1 Count how many hit tests are required for various event scenarios. Hit tests can be expensive and it's often tempting to add more. These values should only ever be changed to go down, not up. |
| 2 | 2 |
| 3 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE ". | 3 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE ". |
| 4 | 4 |
| 5 | 5 |
| 6 PASS successfullyParsed is true | 6 PASS successfullyParsed is true |
| 7 | 7 |
| 8 TEST COMPLETE | 8 TEST COMPLETE |
| 9 Event on a simple div | 9 Event on a simple div |
| 10 --------------------- | 10 --------------------- |
| 11 Initial: 0 | 11 Initial: 0/0 |
| 12 MouseMove: 1 | 12 MouseMove: 1/0 |
| 13 MouseDown: 3 | 13 MouseDown: 3/2 [with multiTargetTapNotification: 3/1] |
|
Rick Byers
2015/06/05 20:48:59
multiTargetTapNotification really shouldn't be cha
dtapuska
2015/06/09 18:21:23
Done.
| |
| 14 MouseUp: 1 | 14 MouseUp: 1/1 [with multiTargetTapNotification: 1/0] |
| 15 Wheel: 1 | 15 Wheel: 1/1 [with multiTargetTapNotification: 1/0] |
| 16 TouchStart: 1 | 16 TouchStart: 1/0 |
| 17 TouchMove: 0 | 17 TouchMove: 0/0 |
| 18 TouchEnd: 0 | 18 TouchEnd: 0/0 |
| 19 GestureTapDown: 2 | 19 GestureTapDown: 2/1 [with multiTargetTapNotification: 2/0] |
| 20 GestureShowPress: 2 | 20 GestureShowPress: 2/1 [with multiTargetTapNotification: 2/0] |
| 21 GestureTap: 4 [with multiTargetTapNotification: 5] | 21 GestureTap: 4/3 [with multiTargetTapNotification: 5/3] |
| 22 GestureScrollBegin: 1 | 22 GestureScrollBegin: 1/1 |
| 23 GestureTapCancel: 1 | 23 GestureTapCancel: 1/0 |
| 24 GestureScrollUpdate: 0 | 24 GestureScrollUpdate: 0/0 |
| 25 GestureScrollEnd: 0 | 25 GestureScrollEnd: 0/0 |
| 26 | 26 |
| 27 Event entirely over one iframe nested in another | 27 Event entirely over one iframe nested in another |
| 28 --------------------- | 28 --------------------- |
| 29 Initial: 0 0 0 | 29 Initial: 0/0 0/0 0/0 |
| 30 MouseMove: 1 1 1 | 30 MouseMove: 1/0 1/0 1/0 [with multiTargetTapNotification: 1/0 1/1 1/1] |
| 31 MouseDown: 2 2 2 | 31 MouseDown: 2/1 2/1 2/1 [with multiTargetTapNotification: 2/0 2/1 2/2] |
| 32 MouseUp: 1 1 1 | 32 MouseUp: 1/1 1/1 1/1 [with multiTargetTapNotification: 1/0 1/1 1/1] |
| 33 Wheel: 1 1 1 | 33 Wheel: 1/1 1/1 1/1 [with multiTargetTapNotification: 1/0 1/1 1/1] |
| 34 TouchStart: 1 1 1 | 34 TouchStart: 1/0 1/0 1/0 |
| 35 TouchMove: 0 0 0 | 35 TouchMove: 0/0 0/0 0/0 |
| 36 TouchEnd: 0 0 0 | 36 TouchEnd: 0/0 0/0 0/0 |
| 37 GestureTapDown: 1 1 2 | 37 GestureTapDown: 1/0 1/0 2/1 |
| 38 GestureShowPress: 1 1 2 | 38 GestureShowPress: 1/0 1/0 2/1 |
| 39 GestureTap: 1 1 4 | 39 GestureTap: 1/0 1/0 4/3 |
| 40 GestureScrollBegin: 1 1 1 | 40 GestureScrollBegin: 1/1 1/1 1/1 [with multiTargetTapNotification: 1/0 1/1 1/1] |
| 41 GestureTapCancel: 1 1 1 | 41 GestureTapCancel: 1/0 1/0 1/0 |
| 42 GestureScrollUpdate: 0 0 0 | 42 GestureScrollUpdate: 0/0 0/0 0/0 |
| 43 GestureScrollEnd: 0 0 0 | 43 GestureScrollEnd: 0/0 0/0 0/0 |
| 44 | 44 |
| 45 Event near boundary of two iframes | 45 Event near boundary of two iframes |
| 46 --------------------- | 46 --------------------- |
| 47 Initial: 0 0 0 | 47 Initial: 0/0 0/0 0/0 |
| 48 MouseMove: 1 1 0 | 48 MouseMove: 1/0 1/0 0/0 [with multiTargetTapNotification: 1/0 1/1 0/0] |
| 49 MouseDown: 2 2 0 | 49 MouseDown: 2/1 2/1 0/0 [with multiTargetTapNotification: 2/0 2/0 0/0] |
| 50 MouseUp: 1 1 0 | 50 MouseUp: 1/1 1/1 0/0 [with multiTargetTapNotification: 1/0 1/1 0/0] |
| 51 Wheel: 1 1 0 | 51 Wheel: 1/1 1/1 0/0 [with multiTargetTapNotification: 1/0 1/1 0/0] |
| 52 TouchStart: 1 1 0 | 52 TouchStart: 1/0 1/0 0/0 |
| 53 TouchMove: 0 0 0 | 53 TouchMove: 0/0 0/0 0/0 |
| 54 TouchEnd: 0 0 0 | 54 TouchEnd: 0/0 0/0 0/0 |
| 55 GestureTapDown: 1 4 16 | 55 GestureTapDown: 1/0 4/0 16/0 |
| 56 GestureShowPress: 1 4 16 | 56 GestureShowPress: 1/0 4/0 16/0 |
| 57 GestureTap: 1 4 16 [with multiTargetTapNotification: 2 8 32] | 57 GestureTap: 1/0 4/0 16/0 [with multiTargetTapNotification: 2/0 8/0 32/0] |
| 58 GestureScrollBegin: 1 1 0 | 58 GestureScrollBegin: 1/1 1/1 0/0 [with multiTargetTapNotification: 1/0 1/1 0/0] |
| 59 GestureTapCancel: 1 1 0 | 59 GestureTapCancel: 1/0 1/1 0/0 |
| 60 GestureScrollUpdate: 0 0 0 | 60 GestureScrollUpdate: 0/0 0/0 0/0 |
| 61 GestureScrollEnd: 0 0 0 | 61 GestureScrollEnd: 0/0 0/0 0/0 |
| 62 | 62 |
| 63 Event on a simple div (desktop viewport) | 63 Event on a simple div (desktop viewport) |
| 64 --------------------- | 64 --------------------- |
| 65 Initial: 0 | 65 Initial: 0/0 |
| 66 MouseMove: 1 | 66 MouseMove: 1/0 |
| 67 MouseDown: 3 | 67 MouseDown: 3/2 [with multiTargetTapNotification: 3/1] |
| 68 MouseUp: 1 | 68 MouseUp: 1/1 [with multiTargetTapNotification: 1/0] |
| 69 Wheel: 1 | 69 Wheel: 1/1 [with multiTargetTapNotification: 1/0] |
| 70 TouchStart: 1 | 70 TouchStart: 1/0 |
| 71 TouchMove: 0 | 71 TouchMove: 0/0 |
| 72 TouchEnd: 0 | 72 TouchEnd: 0/0 |
| 73 GestureTapDown: 2 | 73 GestureTapDown: 2/1 [with multiTargetTapNotification: 2/0] |
| 74 GestureShowPress: 2 | 74 GestureShowPress: 2/1 [with multiTargetTapNotification: 2/0] |
| 75 GestureTap: 4 [with multiTargetTapNotification: 6] | 75 GestureTap: 4/3 [with multiTargetTapNotification: 6/3] |
| 76 GestureScrollBegin: 1 | 76 GestureScrollBegin: 1/1 |
| 77 GestureTapCancel: 1 | 77 GestureTapCancel: 1/0 |
| 78 GestureScrollUpdate: 0 | 78 GestureScrollUpdate: 0/0 |
| 79 GestureScrollEnd: 0 | 79 GestureScrollEnd: 0/0 |
| 80 | 80 |
| 81 | 81 |
| OLD | NEW |
