Bug 1266288 - Track changes to all margin properties for scroll-linked effects. r=mstange
authorKartikaya Gupta <kgupta@mozilla.com>
Thu, 21 Apr 2016 11:41:17 -0400
changeset 332233 9e5cdc2e73c495cd48ced4de33a52047fedb2a55
parent 332232 c47cd2b5f4a6ecfeb4ca629eb335696b5883090b
child 332234 1b2c3cc8c06f7e257a8040c04086a99889356b2d
push id6048
push userkmoir@mozilla.com
push dateMon, 06 Jun 2016 19:02:08 +0000
treeherdermozilla-beta@46d72a56c57d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmstange
bugs1266288
milestone48.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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