f340cdf67edb870645ddf499da690e60eff7f73f: Bug 978833 patch 12 - Use the css::Declaration instead of the css::StyleRule as the matching rule. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:10 +0800 - rev 306959
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 12 - Use the css::Declaration instead of the css::StyleRule as the matching rule. r=heycam This is the key change in this patch series; it changes the object we use for style data (currently nsIStyleRule) identity. It allows removing some hacks we have to deal with that for StyleRule, and avoids having to write similar hacks for nsCSSKeyframeRule and nsCSSPageRule (which are broken without this). I confirmed locally that it is this patch that fixes both of the todo_is mochitests, by building and testing with the patch queue through patch 11, and again through patch 12.
e69922893211b78e5593fbd1c795e66248a357a1: Bug 978833 patch 11 - Add missing #includes in preparation for nsRuleWalker.h #include change in following patch. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:09 +0800 - rev 306958
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 11 - Add missing #includes in preparation for nsRuleWalker.h #include change in following patch. r=heycam Patch 12 changes nsRuleWalker.h from including StyleRule.h to including Declaration.h; this fixes other headers to deal with that change based on the include-what-you-use principle.
da3bf914effe192549fc81b44363fa9ef36a0e69: Bug 978833 patch 10 - Raise consistent exceptions so that the exception doesn't depend on rule destruction timing. r=bzbarsky
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:09 +0800 - rev 306957
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 10 - Raise consistent exceptions so that the exception doesn't depend on rule destruction timing. r=bzbarsky Prior to patch 12, rule destruction for rules that were matched doesn't happen until rule tree GC. This means that GetCSSDeclaration is less likely to return null, but then GetCSSParsingEnvironment might fail. With StyleRule no longer participating in the rule tree, they're more likely to be destroyed quickly, leading to the !olddecl failure case instead of the !env.mPrincipal failure case. This is needed to avoid patch 12 causing: TEST-UNEXPECTED-FAIL | layout/inspector/tests/chrome/test_bug727834.xul | original rule is not available for modification anymore - got "NS_ERROR_FAILURE", expected "NS_ERROR_NOT_AVAILABLE"
3a1bf2c602114a72dfd99a6aeb737f3cb116076b: Bug 978833 patch 9 - Move keyframe !important data assertion to where it will continue to happen. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:09 +0800 - rev 306956
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 9 - Move keyframe !important data assertion to where it will continue to happen. r=heycam The current location of the assertion will stop being called in patch 12 and will go away in patch 15; the new location is valid both before and after patch 12.
0adcd9f3fac0358d859575727b3febeaf420c512: Bug 978833 patch 8 - Call SetImmutable for declarations of @page and keyframe rules. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:09 +0800 - rev 306955
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 8 - Call SetImmutable for declarations of @page and keyframe rules. r=heycam This probably should have been done before, but prior to this patch series, dynamic changes of the declarations on these rules were broken due to rule immutability violations; now that is no longer the case, but to benefit from that, I believe we actually need to mark the declarations as immutable once matched so that dynamic changes will trigger construction of a new declaration (which thus has a new nsIStyleRule identity).
4b633979383a8174c450ed7f77c0f06b6095f404: Bug 978833 patch 7 - Fuse allocation of ImportantStyleData with Declaration. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:09 +0800 - rev 306954
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 7 - Fuse allocation of ImportantStyleData with Declaration. r=heycam Note that this adds a new public API to css::Declaration; the equivalent API is removed from css::StyleRule and nsCSSPageRule in patch 13. But the removal and addition need to be on opposite sides of patch 12. This fused allocation is no larger than having a pointer, and it removes having to worry about cycles.
e62f0b7f0a02d3d6b7be90cef29075a56894738d: Bug 978833 patch 6 - Move ImportantStyleData from StyleRule.{h,cpp} to Declaration.{h,cpp} r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:09 +0800 - rev 306953
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 6 - Move ImportantStyleData from StyleRule.{h,cpp} to Declaration.{h,cpp} r=heycam This is needed for patch 7. Note that this removes an unused "friend class StyleRule;" declaration.
3b535acc6b79600f26f60aaf6ab724573fb46c88: Bug 978833 patch 5 - Rename ImportantRule to ImportantStyleData. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:08 +0800 - rev 306952
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 5 - Rename ImportantRule to ImportantStyleData. r=heycam (This is part of a longer term plan to rename nsIStyleRule to StyleData and nsIStyleRuleProcessor to StyleDataSource. I'm not doing all of that here, though.)
4d51af920be6db324ef06f980bc7ba7520f7ea55: Bug 978833 patch 4 - Add pointer back from css::Declaration to css::Rule. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:08 +0800 - rev 306951
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 4 - Add pointer back from css::Declaration to css::Rule. r=heycam This is used in patch 12, in nsStyleSet::AssertNoCSSRules and in inDOMUtils::GetCSSStyleRules.
d04634d822423fcde66c7dc5a120de9e5e2be167: Bug 978833 patch 3 - Make css::Declaration implement nsIStyleRule. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:08 +0800 - rev 306950
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 3 - Make css::Declaration implement nsIStyleRule. r=heycam We switch to using this implementation instead of the one in css::StyleRule in patch 12. (Yes, implementing QueryInterface for a CID is ugly, but it's the same thing StyleRule does. Unfortunately now we'll need to have it in both places.)
65c8c4d27598055a1255d24f96eafa1850dccba7: Bug 978833 patch 2 - Make css::Declaration reference-counted. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:08 +0800 - rev 306949
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 2 - Make css::Declaration reference-counted. r=heycam This is done in preparation for making it implement nsIStyleRule, which happens in patch 3, and which is used in patch 12.
a377963bf9bbc0a9c658a513137d778b838d0b94: Bug 978833 patch 1 - Add mochitest for bug 978833. r=heycam
L. David Baron <dbaron@dbaron.org> - Thu, 05 Nov 2015 16:44:07 +0800 - rev 306948
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 978833 patch 1 - Add mochitest for bug 978833. r=heycam I confirmed that without the patch series, the tests marked todo_is do fail.
7f6e61d6097c166b541f513fee856d02f246742d: Bug 1218643 - correct a DOM test. r=smaug
Lars T Hansen <lhansen@mozilla.com> - Wed, 04 Nov 2015 12:23:17 +0100 - rev 306947
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 1218643 - correct a DOM test. r=smaug
a196bd265eabe5ac30b12967ca9aa5dc31bbe916: Bug 1218643 - remove support for deprecated asm.js heap length. r=luke
Lars T Hansen <lhansen@mozilla.com> - Mon, 02 Nov 2015 09:07:47 +0100 - rev 306946
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 1218643 - remove support for deprecated asm.js heap length. r=luke
0cdf0710beb8267b58ae5fc15adac10a9e92c29f: Bug 1219403 - r=karlt
Paul Adenot <paul@paul.cx> - Thu, 05 Nov 2015 09:36:25 +0100 - rev 306945
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 1219403 - r=karlt
01428300eb5069a980410aa62b348fe1b07b3a5d: Bug 1220732 - Enable some bc tests. r=mrbkap
Andrew McCreight <continuation@gmail.com> - Tue, 03 Nov 2015 09:04:00 +0100 - rev 306944
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 1220732 - Enable some bc tests. r=mrbkap
70df5b203edb0e9e19c4d64e66bb3a61997a0a6a: Bug 1217557 - set browser.tabs.remote.autostart to False in GeckoInstance; r=automatedtester
Maja Frydrychowicz <mjzffr@gmail.com> - Mon, 02 Nov 2015 18:19:58 -0500 - rev 306943
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 1217557 - set browser.tabs.remote.autostart to False in GeckoInstance; r=automatedtester
f6ec673b3b76e61dfc10a8cf70e5efe861107fa9: bug 1190018 - External protocol handlers don't work in e10s r=mrbkap
Brad Lassey <blassey@mozilla.com> - Tue, 25 Aug 2015 19:42:21 -0400 - rev 306942
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
bug 1190018 - External protocol handlers don't work in e10s r=mrbkap
1cb956020202df4f65eeac7b68794c24610b7a7d: Bug 1212608 - Add parts of firefox_ui_harness to Marionette runner; r=automatedtester
Maja Frydrychowicz <mjzffr@gmail.com> - Fri, 23 Oct 2015 16:43:40 -0400 - rev 306941
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 1212608 - Add parts of firefox_ui_harness to Marionette runner; r=automatedtester * BaseMarionetteRunner: any change to self.bin resets self.marionette and self.tests, so you change binaries between calls to run_tests() * Refactor runtests.py to make it easier to extend/customize the harness for different test suites. Add more error handling.
84a9236266cac6f561908c7c717c05c486305e3c: Bug 1221276 - Don't crash if we don't have a surface. r=nical
Benoit Girard <b56girard@gmail.com> - Tue, 03 Nov 2015 16:07:02 -0500 - rev 306940
Push 7225 by mjzffr@gmail.com at Thu, 05 Nov 2015 20:58:39 +0000
Bug 1221276 - Don't crash if we don't have a surface. r=nical
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip