Bug 1640165 - Address TYLin's and clang-format's comments.
authorEmilio Cobos Álvarez <emilio@crisal.io>
Fri, 22 May 2020 21:55:10 +0000
changeset 531721 9bf0d0baf77b3b4217dc9eb2b7f420f8fd97a92c
parent 531720 1e7e97b9c413a5a135eef8bfe7151d44fd7fc2eb
child 531722 519535f418ad8c35529c565d43996f5597d00bca
push id37442
push userncsoregi@mozilla.com
push dateSat, 23 May 2020 09:21:24 +0000
treeherdermozilla-central@bbcc193fe0f0 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1640165
milestone78.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1640165 - Address TYLin's and clang-format's comments. Differential Revision: https://phabricator.services.mozilla.com/D76556
layout/base/nsCSSFrameConstructor.cpp
layout/base/nsCSSFrameConstructor.h
--- a/layout/base/nsCSSFrameConstructor.cpp
+++ b/layout/base/nsCSSFrameConstructor.cpp
@@ -5300,19 +5300,19 @@ nsCSSFrameConstructor::FindElementTagDat
     return &data;
   }
   switch (aElement.GetNameSpaceID()) {
     case kNameSpaceID_XHTML:
       return FindHTMLData(aElement, aParentFrame, aStyle);
     case kNameSpaceID_MathML:
       return FindMathMLData(aElement, aStyle);
     case kNameSpaceID_SVG:
-      return FindSVGData(
-          aElement, aParentFrame, aFlags.contains(ItemFlag::IsWithinSVGText),
-          aFlags.contains(ItemFlag::AllowTextPathChild), aStyle);
+      return FindSVGData(aElement, aParentFrame,
+                         aFlags.contains(ItemFlag::IsWithinSVGText),
+                         aFlags.contains(ItemFlag::AllowTextPathChild), aStyle);
     case kNameSpaceID_XUL:
       return FindXULTagData(aElement, aStyle);
     default:
       return nullptr;
   }
 }
 
 void nsCSSFrameConstructor::AddFrameConstructionItemsInternal(
--- a/layout/base/nsCSSFrameConstructor.h
+++ b/layout/base/nsCSSFrameConstructor.h
@@ -360,16 +360,17 @@ class nsCSSFrameConstructor final : publ
   already_AddRefed<ComputedStyle> ResolveComputedStyle(nsIContent* aContent);
 
   enum class ItemFlag : uint8_t {
     // Allow page-break before and after items to be created if the
     // style asks for them.
     AllowPageBreak,
     IsGeneratedContent,
     IsWithinSVGText,
+    // The item allows items to be created for SVG <textPath> children.
     AllowTextPathChild,
     // The item is content created by an nsIAnonymousContentCreator frame.
     IsAnonymousContentCreatorContent,
   };
 
   using ItemFlags = mozilla::EnumSet<ItemFlag>;
 
   // Add the frame construction items for the given aContent and aParentFrame