af30bdde5572b84febc0374bc3f5acb2c0c3bd67: Bug 1348981 - only load system add-ons from a built-in list r=kmag
Robert Helmer <rhelmer@mozilla.com> - Thu, 14 Sep 2017 00:36:04 -0700 - rev 669396
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1348981 - only load system add-ons from a built-in list r=kmag MozReview-Commit-ID: A6c5kaLmNPP
dc6f6f59da824ecfd79858b79d85b89a217916da: Bug 1348981 - provide method to re-initialize URLPreloader for xpcshell tests r=kmag
Robert Helmer <rhelmer@mozilla.com> - Thu, 21 Sep 2017 11:24:14 -0700 - rev 669395
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1348981 - provide method to re-initialize URLPreloader for xpcshell tests r=kmag MozReview-Commit-ID: rnjEJc1Uir
b5149335d9edbac1e3f57994a61727278c447009: Bug 1348981 - add Cu.readURI helper for sync file reading on startup r=kmag
Robert Helmer <rhelmer@mozilla.com> - Tue, 19 Sep 2017 12:13:08 -0700 - rev 669394
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1348981 - add Cu.readURI helper for sync file reading on startup r=kmag MozReview-Commit-ID: 1yz25afCUDs
a943c88d0dd567c3f1d043d48a8ea6533de0f722: Bug 1348981 - provide a list of valid system add-ons for packaging r=glandium
Robert Helmer <rhelmer@mozilla.com> - Wed, 13 Sep 2017 22:49:43 -0700 - rev 669393
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1348981 - provide a list of valid system add-ons for packaging r=glandium MozReview-Commit-ID: K2jCHBBJBqw
115aa813430bdcb24dbf7c4a9db5e774bbda4b01: Bug 1402419 - Add crashtest ; r=xidorn
Manish Goregaokar <manishearth@gmail.com> - Fri, 22 Sep 2017 13:47:40 -0700 - rev 669392
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1402419 - Add crashtest ; r=xidorn MozReview-Commit-ID: 9Kuxn9edajM
acd13a6b018c272489ca15d95d429a89fca1dfe7: Bug 1402419 - Update dtoa-short ; r=xidorn
Manish Goregaokar <manishearth@gmail.com> - Fri, 22 Sep 2017 13:47:28 -0700 - rev 669391
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1402419 - Update dtoa-short ; r=xidorn MozReview-Commit-ID: qJSJZlZEKO
9d5c9ff64cfb4ab3838166141c0ab46a0fd32051: servo: Merge #18601 - Remove unused 'untagged_unions' feature gate (from mbrubeck:features); r=KiChjang
Matt Brubeck <mbrubeck@limpet.net> - Fri, 22 Sep 2017 14:39:25 -0500 - rev 669390
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
servo: Merge #18601 - Remove unused 'untagged_unions' feature gate (from mbrubeck:features); r=KiChjang CC #5286 - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes do not require tests because _____ Source-Repo: https://github.com/servo/servo Source-Revision: 352b129dc5b237757ade67448cc42f327780417e
3c347dbecdcb3b5bb440a70ade0eaa84f5d72475: servo: Merge #18514 - Remove sources of panic when laying out an iframe without a nested browsing context (from asajeffrey:layout-dont-panic-if-no-iframe-bc); r=emilio
Alan Jeffrey <ajeffrey@mozilla.com> - Fri, 22 Sep 2017 09:34:12 -0500 - rev 669389
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
servo: Merge #18514 - Remove sources of panic when laying out an iframe without a nested browsing context (from asajeffrey:layout-dont-panic-if-no-iframe-bc); r=emilio <!-- Please describe your changes on the following line: --> At the moment, layout panics if it discovers an iframe without a nested browsing context. Under normal circumstances, this is reasonable, but it requires very tight synchronization between script, layout, the constellation and the compositor. In particular, if a layout is in progress when an iframe's browsing context is discarded, this can trigger panic. This PR fixes this in two ways: 1. Making the pipeline and browsing context ids optional in layout's representation of an iframe. 2. Shutting down layout before discarding a browsing context, rather than after. This is belt and braces. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #17482 and #18477 - [X] These changes do not require tests because the PR is fixing a panic caused by a race condition <!-- 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: c48ef50b7260df7f2e781e15bba37b08f0970062
7f3cc80412c6d80d3d789e44dd6ccb8a4faa81c9: Bug 1393366 - Increase the marionette connect timeout in wptrunner, r=maja_zf
James Graham <james@hoppipolla.co.uk> - Thu, 21 Sep 2017 17:56:43 +0100 - rev 669388
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1393366 - Increase the marionette connect timeout in wptrunner, r=maja_zf This switches to using a timeout of 120s * timeout_multplier, which is twice the current value. The Marionette startup_timout value is removed because it turns out that's only used when marionette starts the Firefox instance and therefore is ignored in this case. MozReview-Commit-ID: 4VA8yB6M1d5
4324d70a4fc5fbf06684d69c1efb526e790621c4: Bug 1396951 - 6. Remove setLayerView/getLayerView; r=snorp
Jim Chen <nchen@mozilla.com> - Fri, 22 Sep 2017 14:35:23 -0400 - rev 669387
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1396951 - 6. Remove setLayerView/getLayerView; r=snorp Remove `GeckoAppShell.setLayerView()/getLayerView()` now that it's no longer used anywhere. MozReview-Commit-ID: 6URNFhSs01P
4a111afcd561d3772bb08b9ed1d8268974ae0f3f: Bug 1396951 - 5. Remove getLayerView usage from Robocop code; r=gbrown
Jim Chen <nchen@mozilla.com> - Fri, 22 Sep 2017 14:35:23 -0400 - rev 669386
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1396951 - 5. Remove getLayerView usage from Robocop code; r=gbrown Find the Fennec LayerView through `Solo.getView()` and the View id instead of going through `GeckoAppShell.getLayerView()`. MozReview-Commit-ID: FVcPM0fYorf
1dd2fccf441f557e909752f957b408180fe7e998: Bug 1396951 - 4. Pass in GeckoView instance when sending a11y event; r=snorp
Jim Chen <nchen@mozilla.com> - Fri, 22 Sep 2017 14:35:23 -0400 - rev 669385
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1396951 - 4. Pass in GeckoView instance when sending a11y event; r=snorp Pass in a `GeckoView` instance when sending a11y events so we're not dependent on `GeckoAppShell.getLayerView()`. However, there's likely more work to be done to make a11y work for any GeckoView. MozReview-Commit-ID: DBeDOX5c3qY
4791f0377e29e97d0b129265b3765f9471c051fe: Bug 1396951 - 3. Move GeckoAppShell.viewSizeChanged() to GeckoLayerClient; r=rbarker
Jim Chen <nchen@mozilla.com> - Fri, 22 Sep 2017 14:35:23 -0400 - rev 669384
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1396951 - 3. Move GeckoAppShell.viewSizeChanged() to GeckoLayerClient; r=rbarker Let `GeckoLayerClient` directly deal with scroll-to-input on resize, instead of going through `GeckoAppShell` and relying on `getLayerView()`. This is a necessary fix to let us remove `getLayerView()`, and in a follow-up bug we should actually fix scroll-to-input to work on any GeckoView. MozReview-Commit-ID: 1xsHh2vg08M
a39d4bbc830671bf44aab3f8294a4357c2e5657a: Bug 1396951 - 2. Don't use getLayerView() in GeckoInputConnection; r=esawin
Jim Chen <nchen@mozilla.com> - Fri, 22 Sep 2017 14:35:22 -0400 - rev 669383
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1396951 - 2. Don't use getLayerView() in GeckoInputConnection; r=esawin In GeckoInputConnection, use the current view available through `getView()`, instead of using `GeckoAppShell.getLayerView()`. MozReview-Commit-ID: Hc9AUz5SNEs
fa3c8eefecef805281123048577dc87e3c9e7181: Bug 1396951 - 1. Add and use HapticFeedbackDelegate; r=snorp
Jim Chen <nchen@mozilla.com> - Fri, 22 Sep 2017 14:35:22 -0400 - rev 669382
Push 81322 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 02:02:53 +0000
Bug 1396951 - 1. Add and use HapticFeedbackDelegate; r=snorp Instead of using `getLayerView()` to perform haptic feedback, this patch adds a `HapticFeedbackDelegate`, which `GeckoApplication` implements to call `performHapticFeedback()` on the active view. Also, use HapticFeedbackDelegate elsewhere in the Fennec codebase where we want to perform haptic feedback. MozReview-Commit-ID: GAArA6yJFNF
7337b93c4f62547df6b8733cc9531b9f10f3d5ce: Bug 1401179 - Add snackbar for bookmark add/remove actions on newtab. r?mcomella draft
Chenxia Liu <liuche@mozilla.com> - Fri, 22 Sep 2017 18:08:07 -0700 - rev 669381
Push 81321 by cliu@mozilla.com at Sat, 23 Sep 2017 02:01:51 +0000
Bug 1401179 - Add snackbar for bookmark add/remove actions on newtab. r?mcomella MozReview-Commit-ID: 7DzoyDSWmVq
7423880b08169ffb565fbf1d753c9e0d40dd527f: Bug 1402472: Make sure to not run any style invalidation in an unstyled children only traversal. r?bholley draft
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 23 Sep 2017 03:34:39 +0200 - rev 669380
Push 81320 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 01:35:53 +0000
Bug 1402472: Make sure to not run any style invalidation in an unstyled children only traversal. r?bholley MozReview-Commit-ID: IFPA7LJpvsZ
488786e883d40293cf23bafbe472924eddb88bde: Bug 1400936: Crashtests. r?bholley draft
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 23 Sep 2017 00:12:33 +0200 - rev 669379
Push 81320 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 01:35:53 +0000
Bug 1400936: Crashtests. r?bholley MozReview-Commit-ID: 6Q07StWSog
2ff9937afa1eb07f617122a77dbe0b05d46e0c8c: Bug 1400936: Clear servo data after children data is cleared, and allow setting the root as the document if the tree is mid-unbind. r?bholley draft
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 23 Sep 2017 00:05:47 +0200 - rev 669378
Push 81320 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 01:35:53 +0000
Bug 1400936: Clear servo data after children data is cleared, and allow setting the root as the document if the tree is mid-unbind. r?bholley This is the actual fix, and makes sure that the state is consistent even if we notify of state changes on a parent during unbind. We potentially do a bit more work than needed given we set the document as the root in that case instead of the parent which could potentially be the root itself, but that's not a huge deal I think, given these cases are rare. If this happens to be a perf problem, we may want to just drop the root during UnbindFromTree if aNullParent == true and the root is a descendant of `this`. MozReview-Commit-ID: A9d2igM0hMr
f7f18be0e9d91f4cc59dc36f7213b0db959c1c38: Bug 1400936: Only tear down the servo data in SetXBLInsertionParent if the parent actually changed. r?bholley draft
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 23 Sep 2017 00:02:59 +0200 - rev 669377
Push 81320 by bmo:emilio@crisal.io at Sat, 23 Sep 2017 01:35:53 +0000
Bug 1400936: Only tear down the servo data in SetXBLInsertionParent if the parent actually changed. r?bholley This is the reason similar assertion failures can't be reproduced with elements and stuff like fieldset and form validity changes. nsBindingManager::ContentRemoved calls SetXBLInsertionParent, which clears all the Servo data from the subtree eagerly, which is a waste when the actual binding parent is the same (null). MozReview-Commit-ID: A5wLKfD4OTL
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip