Bug 1887867 - Fix button-as-subgrid border/padding computations. r=dholbert

Differential Revision: https://phabricator.services.mozilla.com/D206896

layout/forms/nsHTMLButtonControlFrame.cpp

@@ -51,6 +51,7 @@ void nsHTMLButtonControlFrame::Init(nsIContent* aContent,
 }
 
 NS_QUERYFRAME_HEAD(nsHTMLButtonControlFrame)
+  NS_QUERYFRAME_ENTRY(nsHTMLButtonControlFrame)
   NS_QUERYFRAME_ENTRY(nsIFormControlFrame)
 NS_QUERYFRAME_TAIL_INHERITING(nsContainerFrame)
 

layout/generic/nsGridContainerFrame.cpp

@@ -28,6 +28,7 @@
 #include "nsCSSFrameConstructor.h"
 #include "nsDisplayList.h"
 #include "nsFieldSetFrame.h"
+#include "nsHTMLButtonControlFrame.h"
 #include "nsGfxScrollFrame.h"
 #include "nsHashKeys.h"
 #include "nsIFrameInlines.h"  // for nsIFrame::GetLogicalNormalPosition (don't remove)
@@ -3688,29 +3689,47 @@ static Subgrid* SubgridComputeMarginBorderPadding(
   SizeComputationInput sz(subgridFrame, nullptr, cbWM, pmPercentageBasis);
   subgrid->mMarginBorderPadding =
       sz.ComputedLogicalMargin(cbWM) + sz.ComputedLogicalBorderPadding(cbWM);
-
-  if (aGridItem.mFrame != subgridFrame) {
-    nsHTMLScrollFrame* scrollFrame =
-        do_QueryFrame(aGridItem.mFrame->GetScrollTargetFrame());
-    if (scrollFrame) {
-      MOZ_ASSERT(
-          sz.ComputedLogicalMargin(cbWM) == LogicalMargin(cbWM) &&
-              sz.ComputedLogicalBorder(cbWM) == LogicalMargin(cbWM),
-          "A scrolled inner frame should not have any margin or border!");
-
-      // Add the margin and border from the (outer) scroll frame.
-      SizeComputationInput szScrollFrame(aGridItem.mFrame, nullptr, cbWM,
-                                         pmPercentageBasis);
-      subgrid->mMarginBorderPadding +=
-          szScrollFrame.ComputedLogicalMargin(cbWM) +
-          szScrollFrame.ComputedLogicalBorder(cbWM);
-
-      nsMargin ssz = scrollFrame->IntrinsicScrollbarGutterSize();
-      subgrid->mMarginBorderPadding += LogicalMargin(cbWM, ssz);
+  if (aGridItem.mFrame == subgridFrame) {
+    return subgrid;
   }
 
-    if (aGridItem.mFrame->IsFieldSetFrame()) {
-      const auto* f = static_cast<nsFieldSetFrame*>(aGridItem.mFrame);
+  bool scroller = false;
+  nsIFrame* outerFrame = [&]() -> nsIFrame* {
+    if (nsHTMLScrollFrame* scrollFrame =
+            do_QueryFrame(aGridItem.mFrame->GetScrollTargetFrame())) {
+      scroller = true;
+      return scrollFrame;
+    }
+    if (nsHTMLButtonControlFrame* f = do_QueryFrame(aGridItem.mFrame)) {
+      return f;
+    }
+    return nullptr;
+  }();
+
+  if (outerFrame) {
+    MOZ_ASSERT(sz.ComputedLogicalMargin(cbWM) == LogicalMargin(cbWM) &&
+                   sz.ComputedLogicalBorder(cbWM) == LogicalMargin(cbWM),
+               "A scrolled inner frame / button content frame "
+               "should not have any margin or border / padding!");
+
+    // Add the margin and border from the (outer) frame. Padding is factored-in
+    // for scrollers already (except for the scrollbar gutter), but not for
+    // button-content.
+    SizeComputationInput szOuterFrame(outerFrame, nullptr, cbWM,
+                                      pmPercentageBasis);
+    subgrid->mMarginBorderPadding += szOuterFrame.ComputedLogicalMargin(cbWM) +
+                                     szOuterFrame.ComputedLogicalBorder(cbWM);
+    if (scroller) {
+      nsMargin ssz = static_cast<nsHTMLScrollFrame*>(outerFrame)
+                         ->IntrinsicScrollbarGutterSize();
+      subgrid->mMarginBorderPadding += LogicalMargin(cbWM, ssz);
+    } else {
+      subgrid->mMarginBorderPadding +=
+          szOuterFrame.ComputedLogicalPadding(cbWM);
+    }
+  }
+
+  if (nsFieldSetFrame* f = do_QueryFrame(aGridItem.mFrame)) {
     const auto* inner = f->GetInner();
     auto wm = inner->GetWritingMode();
     LogicalPoint pos = inner->GetLogicalPosition(aGridItem.mFrame->GetSize());
@@ -3719,7 +3738,7 @@ static Subgrid* SubgridComputeMarginBorderPadding(
     LogicalMargin offsets(wm, pos.B(wm), 0, 0, 0);
     subgrid->mMarginBorderPadding += offsets.ConvertTo(cbWM, wm);
   }
-  }
+
   return subgrid;
 }
 

testing/web-platform/tests/css/css-grid/subgrid/subgrid-button-ref.html

@@ -0,0 +1,33 @@
+<!doctype html>
+<meta charset="utf-8">
+<title>CSS test reference</title>
+<style>
+.grid {
+  display: grid;
+  width: 400px;
+  grid-template-columns: auto auto 1fr;
+  background-color: #ccc;
+}
+.button {
+  display: grid;
+  grid-template-columns: subgrid;
+  grid-column: span 3;
+  text-align: initial;
+
+  background: transparent;
+  padding: 5px 10px;
+  margin: 0;
+  border: 2px solid;
+  font: inherit;
+}
+.right {
+  text-align: right;
+}
+</style>
+<div class="grid">
+  <div class="button">
+    <span>hello</span>
+    <span>,</span>
+    <span class="right">world</span>
+  </div>
+</div>

testing/web-platform/tests/css/css-grid/subgrid/subgrid-button.html

@@ -0,0 +1,36 @@
+<!doctype html>
+<meta charset="utf-8">
+<link rel="help" href="https://drafts.csswg.org/css-grid-2/#subgrids">
+<link rel="help" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1887867">
+<link rel="match" href="subgrid-button-ref.html">
+<title>Button subgrid</title>
+<style>
+.grid {
+  display: grid;
+  width: 400px;
+  grid-template-columns: auto auto 1fr;
+  background-color: #ccc;
+}
+.button {
+  display: grid;
+  grid-template-columns: subgrid;
+  grid-column: span 3;
+  text-align: initial;
+
+  background: transparent;
+  padding: 5px 10px;
+  margin: 0;
+  border: 2px solid;
+  font: inherit;
+}
+.right {
+  text-align: right;
+}
+</style>
+<div class="grid">
+  <button class="button">
+    <span>hello</span>
+    <span>,</span>
+    <span class="right">world</span>
+  </button>
+</div>