[LFC] Margin box is border box + margins.
authorzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 31 May 2018 14:59:37 +0000 (14:59 +0000)
committerzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 31 May 2018 14:59:37 +0000 (14:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=186124

Reviewed by Antti Koivisto.

* layout/displaytree/DisplayBox.cpp:
(WebCore::Display::Box::marginBox const):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232347 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/layout/displaytree/DisplayBox.cpp

index b4c182d092518b28398995907a6b727890118995..b8c00e19ae4657faeb090bf274adb155562204d8 100644 (file)
@@ -1,3 +1,13 @@
+2018-05-31  Zalan Bujtas  <zalan@apple.com>
+
+        [LFC] Margin box is border box + margins.
+        https://bugs.webkit.org/show_bug.cgi?id=186124
+
+        Reviewed by Antti Koivisto.
+
+        * layout/displaytree/DisplayBox.cpp:
+        (WebCore::Display::Box::marginBox const):
+
 2018-05-31  Zalan Bujtas  <zalan@apple.com>
 
         [LFC] Add quirks mode setter/getting to LayoutContext
index d0c8b9e40f161f62a97ae17f0a55e3ce2aa57abd..df22d8fc44f29b8fb25bf9c419bc853dd37849d5 100644 (file)
@@ -55,10 +55,10 @@ Box::Rect Box::marginBox() const
     ASSERT(m_hasValidMargin);
     auto marginBox = borderBox();
 
-    marginBox.shiftLeftTo(marginBox.left() + m_margin.left);
-    marginBox.shiftTopTo(marginBox.top() + m_margin.top);
-    marginBox.shiftRightTo(marginBox.right() - m_margin.right);
-    marginBox.shiftBottomTo(marginBox.bottom() - m_margin.bottom);
+    marginBox.shiftLeftTo(marginBox.left() - m_margin.left);
+    marginBox.shiftTopTo(marginBox.top() - m_margin.top);
+    marginBox.shiftRightTo(marginBox.right() + m_margin.right);
+    marginBox.shiftBottomTo(marginBox.bottom() + m_margin.bottom);
 
     return marginBox;
 }