Commit graph

18679 commits

Author SHA1 Message Date
Michael Howell
f9a8fe0afe Do not crash on partial calc interpolation
Fixes #12151

Fixes regression introduced by #11924
2016-07-01 21:11:49 -07:00
bors-servo
1ed06777ee Auto merge of #12020 - crepererum:fix/shebang, r=aneeshusa
Add shebang to Linux runservo.sh

<!-- Please describe your changes on the following line: -->
Shebang is missing and therefore `./runservo.sh` does not work under some environments.

I've chosen to use `/usr/bin/env sh` instead of `/bin/sh` or `/usr/bin/sh` to auto-select the right `sh` executable depending on the 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

<!-- Either: -->
- [X] These changes do not require tests because it does not affect servo itself but rather the resulting package under some environments.

<!-- 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/12020)
<!-- Reviewable:end -->
2016-07-01 17:33:22 -07:00
bors-servo
1877c40491 Auto merge of #12004 - imjacobclark:patch-1, r=aneeshusa
Add servo-latest on upload-nightly

Allows for easy linking over to servo-nightly builds.

cc/ @larsbergstrom

---
<!-- 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 #11987 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because they are part of the CI infrastructure

<!-- 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/12004)
<!-- Reviewable:end -->
2016-07-01 15:59:10 -07:00
bors-servo
ec0d3e084d Auto merge of #11946 - emilio:animations-sender, r=bholley
style: Remove the Mutex from new_animations_sender by moving it to the local StyleContext.

<!-- 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 refactoring.

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

As a follow-up, we could move all the data living under a mutex in the
SharedLayoutContext only in order to create the local context to the same place.

This should increase animation performance when there are multiple animations in
one page that happen to be on different threads.

r? @SimonSapin/@mbrubeck for the style/layout, @bholley for the geckolib changes

<!-- 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/11946)
<!-- Reviewable:end -->
2016-07-01 14:12:54 -07:00
Emilio Cobos Álvarez
203d2a62c2
style: Remove the Mutex from new_animations_sender by moving it to the local StyleContext.
As a follow-up, we could move all the data living under a mutex in the
SharedLayoutContext only in order to create the local context to the same place.

This should increase animation performance when there are multiple animations in
one page that happen to be on different threads.
2016-07-01 13:48:06 -07:00
Jacob Clark
6757a374c9 Add servo-latest on upload-nightly
Allows easy linking to nightly builds.
2016-07-01 20:17:27 +01:00
bors-servo
f2d798232f Auto merge of #12035 - alex:patch-1, r=frewsxcv
Disable TLS compression

<!-- 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. -->

This prevents the CRIME attack.

<!-- 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/12035)
<!-- Reviewable:end -->
2016-07-01 08:50:46 -07:00
bors-servo
b2a32ec028 Auto merge of #12008 - servo:keyframes, r=Manishearth
Reorder code in KeyframesAnimation::from_keyframes() to avoid a panic.

Fixes #11999.
Fixes #12006.

<!-- 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/12008)
<!-- Reviewable:end -->
2016-07-01 07:58:50 -07:00
bors-servo
210265d30f Auto merge of #12030 - servo:exit-now, r=Manishearth
Don't panic in LayoutThread::exit_now() if the paint thread is gone a…

Fixes #11996.
Fixes #12005.
Fixes #12007.
Fixes #12011.
Fixes #12026.

<!-- 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/12030)
<!-- Reviewable:end -->
2016-07-01 06:57:15 -07:00
bors-servo
736feec0ad Auto merge of #11974 - ConnorGBrewster:load_blocker_drop_debug_assert, r=asajeffrey
Change load blocker drop assert to debug_assert

<!-- Please describe your changes on the following line: -->
This was causing some crashes for myself and @glennw in browser.html
I will open an issue for this to be investigated later.

---
<!-- 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 `assert` to `debug_assert`

<!-- 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/11974)
<!-- Reviewable:end -->
2016-07-01 06:04:38 -07:00
Alex Gaynor
a0f03dcfac Disable TLS compression
This prevents the CRIME attack.
2016-07-01 07:19:46 -04:00
Ms2ger
0078e6c3bb Don't panic in LayoutThread::exit_now() if the paint thread is gone already.
Fixes #11996.
Fixes #12005.
Fixes #12007.
Fixes #12011.
Fixes #12026.
2016-07-01 12:15:51 +02:00
Ms2ger
70e964b707 Reorder code in KeyframesAnimation::from_keyframes() to avoid a panic.
Fixes #11999.
Fixes #12006.
2016-07-01 12:04:29 +02:00
Marco Neumann
f7b86f349b Add shebang to Linux runservo.sh 2016-07-01 10:56:51 +02:00
bors-servo
0ae07e07e6 Auto merge of #11964 - ConnorGBrewster:fix_ua_comment, r=asajeffrey
Use proper comment syntax in servo.css

<!-- 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 #11963 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because using proper comment syntax in css.

<!-- 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/11964)
<!-- Reviewable:end -->
2016-06-30 23:19:28 -07:00
Lars Bergstrom
4f81384d61 Merge pull request #11979 from ConnorGBrewster/glutinupupup
Update Glutin to pick up borderless window performance fixes on Mac.
2016-07-01 00:07:23 -05:00
Lars Bergstrom
50dca21a97 Merge pull request #11982 from ConnorGBrewster/resources_linux_package
Copy resources on linux package

r=glennw, connorgbrewster
2016-06-30 23:38:06 -05:00
Connor Brewster
322d39442c fix runservo.sh using absolute path 2016-06-30 22:28:22 -06:00
Connor Brewster
0b84921d03 Copy resources on linux package 2016-06-30 22:17:09 -06:00
Lars Bergstrom
5d225d9d66 Merge pull request #11953 from Manishearth/more-res
Bump default resolution to 1024x768
2016-06-30 22:43:02 -05:00
Patrick Walton
7af62f3947 Update Glutin to pick up borderless window performance fixes on Mac.
Large framerate improvement in browser.html.

This takes the Glutin upgrade intended for #11130 but does not address
the underlying issue. A FIXME has been added.
2016-06-30 21:22:39 -06:00
bors-servo
63d99c188c Auto merge of #11977 - larsbergstrom:rollup, r=larsbergstrom
Rollup of 2 pull requests

- Successful merges: #11975, #11976
- Failed merges:

<!-- 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/11977)
<!-- Reviewable:end -->
2016-06-30 21:26:47 -05:00
Lars Bergstrom
e42f6f1747 Rollup merge of #11976 - ConnorGBrewster:bhtmlup, r=asajeffrey
bhtml up

<!-- 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/11976)
<!-- Reviewable:end -->
2016-06-30 21:22:42 -05:00
Lars Bergstrom
b7cda92817 Rollup merge of #11975 - ConnorGBrewster:package_run_script, r=asajeffrey
put run script in the right folder for packaging

<!-- Please describe your changes on the following line: -->
Puts run servo script in the proper folder for packaging

---
<!-- 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 #11944 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because fixing packaging script

<!-- 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/11975)
<!-- Reviewable:end -->
2016-06-30 21:22:42 -05:00
Connor Brewster
855aeebbf4 bhtml up 2016-06-30 19:48:45 -06:00
bors-servo
851931124d Auto merge of #11973 - glennw:demo-fix, r=ConnorGBrewster
Update WR for demo fixes.

<!-- 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. -->

<!-- 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/11973)
<!-- Reviewable:end -->
2016-06-30 19:41:47 -05:00
Connor Brewster
352090875c put run script in the right folder for packaging 2016-06-30 18:33:07 -06:00
Connor Brewster
5060c13493 change load blocker drop assert to debug_assert 2016-06-30 18:13:01 -06:00
Glenn Watson
d2b88e0ac2 Update WR for demo fixes. 2016-07-01 09:32:03 +10:00
E. Dunham
b347c7bfb1 Merge pull request #11971 from larsbergstrom/s3_path
Commit to the correct s3 folder, not file
2016-06-30 15:18:42 -07:00
Lars Bergstrom
041cf80a70 Commit to the correct s3 folder, not file 2016-06-30 16:24:36 -05:00
Connor Brewster
50096c21b0 Use proper comment syntax in servo.css 2016-06-30 11:12:01 -06:00
Manish Goregaokar
e4d3c74928
Bump default resolution to 1024x740 2016-06-30 21:28:50 +05:30
bors-servo
d3a1a4ec7d Auto merge of #11959 - larsbergstrom:change_bucket_name, r=metajack
Change servo s3 build bucket name

<!-- Please describe your changes on the following line: -->
This will allow us to push to URLs like:
http://servo-builds.s3.amazonaws.com/nightly/mac
instead of:
http://servo-developer-preview.s3.amazonaws.com/nightly/mac

We can have different directories for channels, individual releases, etc.

r? @edunham @metajack

cc @tschneidereit

---
<!-- 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. -->

<!-- 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/11959)
<!-- Reviewable:end -->
2016-06-30 10:05:35 -05:00
Lars Bergstrom
bdab11be42 Change bucket name 2016-06-30 09:50:36 -05:00
bors-servo
2ff6c35263 Auto merge of #11954 - paulrouget:bhtmlUpdate, r=Wafflespeanut
update browserhtml

Another browserhtml update to pick performance improvements.

<!-- 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/11954)
<!-- Reviewable:end -->
2016-06-30 07:14:33 -05:00
Paul Rouget
f9e990ee31 update browserhtml 2016-06-30 14:11:27 +02:00
bors-servo
87d08eb4e8 Auto merge of #11952 - cynicaldevil:master, r=Ms2ger
Creates KeyModifiers value using empty() instead of from_bits()

<!-- 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 #11935 .

<!-- Either: -->
- [X] These changes do not require tests because: @jdm said so :P

<!-- 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/11952)
<!-- Reviewable:end -->
2016-06-30 06:19:29 -05:00
Nikhil Shagrithaya
c7fd89a593 Creates KeyModifiers value using empty() instead of from_bits() 2016-06-30 16:09:01 +05:30
bors-servo
2b2b6a3d0c Auto merge of #11951 - paulrouget:bhtmlUpdate, r=nox
browerhtml update

<!-- 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/11951)
<!-- Reviewable:end -->
2016-06-30 05:18:11 -05:00
Paul Rouget
38cec862d9 browerhtml update 2016-06-30 12:16:50 +02:00
bors-servo
26d2ee6355 Auto merge of #11949 - servo:wr, r=glennw
compositing: Only recomposite after a scroll if layers actually moved, and fix the jerky scrolling "pops" during flings on Mac.

See the comments in compositor.rs for more details.

Fixes #11658.

<!-- 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/11949)
<!-- Reviewable:end -->
2016-06-30 04:03:43 -05:00
Patrick Walton
90e970a6d2 compositing: Only recomposite after a scroll if layers actually moved,
and fix the jerky scrolling "pops" during flings on Mac.

See the comments in compositor.rs for more details.

Requires servo/webrender#302 and servo/webrender_traits#64.
2016-06-30 10:18:16 +02:00
bors-servo
8052f5b80d Auto merge of #11931 - izgzhen:fix-html-input-element, r=Manishearth
Fix file-type input element

A simple fix to put `inputElem.files` online

---
<!-- 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] 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. -->

<!-- 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/11931)
<!-- Reviewable:end -->
2016-06-30 01:20:11 -05:00
Zhen Zhang
6ca531fb8f Fix file-type input element 2016-06-30 14:18:11 +08:00
bors-servo
461d7c4eeb Auto merge of #11801 - mbrubeck:snaptopix, r=pcwalton
webrender: Don't use OpenGL round() for snapping pixels

Fixes #11751. See servo/webrender#294 for some details. r? @pcwalton

<!-- 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/11801)
<!-- Reviewable:end -->
2016-06-29 22:35:54 -05:00
bors-servo
0f7e196da6 Auto merge of #11943 - aneeshusa:add-nightly-upload-script, r=larsbergstrom
Add a script to upload nightly packages to S3

<!-- Please describe your changes on the following line: -->

The nightly package name includes the date, and so we use a glob to
locate the package file without hard coding a value. However, globbing
will not work with our Buildbot steps setup because we perform word
splitting ourselves and pass an array to Buildbot, which will directly
exec the array instead of passing it to the shell, meaning globbing
does not occur. Instead, add a script to the servo repo where we can
use globbing, and use `shopt -s failglob` to guard against bad globs.

cc @larsbergstrom @edunham

---
<!-- 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 are needed for #9921  (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [x] These changes do not require tests because will be tested by buildbot

<!-- 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/11943)
<!-- Reviewable:end -->
2016-06-29 21:40:15 -05:00
Aneesh Agrawal
9dc9643487 Add a script to upload nightly packages to S3
The nightly package name includes the date, and so we use a glob to
locate the package file without hard coding a value. However, globbing
will not work with our Buildbot steps setup because we perform word
splitting ourselves and pass an array to Buildbot, which will directly
exec the array instead of passing it to the shell, meaning globbing
does not occur. Instead, add a script to the servo repo where we can
use globbing, and use `shopt -s failglob` to guard against bad globs.
2016-06-29 22:27:14 -04:00
Matt Brubeck
6c00058c5d webrender: Don't use OpenGL round() for snapping pixels
Fixes #11751.
2016-06-29 17:55:31 -07:00
bors-servo
e30b288ed6 Auto merge of #11934 - jdm:navigatefragment, r=emilio
Special-case setting location.hash to the empty string

This avoids endless redirect loops on facebook and makes us compliant with https://github.com/whatwg/html/pull/1318.

---
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #10952
- [X] There are tests for these changes OR

<!-- 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/11934)
<!-- Reviewable:end -->
2016-06-29 17:39:58 -05:00