Remove redundant IsSVGForeignObject() call

The DynamicTo<LayoutSVGForeignObject>(...) is sufficient.

Change-Id: If73029df141d61efab8271e6d980e12dc954a858
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109772
Commit-Queue: Stephen Chenney <schenney@chromium.org>
Auto-Submit: Fredrik Söderquist <fs@opera.com>
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751731}

third_party/blink/renderer/core/paint/svg_container_painter.cc

@@ -83,8 +83,7 @@ void SVGContainerPainter::Paint(const PaintInfo& paint_info) {
     if (continue_rendering) {
       for (LayoutObject* child = layout_svg_container_.FirstChild(); child;
            child = child->NextSibling()) {
-        auto* foreign_object = DynamicTo<LayoutSVGForeignObject>(*child);
-        if (child->IsSVGForeignObject()) {
+        if (auto* foreign_object = DynamicTo<LayoutSVGForeignObject>(*child)) {
           SVGForeignObjectPainter(*foreign_object)
               .PaintLayer(paint_state.GetPaintInfo());
         } else {