Bug 1266288 - Track changes to all margin properties for scroll-linked effects. r?mstange draft
authorKartikaya Gupta <kgupta@mozilla.com>
Thu, 21 Apr 2016 11:41:17 -0400
changeset 354852 3f3689d6ac28d1bc97d87e3e34e9da8d7a522103
parent 354827 1152d99d8c53ac9dae371a6e6d9fab03d3f98697
child 519087 70c840d316c78a283f85590f9820dc17ec953ded
push id16170
push userkgupta@mozilla.com
push dateThu, 21 Apr 2016 15:41:38 +0000
reviewersmstange
bugs1266288
milestone48.0a1
Bug 1266288 - Track changes to all margin properties for scroll-linked effects. r?mstange MozReview-Commit-ID: D0wN1TdFdtM
layout/style/nsDOMCSSDeclaration.cpp
--- a/layout/style/nsDOMCSSDeclaration.cpp
+++ b/layout/style/nsDOMCSSDeclaration.cpp
@@ -80,20 +80,25 @@ nsDOMCSSDeclaration::SetPropertyValue(co
 {
   switch (aPropID) {
     case eCSSProperty_background_position:
     case eCSSProperty_transform:
     case eCSSProperty_top:
     case eCSSProperty_left:
     case eCSSProperty_bottom:
     case eCSSProperty_right:
+    case eCSSProperty_margin:
     case eCSSProperty_margin_top:
     case eCSSProperty_margin_left:
     case eCSSProperty_margin_bottom:
     case eCSSProperty_margin_right:
+    case eCSSProperty_margin_inline_start:
+    case eCSSProperty_margin_inline_end:
+    case eCSSProperty_margin_block_start:
+    case eCSSProperty_margin_block_end:
       mozilla::layers::ScrollLinkedEffectDetector::PositioningPropertyMutated();
       break;
     default:
       break;
   }
 
   if (aValue.IsEmpty()) {
     // If the new value of the property is an empty string we remove the