Refactor HitTestResult to store the HitTestRequest

Source/core/layout/svg/LayoutSVGResourceClipper.cpp

 /*
  * Copyright (C) 2004, 2005, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  * Copyright (C) 2011 Dirk Schulze <krit@webkit.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 229 matching lines @@
 
     point = animatedLocalTransform.inverse().mapPoint(point);
 
     for (SVGElement* childElement = Traversal<SVGElement>::firstChild(*element()); childElement; childElement = Traversal<SVGElement>::nextSibling(*childElement)) {
         LayoutObject* layoutObject = childElement->layoutObject();
         if (!layoutObject)
             continue;
         if (!layoutObject->isSVGShape() && !layoutObject->isSVGText() && !isSVGUseElement(*childElement))
             continue;
         IntPoint hitPoint;
-        HitTestResult result(hitPoint);
-        if (layoutObject->nodeAtFloatPoint(HitTestRequest(HitTestRequest::SVGClipContent), result, point, HitTestForeground))
+        HitTestResult result(HitTestRequest(HitTestRequest::SVGClipContent), hitPoint);

[Rick Byers, 2015/03/31 13:53:42]

nit: is the 'HitTestRequest' here required or can this be shortened?  The
HitTestRequest ctor is not marked 'explicit' so I believe the compiler should
invoke it implicitly when given a HitTestRequestType.

[ramya.v, 2015/04/01 07:13:00]

Done.

+        if (layoutObject->nodeAtFloatPoint(result, point, HitTestForeground))
             return true;
     }
 
     return false;
 }
 
 FloatRect LayoutSVGResourceClipper::resourceBoundingBox(const LayoutObject* object)
 {
     // Resource was not layouted yet. Give back the boundingBox of the object.
     if (selfNeedsLayout())
         return object->objectBoundingBox();
 
     if (m_clipBoundaries.isEmpty())
         calculateClipContentPaintInvalidationRect();
 
     if (clipPathUnits() == SVGUnitTypes::SVG_UNIT_TYPE_OBJECTBOUNDINGBOX) {
         FloatRect objectBoundingBox = object->objectBoundingBox();
         AffineTransform transform;
         transform.translate(objectBoundingBox.x(), objectBoundingBox.y());
         transform.scaleNonUniform(objectBoundingBox.width(), objectBoundingBox.height());
         return transform.mapRect(m_clipBoundaries);
     }
 
     return m_clipBoundaries;
 }
 
 }