aboutsummaryrefslogtreecommitdiffstats
path: root/components
Commit message (Collapse)AuthorAgeFilesLines
* constellation: Minor refactoring to aid legibility.Emilio Cobos Álvarez2016-09-191-21/+27
| | | | | | | | | Two things changed, on one hand, avoid a dumb if chain that could be more idiomatically written with a match expression, and also avoiding use map() to change state. In general I'm pretty surprised for our lack of error reporting in this critical code, but that's not the purpose of this PR.
* Auto merge of #13306 - Jenselme:13247-decode-utf8-with-utf8-lossy, r=Ms2gerbors-servo2016-09-191-7/+7
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Decode UTF-8 with from_utf8_lossy in DedicatedWorkerGlobalScope <!-- 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 #13247 <!-- Either: --> - [X] There are tests for these changes: `./mach test-wpt /workers/semantics/encodings/004.worker` passes <!-- 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="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13306) <!-- Reviewable:end -->
| * Decode UTF-8 with from_utf8_lossy in DedicatedWorkerGlobalScopeJulien Enselme2016-09-191-7/+7
| |
* | Auto merge of #13314 - Phrohdoh:update-string_cache-to29patch, r=KiChjangbors-servo2016-09-191-11/+11
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update string_cache to 0.2.29 <!-- Please describe your changes on the following line: --> Update string-cache from 0.2.28 to 0.2.29 which includes `minlength` for #13313. --- <!-- 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 this is only adding a single string to string-cache (ref https://github.com/servo/string-cache/pull/172) <!-- 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="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13314) <!-- Reviewable:end -->
| * | Update string_cache to 0.2.29Taryn Hill2016-09-181-11/+11
| |/
* | Auto merge of #13307 - bubbles231:code_refactor, r=Manishearthbors-servo2016-09-1890-485/+485
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Code refactoring <!-- Please describe your changes on the following line: --> The code was refactored to follow Rust naming conventions better. --- <!-- 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 help fix #12379. <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because logic was not changed. <!-- 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="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13307) <!-- Reviewable:end -->
| * | Fix my typo on commit. Rest of the fixes will be here soon.Arthur Marble2016-09-181-4/+4
| | |
| * | Made fixes for PR.Arthur Marble2016-09-1812-108/+108
| | |
| * | One last refactoring before fixingArthur Marble2016-09-181-1/+1
| | |
| * | Most of the code refactoring needed to be done is done with this commit.Arthur Marble2016-09-1886-469/+469
| | |
| * | More code refactoring (exampleVar to example_var)Arthur Marble2016-09-185-38/+38
| | |
| * | Refactored a bluetooth* related files from codeStyle to code_style.Arthur Marble2016-09-185-32/+32
| | |
| * | Refactored a couple of files from codeStyle to code_style.Arthur Marble2016-09-173-10/+10
| | |
| * | Refactor isClosed_ to is_closed and relativeContentType to ↵Arthur Marble2016-09-171-7/+7
| | | | | | | | | | | | relative_content_type.
| * | Replace typeString with type_string.Arthur Marble2016-09-172-7/+7
| | |
| * | Refactored metaKey to meta_key where possibleArthur Marble2016-09-176-10/+10
| | |
| * | This commit refactors altKey to alt_key when possible.Arthur Marble2016-09-176-10/+10
| | |
| * | This commit refactors shiftKey to shift_key where possible.Arthur Marble2016-09-176-10/+10
| | |
| * | This commit refactors ctrlKey to ctrl_key wherever possible.Arthur Marble2016-09-176-10/+10
| |/
* | Auto merge of #13227 - jmr0:pipeline_visibility, r=jdmbors-servo2016-09-182-11/+22
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pipeline visibility logic change <!-- Please describe your changes on the following line: --> cc @paulrouget @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] These changes fix #13191 (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. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13227) <!-- Reviewable:end -->
| * altering pipeline visibility change logicjmr02016-09-172-11/+22
| |
* | implement ExtendableEvent as base type for ServiceWorker eventsRahul Sharma2016-09-177-14/+229
| |
* | Auto merge of #13304 - frewsxcv:ownership, r=jdmbors-servo2016-09-161-6/+6
|\ \ | |/ |/| | | | | | | | | | | Don't require `PathBuf` ownership if we don't need it. <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13304) <!-- Reviewable:end -->
| * Don't require `PathBuf` ownership if we don't need it.Corey Farwell2016-09-161-6/+6
| |
* | Auto merge of #12703 - sjmelia:12343_test_activation, r=Manishearthbors-servo2016-09-163-0/+37
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix Issue 12343: Add pref-gated methods and test for activatable element <!-- Please describe your changes on the following line: --> Add methods to activatable elements; gated by a preference, for use in servo-specific WPT tests. I cargo-culted over the "redundant check" from #12322, I don't really understand the possible exploit. Obvious flaw is the copypasta for each element. I understand that the webidl codegen will produce a trait for ActivatableElement, but I can't see how I can provide a default implementation for this? I guess otherwise could be an improvement to have each element just delegate to an Enter.../Exit... method on the Activatable trait? --- <!-- 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 #12343 (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. --> …states <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12703) <!-- Reviewable:end -->
| * Issue 12343: Add pref-gated methods and test for activatable element statesSteve Melia2016-08-083-0/+37
| |
* | Auto merge of #13294 - 6112:referrer-typo-fix, r=jdmbors-servo2016-09-165-47/+47
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix most typoes for: "referer" -> "referrer" Replace most uses of the word "referer" with "referrer", except for `hyper::header::Referer`. Also update the unit tests to compile & pass after those changes. --- <!-- 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 #13286 <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because they're only typo fixes. <!-- 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="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13294) <!-- Reviewable:end -->
| * | Fix most typoes for: "referer" -> "referrer"Nicolas2016-09-165-47/+47
| | |
* | | Auto merge of #12202 - peterjoel:issue_8509, r=noxbors-servo2016-09-166-2/+1201
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DOMMatrix and DOMMatrixReadOnly <!-- 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 #8509. <!-- 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/12202) <!-- Reviewable:end -->
| * | | DOMMatrix and DOMMatrixReadOnlyPeter2016-09-156-2/+1201
| | | | | | | | | | | | | | | | Updated test expectations
* | | | Auto merge of #13281 - gilbertw1:basic-auth-cache-clean, r=jdmbors-servo2016-09-163-8/+10
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update basic auth cache to key off of origin instead of url This pull request's primary purpose is to store basic auth credentials based on the url origin instead of the entire url. This fixes an issue where servo continuously prompts the user for credentials any time a basic auth secured resource is requested even though the user has already entered auth credentials for a different resource from the same origin. The test associated with this PR hides image redirects behind a python handler that requires basic authentication. The reference page loads two images by directly specifying the image to load, while the test page loads the two images using the basic auth redirect handler with only the first image tag providing auth credentials. I'd like to point a few specific items for review: * url::Origin does not derive ```Hash```, so I am using ```ascii_serialization``` as the cache key. This seems like a stable enough representation. * I've updated the http loader to store credentials not only on Success responses, but Redirect responses as well. I stumbled on this because nginx was redirecting 'test' -> 'test/' in my testing, and other browsers were storing the credentials on the redirect response vs. prompting for credentials a second time. * In the test I'm using a timeout to load the second image (without authentication), otherwise the order that the images were loaded was unpredictable. --- <!-- 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 related to these changes - [x] These changes fix #12095 (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13281) <!-- Reviewable:end -->
| * | | store basic auth in cache on redirect responseBryan Gilbert2016-09-151-1/+2
| | | |
| * | | updated basic auth cache to key off of url originBryan Gilbert2016-09-153-7/+8
| | | |
* | | | Auto merge of #13188 - canaltinova:matrix, r=Manishearthbors-servo2016-09-161-6/+744
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement matrix decomposition and interpolation <!-- Please describe your changes on the following line: --> Implement 2D and 3D matrix decomposition and interpolation for animated properties. 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 <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because there are no tests for it and we manually tested it. <!-- 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="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13188) <!-- Reviewable:end -->
| * | | | Add decomposition and interpolation of 3D matricesNazım Can Altınova2016-09-161-24/+504
| | | | |
| * | | | Move decompose and recompose functions to impl and implement interpolationsNazım Can Altınova2016-09-071-125/+177
| | | | |
| * | | | Implement recompose_matrix functionNazım Can Altınova2016-09-061-0/+44
| | | | |
| * | | | Implement Interpolate traits for matrix structsNazım Can Altınova2016-09-061-0/+75
| | | | |
| * | | | Implement decompose_matrix functionNazım Can Altınova2016-09-061-0/+87
| | | | |
* | | | | Auto merge of #13288 - glennw:update-wr-lazy-shaders-scroll, r=pcwaltonbors-servo2016-09-153-2/+17
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update webrender + shaders. <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13288) <!-- Reviewable:end -->
| * | | | Update webrender + shaders.Glenn Watson2016-09-163-2/+17
| | | | |
* | | | | Auto merge of #13280 - tizianasellitto:issue_13019, r=jdmbors-servo2016-09-151-0/+1
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make DOMTokenList Iterable Make DOMTokenList Iterable. - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #13019 - [X] There are tests for these changes (It adds tests/wpt/web-platform-tests/dom/lists/DOMTokenList-Iterable.html) <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13280) <!-- Reviewable:end -->
| * | | | Make DOMTokenList IterableTiziana Sellitto2016-09-151-0/+1
| | | | |
* | | | | Auto merge of #13184 - szeged:requestdevice-refactor, r=jdmbors-servo2016-09-154-91/+269
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Webbluetooth requestDevice refactor and update <!-- Please describe your changes on the following line: --> Refactor requestDevice function according to the specification changes. 1. Moved the `request_bluetooth_devices` algorithm out from the `requestDevice` function. 2. Two new members in `BluetoothRequestDeviceFilter` and one new member in `RequestDeviceOptions`. 3. Also added annotations to the related functions. Related links: https://webbluetoothcg.github.io/web-bluetooth/#device-discovery, https://webbluetoothcg.github.io/web-bluetooth/#dom-bluetooth-requestdevice, https://webbluetoothcg.github.io/web-bluetooth/#request-bluetooth-devices, https://webbluetoothcg.github.io/web-bluetooth/#matches-a-filter --- <!-- 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 #12614 <!-- Either: --> - [x] These changes do not require tests because , there is no WebBluetooth Test API implementation yet. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13184) <!-- Reviewable:end -->
| * | | | | Annotations for requestDevice, request_bluetooth_devices and matches_filter ↵zakorgyula2016-09-142-1/+88
| | | | | | | | | | | | | | | | | | | | | | | | functions
| * | | | | requestDevice updatezakorgyula2016-09-144-34/+108
| | | | | |
| * | | | | requestDevice refactorzakorgyula2016-09-141-78/+95
| | | | | |
* | | | | | Auto merge of #13279 - servo:sm, r=KiChjangbors-servo2016-09-151-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update js. <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13279) <!-- Reviewable:end -->
| * | | | | | Update js.Ms2ger2016-09-151-1/+1
| | |/ / / / | |/| | | |
* | | | | | Auto merge of #13278 - servo:rustup, r=jdmbors-servo2016-09-153-5/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update Rust to 1.13.0-nightly (6ffdda1ba 2016-09-14) <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13278) <!-- Reviewable:end -->