Remove |success| parameter from GeometryMapper's public methods

third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/paint/PaintPropertyTreeBuilderTest.h"
 
 #include "core/html/HTMLIFrameElement.h"
 #include "core/layout/LayoutTreeAsText.h"
 #include "core/paint/ObjectPaintProperties.h"
 #include "core/paint/PaintPropertyTreePrinter.h"
@@ skipping 58 matching lines @@
 
   Settings::setMockScrollbarsEnabled(false);
 }
 
 #define CHECK_VISUAL_RECT(expected, sourceLayoutObject, ancestorLayoutObject,  \
                           slopFactor)                                          \
   do {                                                                         \
     GeometryMapper geometryMapper;                                             \
     LayoutRect source((sourceLayoutObject)->localVisualRect());                \
     source.moveBy((sourceLayoutObject)->paintOffset());                        \
-    bool success = false;                                                      \
     const auto& contentsProperties =                                           \
         *(ancestorLayoutObject)->paintProperties()->contentsProperties();      \
     LayoutRect actual =                                                        \
         LayoutRect(geometryMapper.sourceToDestinationVisualRect(               \
             FloatRect(source), *(sourceLayoutObject)                           \
                                     ->paintProperties()                        \
                                     ->localBorderBoxProperties(),              \
-            contentsProperties, success));                                     \
-    ASSERT_TRUE(success);                                                      \
+            contentsProperties));                                              \
     actual.moveBy(-(ancestorLayoutObject)->paintOffset());                     \
     EXPECT_EQ(expected, actual)                                                \
         << "GeometryMapper: expected: " << expected.toString()                 \
         << ", actual: " << actual.toString();                                  \
                                                                                \
     if (slopFactor == LayoutUnit::max())                                       \
       break;                                                                   \
     LayoutRect slowPathRect = (sourceLayoutObject)->localVisualRect();         \
     (sourceLayoutObject)                                                       \
         ->mapToVisualRectInAncestorSpace(ancestorLayoutObject, slowPathRect);  \
@@ skipping 3044 matching lines @@
 
   LayoutObject* target = getLayoutObjectByElementId("target");
   const auto* localBorderBoxProperties =
       target->paintProperties()->localBorderBoxProperties();
   ASSERT_TRUE(localBorderBoxProperties);
   EXPECT_EQ(LayoutPoint(66, 55), target->paintOffset());
   EXPECT_EQ(effect, localBorderBoxProperties->effect());
 }
 
 }  // namespace blink