Bug 1277131 : Part 4 - rename -moz-table-outer to -moz-table-wrapper. draft
authorAstley Chen <aschen@mozilla.com>
Thu, 16 Jun 2016 13:35:54 +0100
changeset 378402 0dbceaa96644d60db0f829cdd5c15b6cc0686b06
parent 378401 bd9f6dc69cf6a8eab9ae490139091db827baf1db
child 523529 93b6900e316e0140a0037de370fcf08e4ce5fafe
push id21008
push useraschen@mozilla.com
push dateThu, 16 Jun 2016 12:36:36 +0000
bugs1277131
milestone50.0a1
Bug 1277131 : Part 4 - rename -moz-table-outer to -moz-table-wrapper. MozReview-Commit-ID: CsalRdLLcbJ
layout/generic/nsHTMLReflowState.cpp
layout/style/nsCSSAnonBoxList.h
layout/style/res/ua.css
layout/tables/nsTableFrame.cpp
layout/tables/nsTableWrapperFrame.cpp
--- a/layout/generic/nsHTMLReflowState.cpp
+++ b/layout/generic/nsHTMLReflowState.cpp
@@ -2553,17 +2553,17 @@ nsCSSOffsetState::InitOffsets(WritingMod
       // try to do anything like handling 'auto' widths,
       // 'box-sizing', or 'auto' margins.
       ComputedPhysicalPadding().SizeTo(0,0,0,0);
       SetComputedLogicalBorderPadding(
         tableFrame->GetIncludedOuterBCBorder(mWritingMode));
     }
 
     // The margin is inherited to the table wrapper frame via
-    // the ::-moz-table-outer rule in ua.css.
+    // the ::-moz-table-wrapper rule in ua.css.
     ComputedPhysicalMargin().SizeTo(0, 0, 0, 0);
   } else if (aFrameType == nsGkAtoms::scrollbarFrame) {
     // scrollbars may have had their width or height smashed to zero
     // by the associated scrollframe, in which case we must not report
     // any padding or border.
     nsSize size(frame->GetSize());
     if (size.width == 0 || size.height == 0) {
       ComputedPhysicalPadding().SizeTo(0,0,0,0);
--- a/layout/style/nsCSSAnonBoxList.h
+++ b/layout/style/nsCSSAnonBoxList.h
@@ -47,17 +47,17 @@ CSS_ANON_BOX(framesetBlank, ":-moz-frame
 CSS_ANON_BOX(mozDisplayComboboxControlFrame, ":-moz-display-comboboxcontrol-frame")
 CSS_ANON_BOX(htmlCanvasContent, ":-moz-html-canvas-content")
 
 CSS_ANON_BOX(inlineTable, ":-moz-inline-table")
 CSS_ANON_BOX(table, ":-moz-table")
 CSS_ANON_BOX(tableCell, ":-moz-table-cell")
 CSS_ANON_BOX(tableColGroup, ":-moz-table-column-group")
 CSS_ANON_BOX(tableCol, ":-moz-table-column")
-CSS_ANON_BOX(tableWrapper, ":-moz-table-outer")
+CSS_ANON_BOX(tableWrapper, ":-moz-table-wrapper")
 CSS_ANON_BOX(tableRowGroup, ":-moz-table-row-group")
 CSS_ANON_BOX(tableRow, ":-moz-table-row")
 
 CSS_ANON_BOX(canvas, ":-moz-canvas")
 CSS_ANON_BOX(pageBreak, ":-moz-pagebreak")
 CSS_ANON_BOX(page, ":-moz-page")
 CSS_ANON_BOX(pageContent, ":-moz-pagecontent")
 CSS_ANON_BOX(pageSequence, ":-moz-page-sequence")
--- a/layout/style/res/ua.css
+++ b/layout/style/res/ua.css
@@ -16,17 +16,17 @@
   box-sizing: border-box; /* XXX do we really want this? */
 }
 
 *|*::-moz-inline-table {
   display: inline-table !important;
   box-sizing: border-box; /* XXX do we really want this? */
 }
 
-*|*::-moz-table-outer {
+*|*::-moz-table-wrapper {
   display: inherit !important; /* table or inline-table */
   margin: inherit ! important;
   padding: 0 ! important;
   border: none ! important;
   float: inherit;
   clear: inherit;
   position: inherit;
   top: inherit;
--- a/layout/tables/nsTableFrame.cpp
+++ b/layout/tables/nsTableFrame.cpp
@@ -2642,17 +2642,17 @@ nsTableFrame::GetUsedPadding() const
 
   return nsMargin(0,0,0,0);
 }
 
 /* virtual */ nsMargin
 nsTableFrame::GetUsedMargin() const
 {
   // The margin is inherited to the table wrapper frame via
-  // the ::-moz-table-outer rule in ua.css.
+  // the ::-moz-table-wrapper rule in ua.css.
   return nsMargin(0, 0, 0, 0);
 }
 
 NS_DECLARE_FRAME_PROPERTY_DELETABLE(TableBCProperty, BCPropertyData)
 
 BCPropertyData*
 nsTableFrame::GetBCProperty(bool aCreateIfNecessary) const
 {
--- a/layout/tables/nsTableWrapperFrame.cpp
+++ b/layout/tables/nsTableWrapperFrame.cpp
@@ -211,17 +211,17 @@ nsTableWrapperFrame::BuildDisplayListFor
 
 nsStyleContext*
 nsTableWrapperFrame::GetParentStyleContext(nsIFrame** aProviderFrame) const
 {
   // The table wrapper frame and the (inner) table frame split the style
   // data by giving the table frame the style context associated with
   // the table content node and creating a style context for the wrapper
   // frame that is a *child* of the table frame's style context,
-  // matching the ::-moz-table-outer pseudo-element. html.css has a
+  // matching the ::-moz-table-wrapper pseudo-element. html.css has a
   // rule that causes that pseudo-element (and thus the wrapper table)
   // to inherit *some* style properties from the table frame.  The
   // children of the table inherit directly from the inner table, and
   // the table wrapper's style context is a leaf.
 
   return (*aProviderFrame = InnerTableFrame())->StyleContext();
 }