Top down version of algorithm to position margins and floats in LayoutNG

third_party/WebKit/Source/core/layout/ng/ng_box_fragment.cc

Index: third_party/WebKit/Source/core/layout/ng/ng_box_fragment.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_box_fragment.cc b/third_party/WebKit/Source/core/layout/ng/ng_box_fragment.cc
index cda163878d7a046513c72502aa33c946e166e3e8..e24bfa04f4247caa0dfbe9397b510b13ec2fd9a0 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_box_fragment.cc
+++ b/third_party/WebKit/Source/core/layout/ng/ng_box_fragment.cc
@@ -9,12 +9,16 @@
 
 namespace blink {
 
-NGDeprecatedMarginStrut NGBoxFragment::MarginStrut() const {
+const WTF::Optional<NGLogicalOffset>& NGBoxFragment::BfcOffset() const {
+  return toNGPhysicalBoxFragment(physical_fragment_)->BfcOffset();

[ikilpatrick, 2017/01/30 22:26:49]

may want to add a similar WRITING_MODE_IGNORED here.

[Gleb Lanbin, 2017/01/31 00:25:02]

Done.

+}
+
+const NGMarginStrut& NGBoxFragment::EndMarginStrut() const {
   WRITING_MODE_IGNORED(
       "Accessing the margin strut is fine here. Changing the writing mode"
-      "establishes a new formatting context, for which a margin strut is never"
-      "set for a fragment.");
-  return toNGPhysicalBoxFragment(physical_fragment_)->MarginStrut();
+      "establishes a new formatting context, for which a margin strut is"
+      "never set for a fragment.");
+  return toNGPhysicalBoxFragment(physical_fragment_)->EndMarginStrut();
 }
 
 }  // namespace blink