From 5ea4763025d6d738fb36acc0ca59aee091eb0622 Mon Sep 17 00:00:00 2001
From: Adam Barth <abarth@chromium.org>
Date: Thu, 3 Sep 2015 16:12:25 -0700
Subject: [PATCH] defaultHitTestChildren should return whether it hit something

Some clients want to know whether they hit their children.
---
 packages/flutter/lib/src/rendering/box.dart | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/packages/flutter/lib/src/rendering/box.dart b/packages/flutter/lib/src/rendering/box.dart
index 2af20e181c..bd7904e833 100644
--- a/packages/flutter/lib/src/rendering/box.dart
+++ b/packages/flutter/lib/src/rendering/box.dart
@@ -519,7 +519,7 @@ abstract class RenderBoxContainerDefaultsMixin<ChildType extends RenderBox, Pare
     return result;
   }
 
-  void defaultHitTestChildren(HitTestResult result, { Point position }) {
+  bool defaultHitTestChildren(HitTestResult result, { Point position }) {
     // the x, y parameters have the top left of the node's box as the origin
     ChildType child = lastChild;
     while (child != null) {
@@ -527,9 +527,10 @@ abstract class RenderBoxContainerDefaultsMixin<ChildType extends RenderBox, Pare
       Point transformed = new Point(position.x - child.parentData.position.x,
                                     position.y - child.parentData.position.y);
       if (child.hitTest(result, position: transformed))
-        break;
+        return true;
       child = child.parentData.previousSibling;
     }
+    return false;
   }
 
   void defaultPaint(PaintingContext context, Offset offset) {