aboutsummaryrefslogtreecommitdiffstats
path: root/python/servo
Commit message (Collapse)AuthorAgeFilesLines
* Add mach command to update public domain list and use a HashSet instead of a ↵Florian Duraffourg2016-06-091-0/+23
| | | | Vec to lookup public domains
* Auto merge of #11543 - UK992:browserhtml, r=metajackbors-servo2016-06-082-2/+19
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | Fix ./mach --browserhtml Rebased https://github.com/servo/servo/pull/10943 and simplified by https://github.com/servo/servo/pull/10943#issuecomment-221643014. Fixes path related issue with browser.html on Windows and enable borderless window only on OSX. r? @metajack <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11543) <!-- Reviewable:end -->
| * Only enable borderless browserhtml window on OSX.Adam Casey2016-06-072-2/+19
| | | | | | | | Pass browserhtml path as relative path to avoid a mingw->windows path conversion
* | mach: Set a nice icon for the binary on the Mac.Patrick Walton2016-06-081-0/+18
| | | | | | | | | | | | | | | | There's actually a practical reason for this: it helps me pick out the app in Instruments.app. See: http://apple.stackexchange.com/questions/6901/how-can-i-change-a-file-or-folder-icon-using-the-terminal/161984#161984
* | Auto merge of #11516 - DDEFISHER:master, r=metajackbors-servo2016-06-061-0/+9
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | add update-manifest command - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #11369 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11516) <!-- Reviewable:end -->
| * add update-manifest commandDaniel2016-06-061-0/+9
| |
* | Auto merge of #11210 - edunham:mach-package, r=larsbergstrombors-servo2016-06-062-62/+139
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Start on Mach package 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 - these changes don't touch anything that mach build touches> - [ ] `./mach test-tidy --faster` does not report any errors - Tidy errors on some dependencies that I think we'll need for real `package` but aren't using for android - [X] These changes address #9918 (github issue number if applicable). Either: - [ ] There are tests for these changes OR - [x] These changes do not require tests because I don't think Mach has tests yet? Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11210) <!-- Reviewable:end -->
| * | Create `mach package`edunham2016-06-032-62/+139
| | | | | | | | | | | | | | | | | | | | | * Split package commands into their own file * Delete spurious files from build dir * Create runservo.sh to invoke servo with the right browserhtml incantation * Tar it all up with the date and time in the filename
* | | Add FreeBSD to `host_triple`Daniel Robertson2016-06-051-0/+2
|/ / | | | | | | | | Add FreeBSD to `host_triple` so that the correct nightly compiler will be downloaded.
* | mach: Introduce --all flag to run all test suitesAndrew Shu2016-06-021-7/+16
| | | | | | | | Also remove broken "faster" parameter passed to test-tidy
* | Provide a better error message when downloading rustc failsPer Lundberg2016-05-311-2/+2
| | | | | | | | | | | | I was running into the error below locally. The previous version of the code hid away important details that helps when debugging; hence, this suggested change. > Error downloading Rust compiler: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590). URL: https://static-rust-lang-org.s3.amazonaws.com/dist/2016-05-17/rustc-nightly-x86_64-pc-windows-gnu.tar.gz
* | Warn me when I try to sync upstream tests without requesting a commit.Ms2ger2016-05-301-0/+10
| | | | | | | | | | This default invariably wastes my time, and I'm about the only person using the sync argument.
* | Remove the gonk portLars Bergstrom2016-05-273-104/+2
| |
* | Auto merge of #11378 - mbrubeck:debug-mozjs, r=larsbergstrombors-servo2016-05-273-2/+17
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Default to the same Cargo features in all build commands Fixes #8308 and #7327. r? @larsbergstrom <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11378) <!-- Reviewable:end -->
| * | Default to the same Cargo features in all build commandsMatt Brubeck2016-05-273-2/+17
| | | | | | | | | | | | Fixes #8308 and #7327.
* | | Auto merge of #11374 - servo:no-microseconds, r=Ms2gerbors-servo2016-05-251-5/+9
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don’t include microsecons when reporting build times. "0:03:40.817715" is harder to read and no more informative than "0:03:40" <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11374) <!-- Reviewable:end -->
| * | | Don’t include microsecons when reporting build times.Simon Sapin2016-05-241-5/+9
| |/ / | | | | | | | | | "0:03:40.817715" is harder to read and no more informative than "0:03:40"
* | | Auto merge of #11409 - mitchhentges:test-tidy-faster-default, r=aneeshusabors-servo2016-05-251-18/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove --faster flag from test-tidy, go fast by default. Fixes 11217 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 #11217 (github issue number if applicable). Either: - [ ] There are tests for these changes OR - [X] These changes do not require tests because they just change command-line options, which aren't tested Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. This finishes #11267 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11409) <!-- Reviewable:end -->
| * | | Remove --faster flag from test-tidy, go fast by default. Fixes 11217Mitchell Hentges2016-05-251-18/+3
| | | |
* | | | Auto merge of #11393 - perlun:fix/find-cargo-exe-on-windows, r=metajackbors-servo2016-05-251-2/+4
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed the build tools to handle cargo being named cargo.exe It's actually quite weird that I had to do this, but I got errors without it (on Windows). I noted the `BIN_SUFFIX` existed elsewhere so it felt logical to use it here also. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11393) <!-- Reviewable:end -->
| * | | Fixed the build tools to handle cargo being named cargo.exePer Lundberg2016-05-251-2/+4
| |/ / | | | | | | | | | It's actually quite weird that I had to do this, but I got errors without it. I noted the `BIN_SUFFIX` existed elsewhere so it felt logical to use it here also.
* | | Auto merge of #11267 - mitchhentges:test-tidy-sanic-by-default, r=aneeshusabors-servo2016-05-241-2/+17
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add --all option to `mach test-tidy`, with compatibility with --faster 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 --faster` does not report any errors - [X] These changes partially fix #11217 Either: - [ ] There are tests for these changes OR - [X] These changes do not require tests because it just affects `mach` script parameters Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11267) <!-- Reviewable:end -->
| * | Add --all for test-tidy, with backwards-compat with --fasterMitchell Hentges2016-05-241-2/+17
| | |
* | | Auto merge of #11278 - mitchhentges:8348-reduce-duplication, r=aneeshusabors-servo2016-05-241-0/+3
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce 'android-18' duplication by putting it in $ANDROID_TARGET env variable 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 --faster` does not report any errors - [x] These changes fix #8348 Either: - [X] These changes do not require tests because ~~I'm lazy~~ it's a configuration change, and asserting default values is usually tedious and not very helpful Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --- Allegedly, this broke the build [back in the day](https://github.com/servo/servo/pull/8519). Going to see if it's a piece of cake to land now <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11278) <!-- Reviewable:end -->
| * | Reduce 'android-18' duplication, as originally done by wenderenMitchell Hentges2016-05-201-0/+3
| |/ | | | | | | Use android.platform (and $ANDROID_PLATFORM) for configuration, apply 'android-18' default
* | browserhtml updatePaul Rouget2016-05-231-1/+4
| |
* | Make Servo DPI aware on WindowsJack Moffitt2016-05-201-0/+4
|/ | | | | | | | | | | | | | | | This implements system level DPI awareness for Windows. It has three parts: 1. Add a application manifest which is copied alongside servo.exe during build that declares our DPI awareness level. This is needed otherwise DPI queries will return 96dpi and our application will be upscaled on high DPI displays. 2. Rename hidpi_factor to avoid confusion with Glutin's hidpi_factor which does something else. 3. Correctly convert windows sizes on window creation for Windows. Unlike OS X, Windows uses device pixels for window creation.
* Auto merge of #11196 - fduraffourg:master, r=jdmbors-servo2016-05-171-0/+12
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add unit tests for cookies handling 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 #9965 Either: - [X] There are tests for these changes OR - [ ] These changes do not require tests because _____ Add unit tests for the `net` component about cookies. The tests are generated with a new `mach update-net-cookies` command from this repo: https://github.com/abarth/http-state. This PR also includes two trivial bug fixes about cookie handling. From all the tests included, the following ones are currently failing: - cookie_http_state::test_0003 - cookie_http_state::test_0006 - cookie_http_state::test_attribute0004 - cookie_http_state::test_attribute0005 - cookie_http_state::test_attribute0007 - cookie_http_state::test_attribute0008 - cookie_http_state::test_domain0017 - cookie_http_state::test_mozilla0001 - cookie_http_state::test_mozilla0002 - cookie_http_state::test_mozilla0003 - cookie_http_state::test_mozilla0005 - cookie_http_state::test_mozilla0007 - cookie_http_state::test_mozilla0009 - cookie_http_state::test_mozilla0010 - cookie_http_state::test_mozilla0013 `test_000[36]` and `test_mozilla*` are failing because there is currently no method to clean a `net::cookie_storage` from expired cookies. `test_attribute000[4578]` are failing because hyper does not parse the `Secure` attribute correctly. I will open an issue on the upstream project. `test_domain0017` fails because the TLD .org is not on the PUB_DOMAINS list. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11196) <!-- Reviewable:end -->
| * Add unit tests for cookies base on abarth/http-state github repoFlorian Duraffourg2016-05-171-0/+12
| | | | | | | | | | - Add unit tests - Add a mach command to update cookie's unit tests
* | Implement --pref in test-wptAnthony Ramine2016-05-161-0/+8
|/
* Auto merge of #11122 - mbrubeck:unify-builds, r=larsbergstrombors-servo2016-05-112-16/+13
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the same build environment and features for CEF, Servo, Gonk, Geckolib * Remove unnecessary dependencies and features from top-level Cargo.tomls. The features for each crate will be computed based on the union of features specified in the dependency graph. Specifying the same ones again just adds more ways for them to get out of sync. * Move all cargo build environment variables into CommandBase Fixes #11112. r? @metajack (Not included: CI test to make sure #11112 doesn't regress again.) <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11122) <!-- Reviewable:end -->
| * Remove unnecessary dependencies and features from top-level Cargo.tomlsMatt Brubeck2016-05-111-3/+0
| | | | | | | | | | | | The features for each crate will be computed based on the union of features specified in the dependency graph. Specifying the same ones again just adds more ways for them to get out of sync.
| * Move all cargo build environment variables into CommandBaseMatt Brubeck2016-05-102-13/+13
| |
* | Override $CARGO_HOME only if cargo-home-dir is in .servobuildMatt Brubeck2016-05-111-2/+3
|/ | | | | | | | | | | | | | | This preserved the fix from #11097 in the situation where `$CARGO_HOME` and `cargo-home-dir` in `.servobuild` are both present: `.servobuild` should win. But it changes the behavior when `$CARGO_HOME` is present and `.servobuild` is not: Then `$CARGO_HOME` should be used. We now check the following values in order of priority and use the first one that is found: 1. `cargo-home-dir` in `.servobuild` 2. `CARGO_HOME_DIR` in the enivironment 3. default value (`<servo-repo>/.cargo`)
* Override always 'CARGO_HOME' env variable.Tetsuharu OHZEKI2016-05-101-2/+1
| | | | | | | | | | | | | This use always `cargo-home-dir` in `/.servobuild` even if you set `CARGO_HOME` in your shell. If you use [racer][racer] with [rustup (multirust.rs)][rustup], you may set `CARGO_HOME` env variable. Then the previous code would be a problem which does not use `cargo-home-dir` in `/.servobuild`. [racer]: https://github.com/phildawes/racer [rustup]: https://github.com/rust-lang-nursery/rustup.rs
* Auto merge of #11013 - antrik:rust-gdb-cleanup, r=Ms2gerbors-servo2016-05-061-1/+2
|\ | | | | | | | | | | | | | | | | | | | | | | mach run --debug: Use nicer syntax in rust-gdb/rust-lldb check After actually reading a Python tutorial, I realised this can be handled in a more elegant fashion :-) <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11013) <!-- Reviewable:end -->
| * mach run --debug: Cleanup: Use nicer syntax in rust-gdb/rust-lldb checkOlaf Buddenhagen2016-05-041-1/+2
| | | | | | | | | | | | Follow-up on f3cd5d2961d5493f17062562693608b73ced6128 : after actually reading a Python tutorial, I realised this can be handled in a more elegant fashion :-)
* | Auto merge of #10916 - mmatyas:useneon, r=aneeshusabors-servo2016-05-052-17/+24
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use NEON build flag on ARM and AArch64 The NEON flag is already used when building for Android, this patch enables it on other ARM devices too. Note that this patch just adds the build flag to the compilation, for actually enabling the SIMD code in Servo, we'll also need #10900 (but it's not a dependency). <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10916) <!-- Reviewable:end -->
| * | Use NEON build flag on ARM and AArch64Mátyás Mustoha2016-05-051-2/+6
| | |
| * | Improve build target argument handlingMátyás Mustoha2016-05-052-15/+18
| | |
* | | Make build time display more human friendlyFabrice Desré2016-05-041-5/+6
|/ /
* / Upgrade to OpenSSL 1.0.1t on AndroidMatt Brubeck2016-05-031-1/+1
|/
* Auto merge of #10923 - askeing:fix_10922, r=jdmbors-servo2016-04-301-1/+4
|\ | | | | | | | | | | | | | | | | | | | | Remove dir by shutil.rmtree, remove file by os.remove fix #10922 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10923) <!-- Reviewable:end -->
| * Remove dir by shutil.rmtree, remove file by os.removeaskeing2016-04-291-1/+4
| |
* | Auto merge of #10901 - askeing:fix_10882, r=jdmbors-servo2016-04-301-3/+6
|\ \ | |/ |/| | | | | | | | | | | | | | | | | Handle HTTP specific errors then other errors when downloading rustc fix #10882 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10901) <!-- Reviewable:end -->
| * Handle HTTP specific errors then other errors when downloading rustcaskeing2016-04-301-3/+6
| |
* | Auto merge of #10892 - CorcovadoMing:patch-1, r=Wafflespeanutbors-servo2016-04-281-1/+4
|\ \ | |/ |/| | | | | | | | | | | | | | | | | Pass --no-patch by default to update_css fix #10884 according to #9666 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10892) <!-- Reviewable:end -->
| * pass --no-patch by default to update_cssCorcovadoMing2016-04-281-1/+4
| |
* | Make css-properties.json checking a proper unit test.Simon Sapin2016-04-281-24/+0
|/
* Auto merge of #10706 - zwn:unused-extern-crates, r=noxbors-servo2016-04-221-0/+2
|\ | | | | | | | | | | | | | | | | | | | | Turn on unused-extern-crates warning. As discussed in #9256. It should solve second half of the issue. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10706) <!-- Reviewable:end -->