[LayoutNG] Handle border and padding when sizing a block and when placing its children.

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

 // Copyright 2016 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/layout/ng/ng_block_layout_algorithm.h"
 
 #include "core/layout/LayoutBox.h"
 #include "core/layout/ng/ng_constraint_space.h"
 #include "core/layout/ng/ng_fragment_builder.h"
 #include "core/layout/ng/ng_fragment.h"
@@ skipping 29 matching lines @@
     case kStateInit: {
       LayoutUnit inline_size =
           computeInlineSizeForFragment(*constraint_space, *style_);
       // TODO(layout-ng): For quirks mode, should we pass blockSize instead of
       // -1?
       LayoutUnit block_size = computeBlockSizeForFragment(
           *constraint_space, *style_, LayoutUnit(-1));
       constraint_space_for_children_ =
           new NGConstraintSpace(*constraint_space, NGLogicalOffset(),
                                 NGLogicalSize(inline_size, block_size));
-      content_size_ = LayoutUnit();
+      content_size_ =
+          computeBorderAndPaddingBlockStart(*constraint_space, *style_);
 
       builder_ = new NGFragmentBuilder(NGPhysicalFragmentBase::FragmentBox);
       builder_->SetInlineSize(inline_size).SetBlockSize(block_size);
       current_child_ = first_child_;
       state_ = kStateChildLayout;
       return false;
     }
     case kStateChildLayout: {
       if (current_child_) {
         NGFragment* fragment;
         if (!current_child_->Layout(constraint_space_for_children_, &fragment))
           return false;
         NGBoxStrut child_margins = computeMargins(
             *constraint_space_for_children_, *current_child_->Style());
 
         LayoutUnit margin_block_start =
             CollapseMargins(child_margins, fragment->MarginStrut());
 
+        // TODO(layout-ng): This is probably something we shouldn't calculate
+        // over and over again for each child.
+        LayoutUnit content_inline_start_edge =
+            computeBorderAndPaddingInlineStart(*constraint_space, *style_);
+
         // TODO(layout-ng): Support auto margins
         builder_->AddChild(fragment,
-                           NGLogicalOffset(child_margins.inline_start,
+                           NGLogicalOffset(content_inline_start_edge +
+                                               child_margins.inline_start,
                                            content_size_ + margin_block_start));
 
         content_size_ += fragment->BlockSize() + margin_block_start;
         max_inline_size_ =
             std::max(max_inline_size_,
                      fragment->InlineSize() + child_margins.InlineSum());
         current_child_ = current_child_->NextSibling();
         if (current_child_)
           return false;
       }
@@ skipping 39 matching lines @@
     margin_block_start = ComputeCollapsedMarginBlockStart(
         prev_child_margin_strut_, curr_margin_strut);
 
   prev_child_margin_strut_ = curr_margin_strut;
   // TODO(layout-ng): support other Margin Collapsing use cases,
   // i.e. support 0 height elements etc.
   return margin_block_start;
 }
 
 }  // namespace blink