5483d2e6a99edeb90d51a779dd6075a500df6438: servo: Merge #12976 - A few minor cleanups in tidy (from Wafflespeanut:tidy_fixes); r=nox
Ravi Shankar <wafflespeanut@gmail.com> - Tue, 23 Aug 2016 03:29:05 -0500 - rev 368576
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12976 - A few minor cleanups in tidy (from Wafflespeanut:tidy_fixes); r=nox <!-- 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 it's just a cleanup <!-- 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: eec56ca50e2d6624b31f23a870936c02e2674573
1083070f741e7ce1a22d391807e4bd50d06388b7: servo: Merge #12921 - Trigger image.onerror (from paulrouget:imgOnError); r=nox
Paul Rouget <me@paulrouget.com> - Tue, 23 Aug 2016 02:31:48 -0500 - rev 368575
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12921 - Trigger image.onerror (from paulrouget:imgOnError); r=nox --- <!-- 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 #12885 (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: fe09cb5504ebc24c903eae525ffad5c376a12b2b
06dc2cc325db6c68cf36890f33c2c9264a392c71: servo: Merge #12983 - Update glutin to get new OSMesa features (from glennw:update-glutin); r=larsbergstrom
Glenn Watson <github@intuitionlibrary.com> - Tue, 23 Aug 2016 01:37:36 -0500 - rev 368574
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12983 - Update glutin to get new OSMesa features (from glennw:update-glutin); r=larsbergstrom Source-Repo: https://github.com/servo/servo Source-Revision: f99929bee305c8e3d86b06386e77486b62725157
15b420ea45a518918e25759ba7f033a996b656ce: servo: Merge #12971 - Change the separator used in rust path into os.path.join (from tp6vup54:master); r=Wafflespeanut
tp6vup54 <genius091612@gmail.com> - Tue, 23 Aug 2016 00:39:55 -0500 - rev 368573
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12971 - Change the separator used in rust path into os.path.join (from tp6vup54:master); r=Wafflespeanut <!-- Please describe your changes on the following line: --> Trying to solve #12961 and following the suggestion in it. Already run test-unit and no red appeared. --- <!-- 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 - [x] These changes fix #12961 (github issue number if applicable). <!-- Either: --> - [x] These changes do not require tests because it's a cleanup of `mach` <!-- 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: 581e69f8ba1b949dcb5bfa215241d27e3460ee89
aeec11f93e77148c002fdd94352f7f3c2f35e0e3: servo: Merge #12945 - Support multiple backgrounds in both servo and stylo (from Manishearth:bgarray); r=SimonSapin
Manish Goregaokar <manishsmail@gmail.com> - Mon, 22 Aug 2016 22:09:23 -0500 - rev 368572
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12945 - Support multiple backgrounds in both servo and stylo (from Manishearth:bgarray); r=SimonSapin (Commits do not build individually, but split up for review) These patches make all of the background- properties accept multiple values, and add the layout code to display them. Still needs some cleanup, and some testing, but it seems to work. r? @SimonSapin Source-Repo: https://github.com/servo/servo Source-Revision: c5e81f836190ea64df95f67c922b8421a4d2ff78
83fb84bf4975973487d8a06840e0b15f02dac5db: servo: Merge #12933 - Make GeckoDeclarationBlock refcounted (from upsuper:cssom-for-stylo); r=upsuper
Xidorn Quan <me@upsuper.org> - Mon, 22 Aug 2016 21:12:23 -0500 - rev 368571
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12933 - Make GeckoDeclarationBlock refcounted (from upsuper:cssom-for-stylo); r=upsuper <!-- Please describe your changes on the following line: --> This makes GeckoDeclarationBlock refcounted, so that it can share most of the logic Gecko currently uses for its existing `css::Declaration`. Although this is almost only useful for Gecko side, doing this in Servo side eliminates additional object allocation for holding it in the Gecko side. Its Gecko side code change is in [bug 1296186](https://bugzilla.mozilla.org/show_bug.cgi?id=1296186). We may not want to land this before that part also gets reviewed. r? @Manishearth --- <!-- 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 for Gecko binding only <!-- 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: 27f59cc250156892583782597f16a347c3807523
3bd953a3df9e45066db474e816d19551b17ffde4: servo: Merge #12915 - Include "content-type" in cors safelisted request headers (from jeenalee:parse-headers); r=Manishearth
Jeena Lee <ijeenalee@gmail.com> - Mon, 22 Aug 2016 20:17:16 -0500 - rev 368570
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12915 - Include "content-type" in cors safelisted request headers (from jeenalee:parse-headers); r=Manishearth <!-- Please describe your changes on the following line: --> The changes in headers.rs will allow headers with "content-type" name to be classified as cors safelisted request headers, depending on its value according to [the Fetch spec](https://fetch.spec.whatwg.org/#cors-safelisted-request-header). As a result of this change, more request web platform tests pass, whose expected test results are updated with this commit. There is possibly one TODO related to this PR: - Figure out what `name/'invalid'` is in step 4 of the [Headers Delete method](https://fetch.spec.whatwg.org/#dom-headers-delete), and how to implement that. --- <!-- 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 web platform tests for the changes already exist. <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> This commit allows headers with "content-type" name to be classified as valid header name, depending on its value according to [the Fetch spec](https://fetch.spec.whatwg.org/#cors-safelisted-request-header). As a result of this change, more request web platform tests pass, whose expected test results are updated as well. Source-Repo: https://github.com/servo/servo Source-Revision: 1e10f8bbf700a43d4d3082cc00034594a2cefc0e
9c76653e2ddb3937373ad7ad9c818c78bf87abb3: servo: Merge #12938 - Sanitise web fonts (from nox:fontsan); r=larsbergstrom
Keegan McAllister <kmcallister@mozilla.com> - Mon, 22 Aug 2016 19:20:19 -0500 - rev 368569
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12938 - Sanitise web fonts (from nox:fontsan); r=larsbergstrom Source-Repo: https://github.com/servo/servo Source-Revision: 785fcd55806b546e2d5d15d5618fdf575262a012
a3934a0d1a9563d09b7bc56a574261c2802fefad: servo: Merge #12956 - Windows MSVC: Define vcvars in mach.bat (from UK992:msvc-vcvars); r=metajack
UK992 <urbankrajnc92@gmail.com> - Mon, 22 Aug 2016 16:52:15 -0500 - rev 368568
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12956 - Windows MSVC: Define vcvars in mach.bat (from UK992:msvc-vcvars); r=metajack Define vcvars for msvc build, if they have not yet been. If vcvars not exist, show message that visual studio 2015 is not installed and where to download it. Fixes #12948 Source-Repo: https://github.com/servo/servo Source-Revision: 7aa0071e9b59db792742e1a47bd8577764a89fa7
a26b0004d05e62ee4761f60c9e563606578fbd29: servo: Merge #12970 - Move thread_state to style (from servo:thread-state); r=nox
Ms2ger <Ms2ger@gmail.com> - Mon, 22 Aug 2016 10:39:51 -0500 - rev 368567
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12970 - Move thread_state to style (from servo:thread-state); r=nox Source-Repo: https://github.com/servo/servo Source-Revision: 1911606e5c7d81e41b79fa25c41b1d7def7aef62
5b3956922721f9e32a54cf215a51095e1bd19b28: servo: Merge #12969 - Track only the number of important declarations in a declaration block (from cers:master); r=nox
Christian Sonne <cers@geeksbynature.dk> - Mon, 22 Aug 2016 08:46:34 -0500 - rev 368566
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12969 - Track only the number of important declarations in a declaration block (from cers:master); r=nox <!-- Please describe your changes on the following line: --> Track only the number of important declarations in a declaration block --- <!-- 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 #12959 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because "the existing tests should cover it" <!-- 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: 57340e43107e6be5c3ff7fb379c4571b8ae1561b
cdc148903ad5b78752b5f136faeba738cc2d8e2b: servo: Merge #12965 - Update various dependencies (from servo:always-be-updating); r=KiChjang
Ms2ger <Ms2ger@gmail.com> - Mon, 22 Aug 2016 07:52:26 -0500 - rev 368565
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12965 - Update various dependencies (from servo:always-be-updating); r=KiChjang Source-Repo: https://github.com/servo/servo Source-Revision: 32a0493cde3339fbc86867d197f356741ee77c09
e30abec7536067d297b11195df5faf58c0bd6900: servo: Merge #12930 - Fix wrong calculation of inline element's block size (from canaltinova:border-bottom); r=emilio
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 22 Aug 2016 06:57:57 -0500 - rev 368564
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12930 - Fix wrong calculation of inline element's block size (from canaltinova:border-bottom); r=emilio Border, padding and margin properties' top and bottom values of inline elements were affecting the element's height. It shouldn't affect it normally. Fixed it and added a test. --- <!-- 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 #11729 (github issue number if applicable). <!-- Either: --> - [X] There are tests for these changes <!-- 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: 91cee66fe890b283b380142f2c3a076fed2b1024
1717e7008dc7264fff9f4235a1f7d7336011a741: servo: Merge #12918 - 12902 typedefs (from hsinewu:12902-typedefs); r=Manishearth
hsinewu <extraordinary1022@gmail.com> - Mon, 22 Aug 2016 06:01:03 -0500 - rev 368563
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12918 - 12902 typedefs (from hsinewu:12902-typedefs); r=Manishearth <!-- Please describe your changes on the following line: --> Using type alias instead of newtype. Removing duplicated implementation, it's already inherited. No more type constructors and foo.0 --- <!-- 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 #12902 (github issue number if applicable). <!-- Either: --> - [X] These changes do not require tests because it's refactoring. <!-- 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: daee53cb7697459b9ad28a9ee377cd6d0b90c0a7
281631312857179b36e220bf482b341838d030b5: servo: Merge #12898 - Burn SelectedFileId in fire (from izgzhen:burn-selected-file-id); r=Manishearth
Zhen Zhang <izgzhen@gmail.com> - Mon, 22 Aug 2016 03:09:28 -0500 - rev 368562
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12898 - Burn SelectedFileId in fire (from izgzhen:burn-selected-file-id); r=Manishearth r? @Manishearth --- - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors Source-Repo: https://github.com/servo/servo Source-Revision: 43d4a459c0080215de10f43ea74b7860fb196009
c15133b225c5903b93dbe3682465deb5fe259ce6: servo: Merge #12962 - Cached element class names in style sharing cache with lazy computation (from ashrko619:cache-element-classnames); r=notriddle
Ashwin R <ashwinravichandran24@gmail.com> - Mon, 22 Aug 2016 01:22:04 -0500 - rev 368561
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12962 - Cached element class names in style sharing cache with lazy computation (from ashrko619:cache-element-classnames); r=notriddle <!-- Please describe your changes on the following line: --> Fixes #12912 Added a field to StyleSharingCandidate to lazily compute and cache the class names. --- <!-- 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 #12912 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [x] These changes do not require 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: 87cc453ba7824191f120eb9af3bf358ca5bc3d8a
72acbb5883be863f468e4b37f076f1725d9cd933: servo: Merge #12932 - Update for webrender API change, and remove webrender specific hack (from glennw:update-wr2); r=ConnorGBrewster
Glenn Watson <github@intuitionlibrary.com> - Sun, 21 Aug 2016 17:05:55 -0500 - rev 368560
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12932 - Update for webrender API change, and remove webrender specific hack (from glennw:update-wr2); r=ConnorGBrewster <!-- 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. --> in display list building. This relies on https://github.com/servo/webrender/pull/355 being merged first, and will then need the cargo.lock updated. Source-Repo: https://github.com/servo/servo Source-Revision: c6494db520c44b40c159dade25b5f4b8fefdd9b6
00597eba95e91ad70a1146d9404f3197c371d974: servo: Merge #12943 - Merge normal and important declarations in style rules (from servo:merged-declaration-block); r=emilio
Simon Sapin <simon.sapin@exyr.org> - Sun, 21 Aug 2016 03:43:25 -0500 - rev 368559
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12943 - Merge normal and important declarations in style rules (from servo:merged-declaration-block); r=emilio Have a single Vec instead of two. Fix #3426 --- <!-- 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 #3426. <!-- 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: f8b2be1ea4d844d5bddd88df2e5db42731bd6a39
06402ee1ea3213afd946803dc2bf726506c31a14: servo: Merge #12955 - Use try_borrow instead of borrow_state (from nox:try-borrow); r=emilio
Anthony Ramine <n.oxyde@gmail.com> - Sun, 21 Aug 2016 00:47:10 -0500 - rev 368558
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12955 - Use try_borrow instead of borrow_state (from nox:try-borrow); r=emilio Source-Repo: https://github.com/servo/servo Source-Revision: 530c36a99ad7faad585bd9998de44b918aa737bc
b8725e36ac246b19ab54f907fd063eb15f4c33d8: servo: Merge #12877 - Tidy config file (from UK992:tidy-toml); r=Wafflespeanut
UK992 <urbankrajnc92@gmail.com> - Sat, 20 Aug 2016 22:27:06 -0500 - rev 368557
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
servo: Merge #12877 - Tidy config file (from UK992:tidy-toml); r=Wafflespeanut This is wip workaround for https://github.com/servo/servo/issues/10841 Adds ``servo-tidy.toml`` with configs and ignored dirs, files and packages. This will allow to set custom configuration per repo. It's an example how could config file looks like. I want opinion on that, if this is right approaches and how to improve it. cc @edunham Source-Repo: https://github.com/servo/servo Source-Revision: 1c9650cc90090784c63f5af55628ffe6b57c25ed
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 tip