searching for reviewer(mbrubeck)
f79d9ed6303b3af16d4fb7025ba3c97a3f6e4cc2: servo: Merge #15827 - replace match with if let (from moonlightdrive:use-iflet-not-match); r=mbrubeck
Jyotsna Prakash <jyotsna.prakash@gmail.com> - Sun, 05 Mar 2017 19:26:27 -0800 - rev 375013
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15827 - replace match with if let (from moonlightdrive:use-iflet-not-match); r=mbrubeck <!-- Please describe your changes on the following line: --> Use `if let` instead of a `match` with a single pattern. I'm not clear if/how this needs to be tested (beyond what's been checked-off below), so please let me know if there's more to do. Thanks! --- <!-- 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 #15810 <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- 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: 5ccf79e05570342ab9592fbc573229e38756344b
c2b94c0a2c56f9b66c1ff2eee3208c2da36a2622: servo: Merge #15762 - Add support for page zoom via new WR API (from glennw:basic-page-zoom); r=mbrubeck
Glenn Watson <github@intuitionlibrary.com> - Wed, 01 Mar 2017 11:53:35 -0800 - rev 374485
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15762 - Add support for page zoom via new WR API (from glennw:basic-page-zoom); r=mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 67638bdcf27f9bb899059d1e2f8e907f7462cad6
6559514972f54076f719cc172313f02321c68ed4: servo: Merge #15700 - Introduce CSSPixel as a replacement for ViewportPx and PagePx (from glennw:zoom-wip-2); r=mbrubeck
Glenn Watson <github@intuitionlibrary.com> - Thu, 23 Feb 2017 16:01:16 -0800 - rev 373572
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15700 - Introduce CSSPixel as a replacement for ViewportPx and PagePx (from glennw:zoom-wip-2); r=mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 1d13e6a2df42af3cce427a0b2062ac70d28c05d0
6132f6422a9dd00462cc738348d2364905c8c4c0: servo: Merge #15680 - Rename ScreenPx to DeviceIndependentPixel (from glennw:zoom-wip); r=mbrubeck
Glenn Watson <github@intuitionlibrary.com> - Tue, 21 Feb 2017 20:45:20 -0800 - rev 373156
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15680 - Rename ScreenPx to DeviceIndependentPixel (from glennw:zoom-wip); r=mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 03893e25cc00488dea540ce53865f7f696f8f262
c109d9d687e6ef7ee6ab52e09acc2e3a6c60da08: servo: Merge #15675 - Resync bindings, use StyleShapeSource for clip-path (from Manishearth:resync); r=mbrubeck
Manish Goregaokar <manishsmail@gmail.com> - Tue, 21 Feb 2017 10:05:01 -0800 - rev 373064
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15675 - Resync bindings, use StyleShapeSource for clip-path (from Manishearth:resync); r=mbrubeck The bindings are getting harder to rebase over since the checked-in bindings differ from the generated ones. Also includes https://bugzilla.mozilla.org/show_bug.cgi?id=1339314 Source-Repo: https://github.com/servo/servo Source-Revision: ed731c80d710cbb1433c4c83f0630f8c221985e1
e74262ac9b0cb0214356e02428340d0e4ec01901: servo: Merge #15561 - Stop using DOMRefCell<Heap<..>> (from saneyuki:rm-domrefcell-heap); r=mbrubeck
Tetsuharu OHZEKI <saneyuki.snyk@gmail.com> - Wed, 15 Feb 2017 11:11:40 -0800 - rev 372394
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15561 - Stop using DOMRefCell<Heap<..>> (from saneyuki:rm-domrefcell-heap); r=mbrubeck Fix https://github.com/servo/servo/issues/15560 Source-Repo: https://github.com/servo/servo Source-Revision: 3c8daca772aacf59bffdff32b017f8029437a85e
b9f44b5a9ce19fd7957e6f53c7ef3a677bde7761: servo: Merge #15463 - stylo: Implement a bunch of properties (from Manishearth:buncha-props); r=mbrubeck
Manish Goregaokar <manishsmail@gmail.com> - Thu, 09 Feb 2017 15:51:29 -0800 - rev 370776
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15463 - stylo: Implement a bunch of properties (from Manishearth:buncha-props); r=mbrubeck r? @mbrubeck or @heycam Source-Repo: https://github.com/servo/servo Source-Revision: e985ad54229083e5e76b2862cec57ce4fef4433c
8defda71fb1b8c6729e2c1fa9b8d86c1c41c272c: servo: Merge #15424 - Fix some build warnings (from servo:warnings); r=mbrubeck
Ms2ger <Ms2ger@gmail.com> - Tue, 07 Feb 2017 07:16:22 -0800 - rev 370452
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15424 - Fix some build warnings (from servo:warnings); r=mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: c48a326fb37976ec9e808825c9641602bccefa56
347b3a2e6844f48c5c379630b8f68a9737a7d166: servo: Merge #15347 - Upgrade libssl dependency on Debian (from andreastt:libssl-debian); r=mbrubeck
Andreas Tolfsen <ato@mozilla.com> - Thu, 02 Feb 2017 10:12:57 -0800 - rev 369738
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15347 - Upgrade libssl dependency on Debian (from andreastt:libssl-debian); r=mbrubeck - [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). - [ ] There are tests for these changes OR - [x] These changes do not require tests because _non-functional changes_ Source-Repo: https://github.com/servo/servo Source-Revision: 94e3c900294372fd4bc95e694668b2b07aa1c93a
17434a230223a8b066c660c7e27b98eb8c80373d: servo: Merge #15333 - update mouse position when receiving mouse wheel events (from mbrubeck:update_mouse_position_when_receiving_wheel_events); r=mbrubeck
Gregory Terzian <gterzian@users.noreply.github.com> - Wed, 01 Feb 2017 15:48:30 -0800 - rev 369733
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15333 - update mouse position when receiving mouse wheel events (from mbrubeck:update_mouse_position_when_receiving_wheel_events); r=mbrubeck Rebase of #14808. --- - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #14290 (github issue number if applicable). - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ Source-Repo: https://github.com/servo/servo Source-Revision: 8b9dc9392b5d734763f3f92d183ec179da9a6ff4
3d18a0bffba19b6f3f769efe74b16d495f170b2e: servo: Merge #13506 - A test case change is added to #13442: Fix #12193 Servo displays upper level Thai character in wrong place (from veer66:master); r=mbrubeck
Vee Satayamas <vsatayamas@gmail.com> - Wed, 01 Feb 2017 12:08:44 -0800 - rev 369732
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13506 - A test case change is added to #13442: Fix #12193 Servo displays upper level Thai character in wrong place (from veer66:master); r=mbrubeck A test case change is added to #13442: Fix #12193 Servo displays upper level Thai character in wrong place. <!-- 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: --> - [ ] `./mach build -d` does not report any errors - [ ] `./mach test-tidy` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- 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: d161f6ec44ccaf1fa5506e62d6de47306f06a30d
dc55cc11fe9258644d45105fcca1c3d578efe08e: servo: Merge #15186 - Re revert the backed out PRs (from servo:re-revert); r=mbrubeck
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 25 Jan 2017 02:36:15 -0800 - rev 369676
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #15186 - Re revert the backed out PRs (from servo:re-revert); r=mbrubeck r? anyone Source-Repo: https://github.com/servo/servo Source-Revision: 2b623fcc7a4319b217cd52d1049b228a9fcc165a
7ba77a2374a35c20d65c2686e704c0a5ea8aee7b: servo: Merge #13681 - Fix margin size calculation for TableWrapper (from gpoesia:master); r=mbrubeck
Gabriel Poesia <gabriel.poesia@gmail.com> - Mon, 23 Jan 2017 08:55:44 -0800 - rev 369661
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13681 - Fix margin size calculation for TableWrapper (from gpoesia:master); r=mbrubeck <!-- Please describe your changes on the following line: --> Fixes inline size calculation for TableWrapper. The table's width was always reaching the inline size equation solver as a specified variable, which was causing the system to be overdetermined when there was a margin specified for the table, and this caused the overflow reported in #12748. The fix consists in handling three cases when the table's width is not specified: if the preferred size of all columns fits, it is returned; if the minimum size does not fit, it is returned instead (it will overflow), otherwise, it is returned as a free variable (that should be solved together with the margin to some value above the minimum width and below the preferred width). --- - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12748 - [X] There are tests for these changes Source-Repo: https://github.com/servo/servo Source-Revision: b0f91193fd4fbb86fc75fc593a6fdc40dc95bfcd
dfeecb83e320eee6ce86710fd9c4b77de447d350: servo: Merge #14995 - Enable codegen-units=4 in debug mode (from servo:codegen-units); r=mbrubeck
Simon Sapin <simon.sapin@exyr.org> - Sun, 15 Jan 2017 13:23:55 -0800 - rev 369600
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14995 - Enable codegen-units=4 in debug mode (from servo:codegen-units); r=mbrubeck We previously had this and removed it because it caused runtime slowdowns (discussion in #11102), but that was in release mode. Debug mode is already very slow at runtime. In my opinion, debug mode is mostly for the edit-compile-test cycle which is currently dominated by compile times, so sacrificing runtime for compile time makes sense. I’ve observed a ~30% improvement of compile times with this. This figure is consistent between a very fast desktop with 4 cores / 8 threads and a not-as-fast laptop with 2 cores / 4 threads. <!-- 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 - [ ] These changes do not require tests because _____ <!-- 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: ad1b11771b93ced67b409436c7bf0f493ec4663a
765e9e22d9c2248c490413fb85605eeda659f4d9: servo: Merge #14929 - Update image (from servo:update-image); r=mbrubeck
Ms2ger <Ms2ger@gmail.com> - Wed, 11 Jan 2017 05:37:45 -0800 - rev 369564
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14929 - Update image (from servo:update-image); r=mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 12c71c57d178ee5bfd383063ddebb4763006eaf4
4308461871cdc0a98b57761d749dc8c1118f08a3: servo: Merge #14801 - style: Document and force documentation in a big chunk of the style crate (from emilio:no-missing-docs); r=mbrubeck,Manishearth,Wafflespeanut
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 31 Dec 2016 03:19:02 -0800 - rev 369480
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14801 - style: Document and force documentation in a big chunk of the style crate (from emilio:no-missing-docs); r=mbrubeck,Manishearth,Wafflespeanut Style no forced docs for the properties code and similar, but I ran out of time, and I think it's a nice improvement. I'd appreciate a fast-ish turn-around time because this is pretty much prone to bitrot. Source-Repo: https://github.com/servo/servo Source-Revision: bd67163438317daa711b2411ce302aaf5bf4136a
53892ebfac673c13a77ebed39bef9780cdd1ff1e: servo: Merge #14671 - Remove the unused dependency of style to plugins (from servo:style-no-plugins); r=mbrubeck
Simon Sapin <simon.sapin@exyr.org> - Thu, 22 Dec 2016 12:53:33 -0800 - rev 369425
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14671 - Remove the unused dependency of style to plugins (from servo:style-no-plugins); r=mbrubeck <!-- 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 - [ ] These changes do not require tests because _____ <!-- 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: 40527441395622dff6b970d6ff1f9f6503f54567
f8a98d6fefe189332142a2420c6f93cf7c88b06d: servo: Merge #14597 - FIX for Implement scroll transactions (from gterzian:fix_scroll_transactions); r=mbrubeck
Gregory <gterzian@users.noreply.github.com> - Wed, 14 Dec 2016 22:58:55 -0800 - rev 369374
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14597 - FIX for Implement scroll transactions (from gterzian:fix_scroll_transactions); r=mbrubeck <!-- Please describe your changes on the following line: --> Follow up on https://github.com/servo/servo/pull/14470 @mbrubeck @KiChjang @glennw I just found out in the context of https://github.com/servo/webrender/pull/600 I forgot to add a case for the very first scroll event, or else the scrolling on that PR, in a non-Mac OS environment, will only start after an 80ms pause following the initial scroll event... Sorry this slipped through my initial testing... --- <!-- 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 - [ ] These changes do not require tests because _____ <!-- 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: 139c111091210a5002bda668d14debb2e1c68ca9
0dfe7146374f72f7f5b5e2c99ff6df6984a81353: servo: Merge #14592 - Remove the util crate (from asajeffrey:util-goodbye); r=mbrubeck
Alan Jeffrey <ajeffrey@mozilla.com> - Wed, 14 Dec 2016 16:48:42 -0800 - rev 369368
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14592 - Remove the util crate (from asajeffrey:util-goodbye); r=mbrubeck <!-- Please describe your changes on the following line: --> This PR removes the `util` crate. * Replaced the `spawn_named` and `clamp` functions by appropriate uses of `std::thread::Builder::spawn`, `std::cmp::min` and `std::cmp::max`. * Moved `opts`, `prefs` and `resource_files` into a new `config` crate. * Moved `remutex` and `geometry` into their own crates. --- <!-- 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 do not require tests because they are refactorings <!-- 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: 4eb653817f87e5fb47de34356f558eb76ecbca9f
9f5a4f88bf77c4e89885341edf2d869ff74a5540: servo: Merge #14539 - Show the placeholder text for textarea elements (from frewsxcv:textarea-placeholder); r=mbrubeck
Corey Farwell <coreyf@rwell.org> - Wed, 14 Dec 2016 12:13:06 -0800 - rev 369365
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14539 - Show the placeholder text for textarea elements (from frewsxcv:textarea-placeholder); r=mbrubeck Fixes https://github.com/servo/servo/issues/10552. All this logic was taken from htmlinputelement.rs. Source-Repo: https://github.com/servo/servo Source-Revision: 816b2969c34a43c51365ea2b49d90c08d434c9ea
5e6f51510f8927cd014e93a7c9526da8465612cb: servo: Merge #14470 - Implement scroll transactions (from gterzian:implement_scroll_transactions); r=mbrubeck
Gregory <gterzian@users.noreply.github.com> - Mon, 12 Dec 2016 16:27:34 -0800 - rev 369347
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14470 - Implement scroll transactions (from gterzian:implement_scroll_transactions); r=mbrubeck <!-- Please describe your changes on the following line: --> @glennw Here is a first pass at faking Start scroll events by way of 'transactions', as suggested by @mstange at https://github.com/servo/webrender/pull/599#issuecomment-263323105 Since I still don't have a Linux environment available for testing(and my Mac doesn't have enough resources to run a VM at the moment), I tested this with both https://github.com/servo/webrender/pull/599 and https://github.com/servo/webrender/pull/600 on a Mac by: * disabling start and end events by removing the content of these two functions: https://github.com/servo/servo/blob/master/components/compositing/compositor.rs#L1080 and https://github.com/servo/servo/blob/master/components/compositing/compositor.rs#L1093 * Setting `CAN_OVERSCROLL` to false for Mac OS in Webrender https://github.com/servo/webrender/blob/master/webrender/src/frame.rs#L29 * This PR also requires a `./mach update-cargo -a` The desired behavior of both Webrender PR's, based on my manual testing, now also works when there are no end or start scroll events provided by the os. The scroll transactions do not affect normal scrolling on Mac OS, and both PR still work as before on that platform. Both PR in Webrender need some re-basing and cleaning up, as does this one, and I first wanted to put this proposal forward, and also ask if someone has the time to do some testing in a real Linux environment... --- <!-- 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 #13249 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- 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: 92d380c399204d327e74771f5a9b8b2a343acecc
daaa89ca8da2cf8fa5142c69ba50bcf12b306204: servo: Merge #14535 - Introduce a PropertyId enum and use it instead of strings of property names (from servo:property-id); r=mbrubeck
Simon Sapin <simon.sapin@exyr.org> - Sat, 10 Dec 2016 01:16:26 -0800 - rev 369338
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14535 - Introduce a PropertyId enum and use it instead of strings of property names (from servo:property-id); r=mbrubeck <!-- Please describe your changes on the following line: --> * `LonghandId` and `ShorthandId` are C-like enums * `Atom` is used for the name of custom properties. * `PropertyDeclarationId` is the identifier for `PropertyDeclaration`, after parsing and shorthand expansion. (Longhand or custom property.) * `PropertyId` represents any CSS property, e.g. in CSSOM. (Longhand, shorthand, or custom.) CC @upsuper --- <!-- 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 - [ ] These changes do not require tests because _____ <!-- 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: 6dd4b4822fa788694153ee61a04dd9a5dfb748ec
09b489d087dcefcbb919270fda6a4f7876beb042: servo: Merge #14413 - Remove misleading section from non-toplevel Cargo.toml (from servo:jdm-patch-1); r=mbrubeck
Josh Matthews <josh@joshmatthews.net> - Wed, 30 Nov 2016 18:19:08 -0800 - rev 369281
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14413 - Remove misleading section from non-toplevel Cargo.toml (from servo:jdm-patch-1); r=mbrubeck This made me waste some time trying to figure out why my builds were no longer getting debug information after #14381. Source-Repo: https://github.com/servo/servo Source-Revision: 35761782394b29fda0fc67f0fd1884d8e2654f0d
7ec63d91ff9e4fa8d56b2eb03344fe7dd647d084: servo: Merge #14426 - Use intermittents tracker on build.s.o (from Manishearth:intermittents); r=mbrubeck
Manish Goregaokar <manishsmail@gmail.com> - Wed, 30 Nov 2016 14:50:12 -0800 - rev 369278
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14426 - Use intermittents tracker on build.s.o (from Manishearth:intermittents); r=mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 52e2b025d2fc0a52f33a98e1fa7f90d334058a89
39c3620d3f9881b4d4dbbcb8f863c5a008da17c0: servo: Merge #14119 - Address common source of shutdown panics (from servo:jdm-patch-1); r=mbrubeck
Josh Matthews <josh@joshmatthews.net> - Tue, 08 Nov 2016 12:23:56 -0600 - rev 369129
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14119 - Address common source of shutdown panics (from servo:jdm-patch-1); r=mbrubeck Remove easy sources of intermittent failures during test shutdown. Might help #14118 and #14117. Source-Repo: https://github.com/servo/servo Source-Revision: 5173a655219d5eff64d4702718cfa63c2c44e1b3
614acc40ac6df4b83b7a1b9f75c8d16731588c01: servo: Merge #14048 - layout/context: Wrap `image_cache_thread` in a `Mutex<>` as well (from antrik:nosync-ipc_sender); r=mbrubeck
Olaf Buddenhagen <antrik@users.sf.net> - Thu, 03 Nov 2016 21:37:17 -0500 - rev 369084
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #14048 - layout/context: Wrap `image_cache_thread` in a `Mutex<>` as well (from antrik:nosync-ipc_sender); r=mbrubeck `SharedLayoutContext.image_cache_thread` didn't get the `Mutex<>` treatment along with all the other channels in there, because `ipc-channel::Sender` is presently inherently `Sync`. I believe this to be an implementation accident though, that should be rectified in the future -- not something users should actually rely on... Note that sharing senders (be it `mpsc` or `ipc-channel`) in is probably not a good idea anyway: just cloning them -- and letting them handle the sharing internally -- should be both simpler and cheaper. But right now that's how things are handled here; so let's go with the flow... Source-Repo: https://github.com/servo/servo Source-Revision: 7b803860470029a6deadb4d50fc401ddce968f6c
b19b2b0f0253f7d7aef5a3616681baed4e7bda9f: servo: Merge #13829 - Layout viewer fixes (from shinglyu:layout_viewer); r=mbrubeck
Shing Lyu <shing.lyu@gmail.com> - Wed, 02 Nov 2016 18:34:26 -0500 - rev 369071
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13829 - Layout viewer fixes (from shinglyu:layout_viewer); r=mbrubeck <!-- Please describe your changes on the following line: --> This is a followup for #13432. This patch should be landed after #13740 to accommodate the change in JSON format in #13740. It also included some UI layout fixes to make the UI more readable. I'm splitting the commits for easier review, I'll squash them after a r+. --- <!-- 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 it's just a external debug tool <!-- 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: 524ed827007dee1563aed25f3bc3d0edff77ea78
600f956c56fd0b0149ad5c3f8c366d9bd7abbd33: servo: Merge #13870 - layout: Rewrite anonymous table code, simplify and fix table intrinsic width calculation, and improve safety of flexbox code (from pcwalton:anonymous-table-rewrite); r=mbrubeck
Patrick Walton <pcwalton@mimiga.net> - Wed, 26 Oct 2016 20:14:04 -0500 - rev 369021
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13870 - layout: Rewrite anonymous table code, simplify and fix table intrinsic width calculation, and improve safety of flexbox code (from pcwalton:anonymous-table-rewrite); r=mbrubeck Closes #13782. Source-Repo: https://github.com/servo/servo Source-Revision: 6b40f97289c4c5a20b539992c3d511b3bf470f28
0aabb9842849490bc546828cd8d7d9810caaa1a0: servo: Merge #13839 - Add RwLock in more Arc’d things in stylesheets (from servo:locked-style); r=mbrubeck
Simon Sapin <simon.sapin@exyr.org> - Thu, 20 Oct 2016 11:29:27 -0500 - rev 368983
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13839 - Add RwLock in more Arc’d things in stylesheets (from servo:locked-style); r=mbrubeck <!-- Please describe your changes on the following line: --> CSSOM needs hold potentially multiple references (through `Arc`) to these objects, and then mutate them. CC @upsuper r? @mbrubeck --- <!-- 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 - [ ] These changes do not require tests because _____ <!-- 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: 4e5ad268b15aa222c0d049796dde7c6e7bc0897b
3a43d0ab7fc851bfc7f9babaa7734aad5121fdef: servo: Merge #13432 - Restore the layout trace viewer (from shinglyu:layout_viewer_clean); r=mbrubeck
Shing Lyu <shing.lyu@gmail.com> - Mon, 17 Oct 2016 10:06:50 -0500 - rev 368962
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13432 - Restore the layout trace viewer (from shinglyu:layout_viewer_clean); r=mbrubeck <!-- Please describe your changes on the following line: --> This patch restores the layout trace viewer removed in https://github.com/servo/servo/commit/e7510ab90c3482233ca8b188d748216901041970 , with some changes that makes it work with https://github.com/servo/servo/pull/13214 Sorry for the minified 3rd party CSS frameworks, it was from the original version and I haven't got time to replace them with hand-crafted CSS. --- <!-- 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 is a stand-alone debugging tool <!-- 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: 0af9be94298c676f5bf7881847880e6f43955602
9ae3571ea622c029bc3a875fbfe8ff9fc46e6850: servo: Merge #13738 - layout: Don't try to vertically align absolutely positioned children of table rows (from pcwalton:table-vertical-align-absolute); r=mbrubeck
Patrick Walton <pcwalton@mimiga.net> - Thu, 13 Oct 2016 18:12:17 -0500 - rev 368945
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13738 - layout: Don't try to vertically align absolutely positioned children of table rows (from pcwalton:table-vertical-align-absolute); r=mbrubeck Improves http://reddit.com/r/aww. r? @mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: a80abf6c1dd0f14b40499d55129a53d397c3a3cd
69036a52f7c22bd95d219bb2024cf3b32582535c: servo: Merge #13640 - Move some of the CSSStyleDeclaration logic to the style crate (from servo:CSSStyleDeclaration_in_style); r=mbrubeck
Simon Sapin <simon.sapin@exyr.org> - Wed, 12 Oct 2016 14:21:34 -0500 - rev 368928
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13640 - Move some of the CSSStyleDeclaration logic to the style crate (from servo:CSSStyleDeclaration_in_style); r=mbrubeck <!-- Please describe your changes on the following line: --> … so that Stylo can re-use it. Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1295865 --- <!-- 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 refator <!-- 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: 626d093245066755c751510d2c09a9b864b129fd
cd4435e2d8a5ec8acd35431f4ba90f7c16a6799d: servo: Merge #13056 - Implement transition event and infrastructure (from KiChjang:transition-event); r=mbrubeck
Keith Yeung <kungfukeith11@gmail.com> - Wed, 12 Oct 2016 02:08:37 -0500 - rev 368921
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13056 - Implement transition event and infrastructure (from KiChjang:transition-event); r=mbrubeck Fixes #10245. Source-Repo: https://github.com/servo/servo Source-Revision: cd2f950de3bcbf88208dec16f7025ff516473e0d
b44b1bc74fa0d02675fa54d4a45c1cbba0eb20fe: servo: Merge #13602 - layout: Rewrite Servo's `vertical-align` support to match CSS 2.1 § 10.8, and implement `vertical-align: middle` per CSS 2.1 § 10.8.1 (from pcwalton:vertical-align-middle); r=mbrubeck
Patrick Walton <pcwalton@mimiga.net> - Tue, 11 Oct 2016 15:33:47 -0500 - rev 368915
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13602 - layout: Rewrite Servo's `vertical-align` support to match CSS 2.1 § 10.8, and implement `vertical-align: middle` per CSS 2.1 § 10.8.1 (from pcwalton:vertical-align-middle); r=mbrubeck `InlineMetrics` has been split into `InlineMetrics` for fragments and `LineMetrics` for lines. Both structures' fields have been renamed in order to more clearly delineate the difference between *space* and *content*. Vertical positioning of fragments has been reworked to take margins and borders into account only for replaced content. This patch fixes the `vertical_align_super_a.html` reftest. Servo now matches the rendering that Gecko and WebKit produce. Additionally, this includes a test for the popular inline-block centering technique described here: https://s.codepen.io/shshaw/fullpage/gEiDt?#Inline-Block r? @mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 768a699fa5d822173b2247d6348413d09951a8bb
03f6994d31faef1cef7f1c7f8c3b62f22b149fcd: servo: Merge #13633 - Send touch events to root pipeline, and allow forwarding to iframes (from glennw:forward-touch-events); r=mbrubeck
Glenn Watson <github@intuitionlibrary.com> - Fri, 07 Oct 2016 19:07:24 -0500 - rev 368884
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13633 - Send touch events to root pipeline, and allow forwarding to iframes (from glennw:forward-touch-events); r=mbrubeck Instead of letting the compositor try to find the correct scroll layer for a touch event, switch touch events to work the same way that mouse events do. Touch events are now dispatched to the root pipeline, and then forwarded to child iframes as required. Source-Repo: https://github.com/servo/servo Source-Revision: 9d3fc76463cb226ffeb9a6aca7239bbe36543802
4f06a46d436fa9bce9e18d2e499544c328abbb51: servo: Merge #13610 - Allow simulate_mouse_click to work with webrender (from glennw:sim-mouse-wr); r=mbrubeck
Glenn Watson <github@intuitionlibrary.com> - Thu, 06 Oct 2016 14:21:15 -0500 - rev 368872
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13610 - Allow simulate_mouse_click to work with webrender (from glennw:sim-mouse-wr); r=mbrubeck Instead of dispatching to a specific layer, call the dispatch functions for mouse (move) events that handle whether webrender is enabled or not. Source-Repo: https://github.com/servo/servo Source-Revision: 1161d17f674e44d39bdbd948fe6e814954890672
5123797821bdfcdc1d9af12c97875c4228a1d2b7: servo: Merge #13517 - Moved pub_domains to net_traits and did a spring clean (from asajeffrey:net-traits-pub-domain); r=mbrubeck
Alan Jeffrey <ajeffrey@mozilla.com> - Fri, 30 Sep 2016 21:57:36 -0500 - rev 368821
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13517 - Moved pub_domains to net_traits and did a spring clean (from asajeffrey:net-traits-pub-domain); r=mbrubeck <!-- Please describe your changes on the following line: --> Moved the `pub_domains` module from `net` into `net_traits`, so it can be used by crates which don't depend on `net` (in particular `constellation`, which can use the crate for testing similar-origin). I also did quite a bit of tidying up, the functions now do no heap allocation, previously there was quite a bit of String allocation going on. This is the next step to fixing #633. cc @jdm --- <!-- 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] There are tests for these changes (new unit tests) <!-- 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: 128a61eb9d7a7dbb3ad8bdd9218efcc171229c61
ed7aae3d3d5415c00f3dbecc1d4905e39dec120c: servo: Merge #13445 - resources: Give button elements `white-space: normal` again (from pcwalton:button-whitespace); r=mbrubeck
Patrick Walton <pcwalton@mimiga.net> - Fri, 30 Sep 2016 13:16:56 -0500 - rev 368816
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13445 - resources: Give button elements `white-space: normal` again (from pcwalton:button-whitespace); r=mbrubeck Fixes Twitter. Originally regressed in 0a86543e6decf02860a5a98c46e01940e79af6fa. r? @metajack Source-Repo: https://github.com/servo/servo Source-Revision: 6693c92c03c9bfffaa748246416d4db795a6a988
03fdc91db4149e692c59c3255ae2a6fb45e2596f: servo: Merge #13414 - Implement `word-break: keep-all` (#9673) (from flacerdk:master); r=mbrubeck
Felipe Lacerda <fegolac@gmail.com> - Thu, 29 Sep 2016 19:20:55 -0500 - rev 368804
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13414 - Implement `word-break: keep-all` (#9673) (from flacerdk:master); r=mbrubeck <!-- Please describe your changes on the following line: --> Implement the `keep-all` value for the `word-break` property, as specified in [CSS](https://drafts.csswg.org/css-text-3/#word-break-property). The relevant CSSWG tests (in `tests/wpt/css-tests/css-text-3_dev/html/word-break-keep-all-*.htm`) do not currently pass. As far as I can tell, this is because the tests use some JavaScript code that is not working properly. (But then, it seems that most tests in this directory are failing at the moment. I'm not sure what can be done here for now.) --- <!-- 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 #9673. <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- 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: c9442346d3b454d9cbfcbd76d887e4f22f942531
bd478fc9bb0ee7573e39cd9a48fe7d512981dbf1: servo: Merge #13358 - Form action url 11219 (from ducks:form-action-url-11219); r=mbrubeck
Jake Goldsborough <rjgoldsborough@gmail.com> - Tue, 27 Sep 2016 15:28:09 -0500 - rev 368789
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #13358 - Form action url 11219 (from ducks:form-action-url-11219); r=mbrubeck First pass at fixing #10580. I've added a new macro that returns a DomString with either the attr val or the doc url. I then made the form element use that macro on the action attribute. I also added a test that contains an iframe with a form and base url that submits to a page in a resources directory. I made all these changes based on https://github.com/servo/servo/pull/11219#issuecomment-223318881. The only thing I'm confused on is how to change step 8. It looks to just be getting the action so I'm wondering if I need to change either step 9 or 10 instead? --- - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #10580 (github issue number if applicable). - [X] There are tests for these changes OR using that macro with the form action, making the form submit process use base url, adding tests. Source-Repo: https://github.com/servo/servo Source-Revision: 040075ad085f7e74f207241fedba144d710f42b7
af0ad17724919af3ab7997eeb610b0f9c492a8f9: servo: Merge #11726 - Issue 7720: Add target selector and update when scrolling to fragment (from sjmelia:7720_add_target_selector); r=mbrubeck
Steve Melia <steve.j.melia@gmail.com> - Wed, 03 Aug 2016 14:53:30 -0500 - rev 368453
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #11726 - Issue 7720: Add target selector and update when scrolling to fragment (from sjmelia:7720_add_target_selector); r=mbrubeck <!-- Please describe your changes on the following line: --> Add the target pseudo selector and set/unset it during scrolling to fragment. This change is not complete as no repaint is triggered after the selector is added - it will only take effect after a repaint is triggered by e.g. hovering over another element. (See manual test) I would like some help because i'm not sure how to resolve this; I can only think to call window.reflow. I added a manual test case, don't think this counts really! I think the applicable automated test is in /tests/wpt/web-platform-tests/dom/nodes/Element-matches.html but it currently fails, I think due to the above. --- <!-- 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 #7720 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- 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: 7a7bdf51adc3c5051f93c8044797c8fcaf89953c
2de25a8b8e9996cd18083ad2e59f5bbcdfef90d5: servo: Merge #11890 - Bug 10104 - Only restyle nodes that uses viewport percentage units on viewport size change (from shinglyu:viewport-percent-recalc); r=mbrubeck
Shing Lyu <shing.lyu@gmail.com> - Sun, 17 Jul 2016 20:46:24 -0700 - rev 368340
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #11890 - Bug 10104 - Only restyle nodes that uses viewport percentage units on viewport size change (from shinglyu:viewport-percent-recalc); r=mbrubeck <!-- Please describe your changes on the following line: --> Bug 10104 - Only restyle nodes that uses viewport percentage units on viewport size change --- <!-- 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 #10104 (github issue number if applicable). <!-- Either: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- 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: d87ea67bf2efec45368d3e48634c6c16de7486ff
a21320fcf1c0b9f988c437324e9c7d356ad3dc97: servo: Merge #12370 - Fix line-breaking with white-space: pre-wrap/pre-line; (from emilio:wrap); r=mbrubeck
Emilio Cobos Álvarez <ecoal95@gmail.com> - Mon, 11 Jul 2016 01:08:57 -0700 - rev 368290
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12370 - Fix line-breaking with white-space: pre-wrap/pre-line; (from emilio:wrap); r=mbrubeck Fix line-breaking with white-space: pre-wrap/pre-line; <!-- Please describe your changes on the following line: --> Not sure if this is the appropriate fix, but we'll know soon. --- <!-- 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 #12369 <!-- Either: --> - [x] There are tests for these changes OR <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> r? @mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 8ded106186aa0855a2bd991d573e7095590efca1
b6b103551d8579b645e159e6b969aa2c7cbda559: servo: Merge #12293 - Correct negative margins in centered table cells (from notriddle:margins_in_tables); r=mbrubeck
Michael Howell <michael@notriddle.com> - Fri, 08 Jul 2016 15:25:07 -0700 - rev 368275
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12293 - Correct negative margins in centered table cells (from notriddle:margins_in_tables); r=mbrubeck Fixes the cell-filling links on @AelitaBot [queue viewer page](http://ec2-52-26-55-30.us-west-2.compute.amazonaws.com/). --- - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] Couldn't find an applicable issue, and by the time I reduced the test case I'd already found the bug - [X] There are tests for these changes Source-Repo: https://github.com/servo/servo Source-Revision: 7afe975281326490cdbf0ee3c0d151a2ecb13784
0085c13d54dccdfe2516f23d75dd960ec3d62a1b: servo: Merge #12113 - Removing u8 from HTMLIframeElement.rs file (from Coder206:u8); r=mbrubeck,emilio
Patrick Trottier <ptrottier@localhost.localdomain> - Sat, 02 Jul 2016 08:13:21 -0700 - rev 368209
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12113 - Removing u8 from HTMLIframeElement.rs file (from Coder206:u8); r=mbrubeck,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 - [x] These changes fix #11599 (github issue number if applicable). <!-- Either: --> - [x] These changes do not require tests because @nox did not request any so I am not sure which to do. <!-- 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: 40909eda84522f578f9ae6758c6d2b5689f8eb66
1d4eabfdbb1cdf76fc963efac9045bf70599d831: servo: Merge #11748 - stylo: Support the rest of the background-xxx P1 properties (from emilio:stylo-background-xxx); r=mbrubeck
Emilio Cobos Álvarez <me@emiliocobos.me> - Thu, 16 Jun 2016 04:44:44 -0500 - rev 368107
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #11748 - stylo: Support the rest of the background-xxx P1 properties (from emilio:stylo-background-xxx); r=mbrubeck <!-- 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 <!-- Either: --> - [x] These changes do not require tests because geckolib-only changes <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> We do the same that with background-{image, repeat}, we just use the first element until servo can parse and use a list efficiently. cc/r? @mbrubeck @bholley @heycam Source-Repo: https://github.com/servo/servo Source-Revision: 6d4b9e65e652166adbdee6c16fe12d27d8df6418
11f80e2094c601be5ef4704b2d6cfd1e760a8d41: servo: Merge #11746 - Image with height defined in % resizes properly (from DarinM223:master); r=mbrubeck
Darin Minamoto <darinm223@gmail.com> - Thu, 16 Jun 2016 03:57:38 -0500 - rev 368106
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #11746 - Image with height defined in % resizes properly (from DarinM223:master); r=mbrubeck <!-- Please describe your changes on the following line: --> Checked the html in the github issue and the image looks like it is resized properly in Servo. Fixed image resizing when height is specified in percentages. --- <!-- 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 #11723 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because this issue is marked with has-test? <!-- 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: d2db39634e6e39f4cb054e0b443339409ae69350
deb7ba588bea4ce43cae8e3a465a48eb5c687411: servo: Merge #11745 - Update to rustc 2016-06-14 (from servo:rustup); r=mbrubeck
Ms2ger <Ms2ger@gmail.com> - Tue, 14 Jun 2016 18:32:37 -0500 - rev 368101
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #11745 - Update to rustc 2016-06-14 (from servo:rustup); r=mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 40e2b7d674531a36af1f96ac079e036dcaf1304f
86e3ca7e08e578761feb3448930fce01975edf6f: servo: Merge #11691 - Add very simple Android browser page load support (from larsbergstrom:android_load); r=mbrubeck
Lars Bergstrom <lars@lars.com> - Thu, 09 Jun 2016 22:45:23 -0500 - rev 368084
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #11691 - Add very simple Android browser page load support (from larsbergstrom:android_load); r=mbrubeck I've long had this set of private patches that enables actually demoing Servo on Android without having the device connected, but they're a bit hackish due to some current limitations in our windowing toolkit library. I'm considering committing them, though, as it makes the resulting APK *actually* somewhat usable. Thoughts / r? @mbrubeck Source-Repo: https://github.com/servo/servo Source-Revision: 2b32ec430124ef19bf685610c74367b520c020aa
eaa60a63e80bfe6564ecf428b37af1bbecf5b3f8: servo: Merge #11687 - Android basedir (from larsbergstrom:android_basedir); r=mbrubeck
Lars Bergstrom <lars@lars.com> - Thu, 09 Jun 2016 15:57:56 -0500 - rev 368080
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #11687 - Android basedir (from larsbergstrom:android_basedir); r=mbrubeck <!-- Please describe your changes on the following line: --> This fixes #11633, as when the new basedir stuff that uses xdg was added, it did not preserve the previous Android behavior, which used `/sdcard/servo`. Obviously, long-term we should be calling into the Android runtime to determine the per-run installation path and be both unpacking and loading resources from there, but this restores the previous behavior. r? @mbrubeck --- <!-- 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 #11633 (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes OR - [ x] These changes do not require tests because _____ <!-- 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: ebb1cb30f758273ba3efbe1e1ae3a504faed8293