4a1434cdcae32760d8fef0236fa92545a5c4cc65: Bug 1409446 - Work around the dual-ancestor case that WR doesn't handle yet. r=mstange
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 15:45:59 -0400 - rev 685814
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1409446 - Work around the dual-ancestor case that WR doesn't handle yet. r=mstange Bug 1409442 is tracking a change that will allow scroll layers to have multiple ancestors. Without that, there are cases we cannot properly handle, and so we need to ignore a clip in those scenarios. This patch makes sure we do that instead of crashing. MozReview-Commit-ID: 7AU4uyzT6if
40c84480b4d4ce3d806d4d27d705b5fae02eb876: Bug 1409446 - Deal with scenario of two interchangeable DisplayItemClipChain objects causing a cache miss. r=mstange
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 15:45:59 -0400 - rev 685813
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1409446 - Deal with scenario of two interchangeable DisplayItemClipChain objects causing a cache miss. r=mstange MozReview-Commit-ID: 4FQvOmMUUKH
15f491515ba6af017ee3e975c74b13a64af2b2a2: Bug 1409446 - Properly handle out-of-band clips in the new ScrollingLayersHelper code. r=mstange
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 15:45:59 -0400 - rev 685812
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1409446 - Properly handle out-of-band clips in the new ScrollingLayersHelper code. r=mstange When display items (such as mask items) push an out-of-band clip, we can't use clip ids from, or update clip ids into, the cache. We also need to ensure we take these out-of-band clips into account when determining the parent for a new clip we are going to define. MozReview-Commit-ID: GcUI2Hf6SLB
75e3a4777a7f7aeba2cea8bae4ff64c67712f578: Bug 1409446 - Modify the extra-clip flag to instead track more useful information. r=ethlin,mstange
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 15:45:59 -0400 - rev 685811
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1409446 - Modify the extra-clip flag to instead track more useful information. r=ethlin,mstange Instead of just keeping a count of how many "extra clips" (aka out-of-band clips) we have pushed, track more complex information for each clip. In particular, track the display item's normal clip chain, as well as the clip id of the extra clip that was pushed. This will be needed to override clip cache information in the next patch. MozReview-Commit-ID: AWKDTkelhyL
408acda8e5e5ed94cdda9284a37802b723cc8172: Bug 1409446 - Write the new recursive method to define clip chains. r=mstange
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 15:45:58 -0400 - rev 685810
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1409446 - Write the new recursive method to define clip chains. r=mstange This code is more straightforward in its recursion than the old code, and provides a relatively clean way to explicitly pass the desired parent when defining a new clip or scroll layer. Refer to the documentation in the patch for more details. Note that this patch provides the basic recursive algorithm to define the clips and scroll layers, although it omits some of the complicating edge cases which will be added in later patches. The new code is not invoked from anywhere until all the edge case handling has been done. MozReview-Commit-ID: 7z51Kd7LlPU
b1465f5c3f350ba6bef1cf839dff39d990c670df: Bug 1409446 - Unify the clip stack in WebRenderAPI. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 15:45:58 -0400 - rev 685809
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1409446 - Unify the clip stack in WebRenderAPI. r=jrmuizel By using a variant we can keep a single stack of both clips and scrollframes, rather than two separate stacks. This is important because we will want to know how the things are interleaved (e.g. if the last thing that was pushed was a clip or a scrollframe). MozReview-Commit-ID: DbhDj2tTq64
4ae5fc22aecf326e49b0bcdd3068a4a690e2a27f: Bug 1409446 - Remove unused function. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 15:45:58 -0400 - rev 685808
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1409446 - Remove unused function. r=jrmuizel We don't use the code to track the parents of each scroll id, so we can dump it and just keep a set of scroll ids that we've defined to avoid redefining them. MozReview-Commit-ID: HY8y7xt9AJ6
793acbd73126d1ce000657503ecb32c37f1c9e5b: Bug 1409446 - Wire up the DefineClip and DefineScrollLayer APIs to allow specifying ancestry. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 24 Oct 2017 15:45:57 -0400 - rev 685807
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1409446 - Wire up the DefineClip and DefineScrollLayer APIs to allow specifying ancestry. r=jrmuizel The APIs now allow providing the parent clip or scroll info explicitly instead of having to push it on the stack. For now we just pass Nothing() to preserve the existing behaviour, so this change is a functinoal no-op. MozReview-Commit-ID: dtNamN595
cb757b237396115d23516005b66c71427d227d43: Bug 1411132 - Change layout.display-list.retain UpdatePolicy to Once r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Tue, 24 Oct 2017 17:42:18 +0200 - rev 685806
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1411132 - Change layout.display-list.retain UpdatePolicy to Once r=mattwoodrow MozReview-Commit-ID: VJPTFEfKkn
6d6d75c9f64f65ecf8c71983a29f036d477c6ed6: servo: Merge #19003 - Make test-stylo compile an empty crate on Windows without Gecko (from servo:stylo_tests_windows); r=emilio
Simon Sapin <simon.sapin@exyr.org> - Tue, 24 Oct 2017 13:32:39 -0500 - rev 685805
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
servo: Merge #19003 - Make test-stylo compile an empty crate on Windows without Gecko (from servo:stylo_tests_windows); r=emilio That is, in cases where it would fail to link. This will help make Rust CI be gated on compiling Stylo: https://github.com/rust-lang/rust/pull/44603 Source-Repo: https://github.com/servo/servo Source-Revision: 38fe9533b93e985657f99a29772bf3d3c8694822
2f41083748aec88ddef355a0de1e7b5db1da26df: Backed out changeset dab1c5badddb (bug 1407679) for failing server startup in mochitests on Android. r=backout on a CLOSED TREE
Sebastian Hengst <archaeopteryx@coole-files.de> - Tue, 24 Oct 2017 21:18:26 +0200 - rev 685804
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Backed out changeset dab1c5badddb (bug 1407679) for failing server startup in mochitests on Android. r=backout on a CLOSED TREE
127d2762b087b5e5d5b6c32683e91fe483963302: Bug 1408811 - Update test expectation for this bug. r=emilio
Xidorn Quan <me@upsuper.org> - Mon, 23 Oct 2017 15:21:46 +1100 - rev 685803
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1408811 - Update test expectation for this bug. r=emilio on a CLOSED TREE MozReview-Commit-ID: H4glCWmL2vc
b3dcf61084abe620b3f29df70055fdeeeb36f0d0: servo: Merge #19002 - style: Move background-repeat and mask-repeat outside of mako (from emilio:kill-more-mako); r=jdm
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 24 Oct 2017 12:02:05 -0500 - rev 685802
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
servo: Merge #19002 - style: Move background-repeat and mask-repeat outside of mako (from emilio:kill-more-mako); r=jdm style: Move background-repeat and mask-repeat outside of mako. Source-Repo: https://github.com/servo/servo Source-Revision: 347176df257978bdb1c413f632c9fc870de4ff76
dab1c5badddbf3dafd34b52700c3e1129f76d632: Bug 1407679 - Merge nsIIOService and nsIIOService2 r=nwgh
Valentin Gosu <valentin.gosu@gmail.com> - Fri, 20 Oct 2017 16:41:07 +0200 - rev 685801
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1407679 - Merge nsIIOService and nsIIOService2 r=nwgh Removes deprecated nsIIOService2.newChannelFromURIWithProxyFlags MozReview-Commit-ID: Gr3nA25kcre
61e69d3a6cf72e2750211a156a6168b7f9149ba1: No bug - Revendor rust dependencies
Servo VCS Sync <servo-vcs-sync@mozilla.com> - Tue, 24 Oct 2017 17:55:24 +0000 - rev 685800
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
No bug - Revendor rust dependencies
a172387463ec5fa13dd814dfd11a5e90e1ac84c6: Bug 1403213 - Update another reference to nsstring; r=mystor
Gregory Szorc <gps@mozilla.com> - Tue, 24 Oct 2017 10:52:55 -0700 - rev 685799
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1403213 - Update another reference to nsstring; r=mystor MozReview-Commit-ID: 4iVvbKi0LlL
e07ca12a3b94509be582df83e4ecdc36c13bba4d: Bug 1403213 - Move nsstring into servo/support/gecko/nsstring, r=froydnj
Nika Layzell <nika@thelayzells.com> - Mon, 23 Oct 2017 15:37:25 -0400 - rev 685798
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1403213 - Move nsstring into servo/support/gecko/nsstring, r=froydnj MozReview-Commit-ID: 8ucStGkxmj7
d79653ad062ba1735c0506fa4f2df1e2021b3599: Bug 1411323 - Use 'pop-up' instead of 'popup' in popup permission strings in the site identity section. r=flod
Prathiksha <prathikshaprasadsuman@gmail.com> - Tue, 24 Oct 2017 22:27:54 +0530 - rev 685797
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
Bug 1411323 - Use 'pop-up' instead of 'popup' in popup permission strings in the site identity section. r=flod MozReview-Commit-ID: 9BKtOhG94Az
03a03dbbfcdd0d60039244f02051f622093c51ea: servo: Merge #18951 - Wait as late as possible to assign ClipIds (from mrobinson:later-id-assignment-for-clip-scroll-nodes); r=glennw,emilio
Martin Robinson <mrobinson@igalia.com> - Tue, 24 Oct 2017 09:46:08 -0500 - rev 685796
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
servo: Merge #18951 - Wait as late as possible to assign ClipIds (from mrobinson:later-id-assignment-for-clip-scroll-nodes); r=glennw,emilio <!-- Please describe your changes on the following line: --> --- <!-- 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 - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [x] These changes do not require tests because they should not change behavior. <!-- 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: f26aa3b016beb5717d8a53477c162dd832b24d7e
245e58c05b97a1add2249758701f4d6a8d7ffc1c: servo: Merge #19001 - style: Minimal ParserContext cleanup (from emilio:cleanup-parser-context); r=upsuper
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 24 Oct 2017 05:31:07 -0500 - rev 685795
Push 86016 by kgupta@mozilla.com at Wed, 25 Oct 2017 01:53:44 +0000
servo: Merge #19001 - style: Minimal ParserContext cleanup (from emilio:cleanup-parser-context); r=upsuper Source-Repo: https://github.com/servo/servo Source-Revision: 2536e1ee9866f201680942a4641bc019d4cfb86c
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip