{"push_id":857924127,"size":7,"distinct_size":7,"ref":"refs/heads/inbound","head":"62dcb6818dcb72e266bba09fe3ba461582c1eed7","before":"84204751300d20592aa56d2735c078c8a56c66c9","commits":[{"sha":"f6a21597aafa1b163939a276ded471ca36c973ba","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1221436 patch 1 - Have a pointer back from the css::Declaration for style attributes to the nsHTMLCSSStyleSheet.  r=heycam\n\nThis is needed for patch 2, since we need a replacement for the\nmechanism to get from rules to the style attribute sheet\n(nsHTMLCSSStyleSheet) that works instead for declarations rather than\nrules.\n\nThis pointer will be merged with the owning rule pointer in patch 4, but\nwe can't do that until after patch 2.\n\n--HG--\nextra : commitid : CzWSnFhsbcq","distinct":true,"url":"https://api.github.com/repos/mozilla/gecko-dev/commits/f6a21597aafa1b163939a276ded471ca36c973ba"},{"sha":"a3eadb1b865005b74390b27a0e49c8b480d96b56","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1221436 patch 2 - For style attributes, only store a css::Declaration instead of a css::StyleRule.  r=heycam\n\nNow that Declaration implements nsIStyleRule, we don't need the memory\noverhead of storing a StyleRule object for style attributes.\n\nWe also need to change this prior to patch 5, because the changes in\npatch 5 that will allow rules to change (but declarations not) would\notherwise break due to style attribute object merging done by\nnsAttrValue::ParseStyleAttribute.\n\n--HG--\nextra : commitid : Ij4yRdba7wa","distinct":true,"url":"https://api.github.com/repos/mozilla/gecko-dev/commits/a3eadb1b865005b74390b27a0e49c8b480d96b56"},{"sha":"a5313b48ed854e420e2bad609370877927763061","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1221436 patch 3 - Remove Rule::SetHTMLCSSStyleSheet and related code, now unused.  r=heycam\n\nThis was made unused by patch 2.\n\n--HG--\nextra : commitid : 5nbboJpVYvk","distinct":true,"url":"https://api.github.com/repos/mozilla/gecko-dev/commits/a5313b48ed854e420e2bad609370877927763061"},{"sha":"4c8c0044c6ef95828b4843a33d6daa08bf1ec822","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1221436 patch 4 - Use the same pointer to store the owning rule and the nsHTMLCSSStyleSheet on css::Declaration, since we never need both.  r=heycam\n\nThe pointer to nsHTMLCSSStyleSheet was added in patch 1 (which was\nneeded before patch 2), but it was only patch 2 that created the\ninvariant that we'd only have one pointer or the other, and never both.\nThus this needs to be done separately, after patch 1.\n\n--HG--\nextra : commitid : 4KGy9sevepg","distinct":true,"url":"https://api.github.com/repos/mozilla/gecko-dev/commits/4c8c0044c6ef95828b4843a33d6daa08bf1ec822"},{"sha":"c054fd2a0949c1a61fdce22f8bd91d1b149de752","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1221436 patch 5 - Stop generating new style rules in DeclarationChanged (and rename it to SetDeclaration), since we no longer need a new style rule for nsIStyleRule identity rules.  r=heycam\n\n(I think the change DOMCSSDeclarationImpl::GetCSSDeclaration fixes a\npurely theoretical bug that would happen if it were possible to get\nahold of a CSS rule without calling either\nCSSStyleSheet::EnsureUniqueInner (which is called by\nCSSRuleListImpl::IndexedGetter) or Declaration::SetImmutable (which is\ncalled by rule matching, including that triggered by\ninIDomUtils::GetCSSStyleRules).  If that were possible, then mutating\nsuch a declaration would mutate the shared Declaration, and then call\nWillDirty which would clone the now-mutated Declaration, leading to the\nmutation applying to both sheets that shared it rather than just the one\nthat should have been modified.)\n\nThis is the simplification allowed by bug 978833 patch 11 (and part of\nwhat we would otherwise have needed to duplicate in @page and keyframe\nrules; we'd also have needed to duplicate the object split between the\ninternal object and the DOM-exposed object, but for style rules that's\nprobably worth keeping for the memory savings).\n\n--HG--\nextra : commitid : 4LV6uqm7iYf","distinct":true,"url":"https://api.github.com/repos/mozilla/gecko-dev/commits/c054fd2a0949c1a61fdce22f8bd91d1b149de752"},{"sha":"764b2b1601666a07c0480409caebdf414e302b32","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1221436 patch 6 - Remove mechanism for replacing style rules.  r=heycam\n\nThis is no longer used, thanks to patch 5.\n\n--HG--\nextra : commitid : 9E12f2cAoMn","distinct":true,"url":"https://api.github.com/repos/mozilla/gecko-dev/commits/764b2b1601666a07c0480409caebdf414e302b32"},{"sha":"62dcb6818dcb72e266bba09fe3ba461582c1eed7","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1221436 patch 7 - Remove ReplaceStyleRule/ReplaceRuleInGroup mechanism.  r=heycam\n\nThis is no longer used, thanks to patch 5.\n\n--HG--\nextra : commitid : HUH1X5rr3Ij","distinct":true,"url":"https://api.github.com/repos/mozilla/gecko-dev/commits/62dcb6818dcb72e266bba09fe3ba461582c1eed7"}]}