5c918b5507a672cd7f21a5ebb20013de8a46d0fe: Bug 1363321 - Part 2 - Reset crash loop counter after a presumably succesful startup. r=jchen
Jan Henning <jh+bugzilla@buttercookie.de> - Tue, 09 May 2017 23:12:54 +0200 - rev 357652
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1363321 - Part 2 - Reset crash loop counter after a presumably succesful startup. r=jchen At the moment, our crash loop counter that turns off session restoring when multiple crashes happen in a row is reset only after a successful background-foreground cycle. Since its purpose is to break out of startup crash loops caused by the currently loaded tab that is automatically loaded after starting the app, it makes sense to also reset it if we didn't crash soon after starting up. Otherwise, if the user remains within Firefox, another crash long after starting would still count towards the crash loop counter. Desktop already employs a similar logic for deciding whether to automatically enable safe mode or not after a crash. Therefore, we use the same amount of time (30 s) after starting Gecko for resetting the counter. We also take the opportunity to make our logic a bit more conservative and only turn off session restoring after two successive crashes in a row (desktop uses 3 by default). MozReview-Commit-ID: CQDutkDgmfc
5ce60a1ee7c9b65f67fb34428e74166db440b75a: Bug 1363321 - Part 1 - Always copy crash restore settings to Java on startup. r=jchen
Jan Henning <jh+bugzilla@buttercookie.de> - Wed, 10 May 2017 20:20:50 +0200 - rev 357651
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1363321 - Part 1 - Always copy crash restore settings to Java on startup. r=jchen Those settings need to be stored in our shared preferences since they're needed before Gecko is available, however in order to allow them to be easily configured, we also expose them in about:config and sync them to the shared prefs via a change listener. This however means that changes in the *default* value of those prefs won't be picked up. Therefore we now simply unconditionally copy the setting state into the shared preferences on startup and omit the change listener instead, as they're exposed in about:config only and there is in fact no pressing need for changes there to immediately take effect. MozReview-Commit-ID: 7c950F1nkdj
0324db517245c0d311f56e5c0a2e53dacee409ef: Bug 1361843 part 4. Remove the now-unused mBatching member of ServoStyleSet. r=emilio
Boris Zbarsky <bzbarsky@mit.edu> - Wed, 10 May 2017 13:13:50 -0400 - rev 357650
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1361843 part 4. Remove the now-unused mBatching member of ServoStyleSet. r=emilio MozReview-Commit-ID: DvyZkQh6q8A
c64fde685fb178171948e34cb06690ddd562b25d: Bug 1361843 part 3. Clear the stylist when styleshets change, but don't do anything else until there's an explicit sheet flush. r=emilio
Boris Zbarsky <bzbarsky@mit.edu> - Wed, 10 May 2017 13:13:45 -0400 - rev 357649
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1361843 part 3. Clear the stylist when styleshets change, but don't do anything else until there's an explicit sheet flush. r=emilio MozReview-Commit-ID: IhK1ECeuYbM
97b43ea69c41690698b3ab9adc8a7567e3258c3b: Bug 1361843 part 2. Track whether the stylist may need a rebuild in ServoStyleSet and do on-demand rebuilds as needed. r=emilio
Boris Zbarsky <bzbarsky@mit.edu> - Wed, 10 May 2017 13:13:39 -0400 - rev 357648
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1361843 part 2. Track whether the stylist may need a rebuild in ServoStyleSet and do on-demand rebuilds as needed. r=emilio This does not remove the eager rebuilds we're doing yet. I'm not completely happy with the ad-hoc manner in which we end up doing rebuilds. I considered doing something where we'd make stylist a non-public member of PerDocumentStyleDataImpl and have a getter that updates the stylist on get, with maybe a separate non-flushing getter for special situations, if those arise. But that requires that we make various cases where we currently have a non-mut PerDocumentStyleDataImpl use a mut one. Maybe that would be the right tradeoff... I'm also not sure whether the naming is right here. Maybe we should just talk about needing a stylesheet flush, not a stylist rebuild, in ServoStyleSet? MozReview-Commit-ID: 9C7BG5ygm79
7a8eb08d31d735a8403a95f62c5d524e164793a5: Bug 1363902 - Simplify some browser chrome tests by utilizing TestUtils.topicObserved. r=mikedeboer
Dão Gottwald <dao@mozilla.com> - Thu, 11 May 2017 00:24:37 +0200 - rev 357647
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1363902 - Simplify some browser chrome tests by utilizing TestUtils.topicObserved. r=mikedeboer MozReview-Commit-ID: Flc8iE19Gkt
0a6194b15bd6c8a8c844336dfa2e5e3420b88707: Bug 1353360 - implement Photon design spec for panel menus. r=Gijs
Mike de Boer <mdeboer@mozilla.com> - Wed, 10 May 2017 17:29:36 -0400 - rev 357646
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1353360 - implement Photon design spec for panel menus. r=Gijs MozReview-Commit-ID: BPMkiiP31BP
fbe4a19a42e2239a9cf119873509bda67942407f: servo: Merge #16798 - Allow stylo to do lazier stylist rebuilds (from bzbarsky:stylist-lazier-updates); r=emilio
Boris Zbarsky <bzbarsky@mit.edu> - Wed, 10 May 2017 16:22:06 -0500 - rev 357645
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
servo: Merge #16798 - Allow stylo to do lazier stylist rebuilds (from bzbarsky:stylist-lazier-updates); r=emilio --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [ ] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [ ] These changes fix https://bugzilla.mozilla.org/show_bug.cgi?id=1361843 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because it's optimization, which should have no observable behavior difference. <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Source-Repo: https://github.com/servo/servo Source-Revision: c6c0252b19f82ba98ca8a102435067d15f5a1356
2150351429b5c9515589facd9f2aefb1e4640842: Bug 1355343: Take all the snapshots into account. r=bholley
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 07 May 2017 16:36:47 +0200 - rev 357644
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1355343: Take all the snapshots into account. r=bholley I've chosen this approach mainly because there's no other good way to guarantee the model is correct than holding the snapshots alive until a style refresh. What I tried before this (storing them in a sort of "immutable element data") is a pain, since we call into style from the frame constructor and other content notifications, which makes keeping track of which snapshots should be cleared an which shouldn't an insane task. Ideally we'd have a single entry-point for style, but that's not the case right now, and changing that requires pretty non-trivial changes to the frame constructor. MozReview-Commit-ID: FF1KWZv2iBM
eaf4e461619e685ef1eefaf31464ad0f1c133448: Bug 1355343: Move node restyle bits to Element, and add bits for snapshot handling. r=bholley
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 16 Apr 2017 08:16:24 +0200 - rev 357643
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1355343: Move node restyle bits to Element, and add bits for snapshot handling. r=bholley MozReview-Commit-ID: 6OrUKX5RcBq Signed-off-by: Emilio Cobos Álvarez <emilio@crisal.io>
37bea3f592dc00e7ebcf5c016c40bd9d37406b9a: Bug 1341102 - Hopefully fix busted reftest annotations by removing whitespaces. r=me on a CLOSED TREE
Sebastian Hengst <archaeopteryx@coole-files.de> - Thu, 11 May 2017 00:08:45 +0200 - rev 357642
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1341102 - Hopefully fix busted reftest annotations by removing whitespaces. r=me on a CLOSED TREE
d0e5a5ba01b5078071fc195897b083563dc4f918: Bug 1344398 part 3: Add reftests for dynamic changes to "overflow" on html & body elements. r=tnikkel
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 10 May 2017 13:55:37 -0700 - rev 357641
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1344398 part 3: Add reftests for dynamic changes to "overflow" on html & body elements. r=tnikkel MozReview-Commit-ID: 8WP2qDiKpyP
d70f9de401d140408973d260aa9967473e1a2da6: Bug 1344398 part 2: React to some CSS 'overflow' changes on body/html by simply calling UpdateViewportScrollbarStylesOverride() instead of reframing. r=bz,tnikkel
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 10 May 2017 13:53:27 -0700 - rev 357640
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1344398 part 2: React to some CSS 'overflow' changes on body/html by simply calling UpdateViewportScrollbarStylesOverride() instead of reframing. r=bz,tnikkel MozReview-Commit-ID: Lpr9aNv9hAB
647d0bb3714d6adf904e83ad9e46706f529cb934: Bug 1344398 part 1: Move presContext variable a little earlier in a nsCSSFrameConstructor method. r=tnikkel
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 10 May 2017 13:53:25 -0700 - rev 357639
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1344398 part 1: Move presContext variable a little earlier in a nsCSSFrameConstructor method. r=tnikkel Also, assert that this presContext is non-null (which we already implicitly assume, when we dereference it without null-checking it). MozReview-Commit-ID: IIYUqGQTOnb
13a6ff7f76c814b7d05a8e61247ddb48c29beda4: Bug 1363095 - Remove unused PREF_SKIPDEFAULTBROWSERCHECK define. r=jimm
Dão Gottwald <dao@mozilla.com> - Wed, 10 May 2017 22:20:06 +0200 - rev 357638
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1363095 - Remove unused PREF_SKIPDEFAULTBROWSERCHECK define. r=jimm MozReview-Commit-ID: 962MPM09tDw
a9dedd866433f9d5f8979942559cc49e57e2aeb4: Backed out 2 changesets (bug 1362800) for eslint failures a=backout
Wes Kocher <wkocher@mozilla.com> - Wed, 10 May 2017 14:42:49 -0700 - rev 357637
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Backed out 2 changesets (bug 1362800) for eslint failures a=backout Backed out changeset 0492e6f16df1 (bug 1362800) Backed out changeset 785cf0c4b67b (bug 1362800) MozReview-Commit-ID: EY2RYz7lwtI
42168646d2997c974f9ae9e692453d9a0f615c1d: Bug 1341102 - Update reftest expectations; r=manishearth
Manish Goregaokar <manishearth@gmail.com> - Wed, 10 May 2017 14:32:45 -0700 - rev 357636
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1341102 - Update reftest expectations; r=manishearth MozReview-Commit-ID: IJkqsizeaQH
305d9e751c2437fc881c34ece01f62af55e92704: No bug - Revendor rust dependencies
Servo VCS Sync <servo-vcs-sync@mozilla.com> - Wed, 10 May 2017 21:23:02 +0000 - rev 357635
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
No bug - Revendor rust dependencies
f190f2c09ad8028a7d11d58aa7e4195691cdd25e: servo: Merge #16778 - Take all the snapshots into account in the style system (from emilio:snapshots); r=bholley
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 10 May 2017 15:08:59 -0500 - rev 357634
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
servo: Merge #16778 - Take all the snapshots into account in the style system (from emilio:snapshots); r=bholley See [bug 1355343](https://bugzilla.mozilla.org/show_bug.cgi?id=1355343). The servo part of this patch presumably needs some polishing, let's see. Source-Repo: https://github.com/servo/servo Source-Revision: c8171ed5d7d4b715ab30795311724420c95dfa10
0492e6f16df1050ac69ceba7801e52a1d68bad9e: Bug 1362800 - Expose geckoProfiler.getProfileAsArrayBuffer. r=kmag
Markus Stange <mstange@themasta.com> - Wed, 10 May 2017 17:09:51 -0400 - rev 357633
Push 31797 by cbook@mozilla.com at Thu, 11 May 2017 10:44:37 +0000
Bug 1362800 - Expose geckoProfiler.getProfileAsArrayBuffer. r=kmag MozReview-Commit-ID: 7uFPWAhh25L
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip