Ink splashes should start at the touch point

sky/sdk/lib/framework/rendering/object.dart

Index: sky/sdk/lib/framework/rendering/object.dart
diff --git a/sky/sdk/lib/framework/rendering/object.dart b/sky/sdk/lib/framework/rendering/object.dart
index e6f8ba72814da5726e8fff04dd6e7190e94ad1ff..f08dff4b6a28f7a54303980a236a1f8f8503965f 100644
--- a/sky/sdk/lib/framework/rendering/object.dart
+++ b/sky/sdk/lib/framework/rendering/object.dart
@@ -204,7 +204,7 @@ abstract class RenderObject extends AbstractNode {
 
   // EVENTS
 
-  void handleEvent(sky.Event event) {
+  void handleEvent(sky.Event event, HitTestData data) {
     // override this if you have a client, to hand it to the client
     // override this if you want to do anything with the event
   }
@@ -253,13 +253,19 @@ abstract class RenderObject extends AbstractNode {
 
 }
 
+class HitTestData {

[Hixie, 2015/06/10 20:51:49]

HitTestEntry

and rename all the variables that refer to it as "data" to "entry".

+  final RenderObject entry;

[Hixie, 2015/06/10 20:51:49]

Rename this to "target".

+
+  HitTestData(this.entry);
+}

[Hixie, 2015/06/10 20:51:49]

Constructor first, then fields.
Also, make the constructor const. Not that it really matters since RenderObjects
can't be const, but...

+
 class HitTestResult {
-  final List<RenderObject> path = new List<RenderObject>();
+  final List<HitTestData> path = new List<HitTestData>();
 
-  RenderObject get result => path.first;
+  RenderObject get result => path.first.entry;

[Hixie, 2015/06/10 20:51:49]

If we don't use this, drop it.
If we do, maybe rename it "finalTarget" or something?

 
-  void add(RenderObject node) {
-    path.add(node);
+  void add(HitTestData data) {
+    path.add(data);
   }
 }