{"push_id":610254107,"size":306,"distinct_size":7,"ref":"refs/heads/dougt_push_api_hacking","head":"32e9d991e23b474550af8464d56f95f1c0eeaf89","before":"38daad536ffb368e58cd97ba136925a2f19aea00","commits":[{"sha":"f913231fd159465ac779b18fb1664a5e650c2a8b","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1010675 - Stop allowing button contents to overflow into the CSS padding and border (although still allow them to overflow into our internal focuspadding when the min-content width says the contents don't fit).  r=dholbert\n\nNote that this replaces the code that allows eroding the space with new\ncode that reduces the focusPadding value.\n\n(Also, we previously didn't count the focusPadding towards what could be\neroded, which meant we wouldn't quite get to the edge of the padding and\nborder, because we weren't counting the extra for the focusPadding.)\n\nThe existing reftests that I'm changing from == to != are ones that were\nspecifically testing issues related to erosion of padding.\n\nThe change to 491180-{1,2}-ref.html is because we now *do* erode the\nfocusPadding, which is 3px in the horizontal dimensions (see the\nbutton::-moz-focus-inner styles in forms.css), and that was the only\nnonzero style on the button in 491180-{1,2}.html.","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/f913231fd159465ac779b18fb1664a5e650c2a8b"},{"sha":"43ffda7be510623e151a154976424432ec1c0a65","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1125455 patch 1 - Add boolean for whether an animation of a property wins in the CSS cascade.  r=birtles","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/43ffda7be510623e151a154976424432ec1c0a65"},{"sha":"f93a10293b791bf71fccfe622bb95dae10917b09","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1125455 patch 2 - Set mWinsInCascade for transitions based on whether there are animations.  r=birtles","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/f93a10293b791bf71fccfe622bb95dae10917b09"},{"sha":"beb96f19182a3fc1c64e0328c3396277517781a4","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1125455 patch 3 - Add mochitests for animations overriding transitions.  r=birtles\n\nNote that (at this stage) some of the tests in both files fail (which I\nhave verified locally), as noted by the todos and FIXMEs in the test,\nwhich will be removed in later patches in this bug, as the failures are\nfixed.","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/beb96f19182a3fc1c64e0328c3396277517781a4"},{"sha":"c64617d10d99652e542864a873d153679250d728","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1125455 patch 4 - For main-thread application of transitions, don't apply transitions when animations are also running.  r=birtles\n\nI've verified locally that this patch (not others in this series) fixes\nthe test failures that match the test changes in this patch.","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/c64617d10d99652e542864a873d153679250d728"},{"sha":"40bd226b51c3a005d75e45ac2d76b2f2801465e5","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1125455 patch 5 - Add Animation::GetAnimationOfProperty.  r=birtles\n\nThis is needed for patch 6.","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/40bd226b51c3a005d75e45ac2d76b2f2801465e5"},{"sha":"92c8f10ebf7bc31ec0d01ae33d054e01e71a09e7","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1125455 patch 6 - Only search the properties list of the animation once when adding animations to the compositor.  r=birtles\n\nThis removes the duplication where AddAnimationsForProperty calls\nHasAnimationOfProperty which goes over the list once, and then\nAddAnimationForProperty searches the list again and skips all but the\nitem found before.\n\nIt also makes it easier, in patch 7, to perform additional tests on the\nitem that we found.","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/92c8f10ebf7bc31ec0d01ae33d054e01e71a09e7"},{"sha":"844ee17b53840afc70a12fd59290e5b570bc527d","author":{"email":"7ca824ce4c0c70c001cea10982fa723d4457a91b@dbaron.org","name":"L. David Baron"},"message":"Bug 1125455 patch 7 - For compositor-thread application of transitions, don't apply transitions when animations are also running.  r=birtles\n\nI've verified locally that this patch (not others in this series) fixes\nthe test failures that match the test changes in this patch.","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/844ee17b53840afc70a12fd59290e5b570bc527d"},{"sha":"c9ec579ba23846f37b1d6e3f8fb2c2cf1c7e06e4","author":{"email":"f6530d0c97d9593439974703a5d13c9ab311584e@gmail.com","name":"Xidorn Quan"},"message":"Backout a6dbd23da598 (bug 1139283) since it is currently not needed.\n\n--HG--\nextra : source : 3acc263e7d18fc2946bd043edcf7ae865e3874fe","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/c9ec579ba23846f37b1d6e3f8fb2c2cf1c7e06e4"},{"sha":"fec249e9b18f062ba366ac41d9727104da5f6941","author":{"email":"93ceec6df365c7330fbd7f3e4885a2f8cc9462db@gmail.com","name":"Brian Birtles"},"message":"Bug 1117603 part 1 - Don't assume style rules have been refreshed in GetAnimationRule; r=dbaron\n\nTypically when GetAnimationRule is called, at least for CSS Animations, the\nanimation style rule will have been refreshed. However, in some cases such as\nwhen the Web Animations API is used, the style rule will be marked as needing\nto be refreshed outside of the usual flow. This rule will be refreshed when\nnsAnimationManager::WillRefresh flushes animations but if the refresh driver\nfor the chrome document fires first, we will visit GetAnimationRule before\nthis happens.\n\nThis patch removes the assertion that expects animations to have been\nrefreshed by the time we reach GetAnimationRule causing it to update\nthe animation style rule as necessary.","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/fec249e9b18f062ba366ac41d9727104da5f6941"},{"sha":"d2405f9465f617123f900a0d31a67e272654547b","author":{"email":"93ceec6df365c7330fbd7f3e4885a2f8cc9462db@gmail.com","name":"Brian Birtles"},"message":"Bug 1117603 part 2 - Don't unregister from the refresh driver unless we are also queueing events; r=dbaron","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/d2405f9465f617123f900a0d31a67e272654547b"},{"sha":"b4efb87d4a684e3c76f43e073a030c9c78dd7c70","author":{"email":"f6530d0c97d9593439974703a5d13c9ab311584e@gmail.com","name":"Xidorn Quan"},"message":"Bug 1145036 part 1 - Rename mBorderSpacingX/Y to mBorderSpacingCol/Row. r=heycam\n\n--HG--\nextra : source : 4f0a5e14993dd26e4a25d5f05ea1ec274bebe4e6","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/b4efb87d4a684e3c76f43e073a030c9c78dd7c70"},{"sha":"69bd92ee0ec99a5ba66f6eef41d6a2c381248e1d","author":{"email":"f6530d0c97d9593439974703a5d13c9ab311584e@gmail.com","name":"Xidorn Quan"},"message":"Bug 1145036 part 2 - Rename nsTableFrame::GetCellSpacingX/Y to GetCol/RowSpacing. r=roc\n\n--HG--\nextra : source : f499ce6fd4e7d007a053fb5dda47d32c698d71ce","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/69bd92ee0ec99a5ba66f6eef41d6a2c381248e1d"},{"sha":"523fb231d0cf79c825dc4dca267022b298a39088","author":{"email":"d10450b549249b68cdafbd72422563249912b91a@mozilla.com","name":"Steve Fink"},"message":"Bug 1120655 - Rename lastGC -> destroyingRuntime, r=terrence\n\n--HG--\nextra : rebase_source : b52971daceea1d1f1579b1af68e07b7e1965f86d","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/523fb231d0cf79c825dc4dca267022b298a39088"},{"sha":"3c7aa89ef52310943f1c00fd9323ef7fc6639efb","author":{"email":"d10450b549249b68cdafbd72422563249912b91a@mozilla.com","name":"Steve Fink"},"message":"Bug 1120655 - Suppress zone/compartment collection while iterating, r=terrence\n\n--HG--\nextra : rebase_source : 6f5c36715e74d2ddbc519d882581bf43401a3c0d","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/3c7aa89ef52310943f1c00fd9323ef7fc6639efb"},{"sha":"ccaabb9de0b5827a4da5aa136c49ce1f0fc98eb6","author":{"email":"a129bf763f99531def15560dcd67a8ee94032ae1@mit.edu","name":"Boris Zbarsky"},"message":"Bug 1144743 part 1.  Add a hasPollutedGlobalScope flag to scripts.  r=luke","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/ccaabb9de0b5827a4da5aa136c49ce1f0fc98eb6"},{"sha":"135a28f4b849726150e288bedf5295c8da586acd","author":{"email":"a129bf763f99531def15560dcd67a8ee94032ae1@mit.edu","name":"Boris Zbarsky"},"message":"Bug 1144743 part 2.  Add an option to JS::CompileOptions for hasPollutedGlobalScope.  r=luke","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/135a28f4b849726150e288bedf5295c8da586acd"},{"sha":"948d4ebefa1be8b629dd191a2a3640a0e283b3ea","author":{"email":"a129bf763f99531def15560dcd67a8ee94032ae1@mit.edu","name":"Boris Zbarsky"},"message":"Bug 1144743 part 3.  Set hasPollutedGlobalScope when we're compiling scripts we know will be executed with a non-global scope without cloning them.  r=luke","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/948d4ebefa1be8b629dd191a2a3640a0e283b3ea"},{"sha":"3cfd21de39679cec4055c52f2f191f123225bb64","author":{"email":"a129bf763f99531def15560dcd67a8ee94032ae1@mit.edu","name":"Boris Zbarsky"},"message":"Bug 1144743 part 4.  Set the hasPollutedGlobalScope flag correctly when compiling functions.  r=luke","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/3cfd21de39679cec4055c52f2f191f123225bb64"},{"sha":"9864e3519dbf4d521d735e31700bd5d205c3b71d","author":{"email":"a129bf763f99531def15560dcd67a8ee94032ae1@mit.edu","name":"Boris Zbarsky"},"message":"Bug 1144743 part 5.  Set the hasPollutedGlobalScope flag correctly when cloning functions.  r=luke","distinct":false,"url":"https://api.github.com/repos/dougt/gecko-dev/commits/9864e3519dbf4d521d735e31700bd5d205c3b71d"}]}