Drop unnecessary Children() accessor from LayoutSVG{Container,Root}

Change-Id: I4187fdbbb265aab9d679781130c923936a4ad9b9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2494884
Auto-Submit: Fredrik Söderquist <fs@opera.com>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#820227}

third_party/blink/renderer/core/layout/svg/layout_svg_container.h

@@ -43,13 +43,13 @@ class LayoutSVGContainer : public LayoutSVGModelObject {
 
   LayoutObject* FirstChild() const {
     NOT_DESTROYED();
-    DCHECK_EQ(Children(), VirtualChildren());
-    return Children()->FirstChild();
+    DCHECK_EQ(&content_.Children(), VirtualChildren());
+    return content_.Children().FirstChild();
   }
   LayoutObject* LastChild() const {
     NOT_DESTROYED();
-    DCHECK_EQ(Children(), VirtualChildren());
-    return Children()->LastChild();
+    DCHECK_EQ(&content_.Children(), VirtualChildren());
+    return content_.Children().LastChild();
   }
 
   void Paint(const PaintInfo&) const override;
@@ -82,11 +82,11 @@ class LayoutSVGContainer : public LayoutSVGModelObject {
  protected:
   LayoutObjectChildList* VirtualChildren() final {
     NOT_DESTROYED();
-    return Children();
+    return &content_.Children();
   }
   const LayoutObjectChildList* VirtualChildren() const final {
     NOT_DESTROYED();
-    return Children();
+    return &content_.Children();
   }
   SVGContentContainer& Content() { return content_; }
 
@@ -119,15 +119,6 @@ class LayoutSVGContainer : public LayoutSVGModelObject {
   void DescendantIsolationRequirementsChanged(DescendantIsolationState) final;
 
  private:
-  const LayoutObjectChildList* Children() const {
-    NOT_DESTROYED();
-    return &content_.Children();
-  }
-  LayoutObjectChildList* Children() {
-    NOT_DESTROYED();
-    return &content_.Children();
-  }
-
   SVGContentContainer content_;
   // TODO(fs): Some of this state can move to the "child list" object.
   FloatRect object_bounding_box_;

third_party/blink/renderer/core/layout/svg/layout_svg_root.h

@@ -48,13 +48,13 @@ class CORE_EXPORT LayoutSVGRoot final : public LayoutReplaced {
 
   LayoutObject* FirstChild() const {
     NOT_DESTROYED();
-    DCHECK_EQ(Children(), VirtualChildren());
-    return Children()->FirstChild();
+    DCHECK_EQ(&content_.Children(), VirtualChildren());
+    return content_.Children().FirstChild();
   }
   LayoutObject* LastChild() const {
     NOT_DESTROYED();
-    DCHECK_EQ(Children(), VirtualChildren());
-    return Children()->LastChild();
+    DCHECK_EQ(&content_.Children(), VirtualChildren());
+    return content_.Children().LastChild();
   }
 
   bool IsLayoutSizeChanged() const {
@@ -123,22 +123,13 @@ class CORE_EXPORT LayoutSVGRoot final : public LayoutReplaced {
   }
   LayoutRect ComputeContentsVisualOverflow() const;
 
-  const LayoutObjectChildList* Children() const {
-    NOT_DESTROYED();
-    return &content_.Children();
-  }
-  LayoutObjectChildList* Children() {
-    NOT_DESTROYED();
-    return &content_.Children();
-  }
-
   LayoutObjectChildList* VirtualChildren() override {
     NOT_DESTROYED();
-    return Children();
+    return &content_.Children();
   }
   const LayoutObjectChildList* VirtualChildren() const override {
     NOT_DESTROYED();
-    return Children();
+    return &content_.Children();
   }
 
   bool IsOfType(LayoutObjectType type) const override {