Skip hit testing containers with invalid bounding boxes

third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp
index 1e62ca9fcd732c047ec41330e5413b32b2ead2ca..c43d548004ed1b7a961bec0a49d4d5c95ddb6480 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp
@@ -187,10 +187,10 @@ bool LayoutSVGContainer::nodeAtFloatPoint(HitTestResult& result, const FloatPoin
         }
     }
 
-    // pointer-events=boundingBox makes it possible for containers to be direct targets
+    // pointer-events: bounding-box makes it possible for containers to be direct targets.
     if (style()->pointerEvents() == PE_BOUNDINGBOX) {
-        ASSERT(isObjectBoundingBoxValid());
-        if (objectBoundingBox().contains(localPoint)) {
+        // Check for a valid bounding box because it can be invalid for empty containers.

[fs, 2016/08/11 15:22:04]

Nit: s/can/will/ ?

[pdr., 2016/08/11 22:51:24]

weaselwords!

Fixed

+        if (isObjectBoundingBoxValid() && objectBoundingBox().contains(localPoint)) {
             const LayoutPoint& localLayoutPoint = roundedLayoutPoint(localPoint);
             updateHitTestResult(result, localLayoutPoint);
             if (result.addNodeToListBasedTestResult(element(), localLayoutPoint) == StopHitTesting)