Commit graph

15311 commits

Author SHA1 Message Date
Bobby Holley
bf1a7d243f Remove the dependency of parallel DOM traversal and style calculation on LayoutContext. 2016-01-06 18:24:18 -08:00
Bobby Holley
ebc5eb1b98 Make parallel DOM traversal and style calculation operate on TNode instead of LayoutNode. 2016-01-06 17:58:53 -08:00
Patrick Walton
154f970b5b Remove the fontgroup address cache.
It's not clear to me that it actually helps. Moreover, it's wrong and
results in fonts randomly changing e.g. on GitHub when mousing over
elements.
2016-01-06 17:06:59 -08:00
David Raifaizen
76f689cd06 Changed blob to use DataSlice with Arc in order to limit wasteful copying of byte vector 2016-01-06 19:41:52 -05:00
bors-servo
238a8786de Auto merge of #9029 - KiChjang:event-source, r=jdm
Implement EventSource and update test expectations

Partial #8925.

Most of the processing model for `EventSource` are unimplemented because we currently don't have a fetch implementation.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9029)
<!-- Reviewable:end -->
2016-01-07 00:38:25 +05:30
Patrick Walton
fdb791bdd5 layout: Take the value of background-clip into account when
determining the painting area for background images and gradients.

See: https://drafts.csswg.org/css-backgrounds-3/#the-background-clip
2016-01-06 10:39:51 -08:00
Patrick Walton
c9242309cb tests: Make linear_gradients_lengths_a.html and
`linear_gradients_lengths_ref.html` use `background-clip: content-box`.

The tests clearly expect the background painting area to have width of
100px, but since the default value of `background-clip` is `border-box`
and the boxes in question have a 1-pixel border the background painting
areas are actually 102px. This commit fixes the tests so that the width
of the background painting area is actually 100px.
2016-01-06 10:39:45 -08:00
bors-servo
f4a2fe9501 Auto merge of #9164 - paulrouget:updatePipelineURL, r=jdm
update pipeline url after redirections

`pipeline.url` is not updated after a redirection.

Fixes #8902

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9164)
<!-- Reviewable:end -->
2016-01-06 21:53:41 +05:30
bors-servo
99689bc6b0 Auto merge of #9165 - KiChjang:extract-response, r=Ms2ger
Extract Response from net_traits/lib.rs

This puts them in a new file located at `net_traits/response.rs`.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9165)
<!-- Reviewable:end -->
2016-01-06 20:17:05 +05:30
Keith Yeung
05dc512055 Extract Response from net_traits/lib.rs 2016-01-06 09:45:22 -05:00
Paul Rouget
268e285439 update pipeline url after redirections 2016-01-06 06:49:49 +01:00
bors-servo
64e968d8bc Auto merge of #9119 - TheKK:input_element_size, r=eefriedman
Parse size attribute of HTMLInputElemnt correctly

Should fix #8773

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9119)
<!-- Reviewable:end -->
2016-01-06 08:57:19 +05:30
Ying-Ruei Liang (KK)
74905f0f3c Invalid type attribute for HTMLInputElement now fallback as text type (fixes #8773) 2016-01-06 10:56:00 +08:00
Keith Yeung
1d62db405e Implement EventSource and update test expectations 2016-01-05 21:47:43 -05:00
bors-servo
25c2b7dec6 Auto merge of #9161 - frewsxcv:cleanup-match, r=jdm
Reduce 'match' body, remove unnecessary clone

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9161)
<!-- Reviewable:end -->
2016-01-06 08:02:33 +05:30
Corey Farwell
639fbc2275 Reduce 'match' body, remove unnecessary clone 2016-01-05 16:55:37 -08:00
bors-servo
8b855e4456 Auto merge of #9160 - nerith:spec, r=jdm
Update FileAPI spec links

The links in components/script/dom/blob.rs were pointing to the old location.

Fixes #9157.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9160)
<!-- Reviewable:end -->
2016-01-06 03:04:21 +05:30
bors-servo
e3dd36f0f3 Auto merge of #9159 - saneyuki:9158, r=jdm
gfx: handle the ResponseAction::ResponseComplete error case in font_cache_task

Fix #9158

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9159)
<!-- Reviewable:end -->
2016-01-06 02:29:26 +05:30
Brandon Fairchild
b5e121a312 Update FileAPI spec links
The links in components/script/dom/blob.rs were pointing
to the old location.

Fixes #9157.
2016-01-05 15:48:30 -05:00
bors-servo
4ab1cdece6 Auto merge of #8970 - ecoal95:webgl-array-buffer-data, r=jdm
webgl: Make a general way to get data from a JS array buffer view

This fixes an invalid length being reported from
`float32_array_to_slice` (which used the byte length), and also to
generalize getting data from a JS array buffer view, to reduce code
duplication.

The pending type safety issues, like where we could send a `UInt16Array`
where we expect a `Float32` one, should be solved by IDL bindings in
some cases, like `uniform[n]fv` or `vertexAttrib[n]fv`, and with extra
checks in others, like in the pending `texImage2D(..., ArrayBufferView)`.

r? @jdm

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8970)
<!-- Reviewable:end -->
2016-01-06 01:54:16 +05:30
Tetsuharu OHZEKI
574f934314 gfx: handle the ResponseAction::ResponseComplete error case in font_cache_task 2016-01-06 04:59:10 +09:00
bors-servo
1ea2341a5d Auto merge of #9153 - nikkisquared:http_network_fetch, r=jdm
Partial implementation of HTTP-network Fetch step

I've been working on making a partial implementation of HTTP-network Fetch[1] so that I can begin testing the entire Fetch protocol as implemented in Servo. The empty steps are intentionally so, including the incomplete Step 5, which I started without realizing I don't need to do basic testing.

[1] https://fetch.spec.whatwg.org/#http-network-fetch

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9153)
<!-- Reviewable:end -->
2016-01-06 00:46:17 +05:30
Nikki
af310f7874 minimal implementation of http_network_fetch for testing purposes 2016-01-05 12:14:07 -07:00
bors-servo
e94a530597 Auto merge of #8565 - gsnedders:more_css_module_tests, r=larsbergstrom
Enable more CSS module tests

As noted, this excludes Fonts, on the assumption #8374 will land first. This should get us testing every property we support, I think.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8565)
<!-- Reviewable:end -->
2016-01-05 15:48:35 +05:30
bors-servo
b7a3906664 Auto merge of #9128 - adrianheine:patch-1, r=jdm
Add autoconf to build dependencies

`autoheader` from `autoconf` was necessary for the build on my system.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9128)
<!-- Reviewable:end -->
2016-01-05 06:00:30 +05:30
Geoffrey Sneddon
5845a60aaf Reorder CSS tests to be in alphabetical order. 2016-01-04 21:46:53 +00:00
Geoffrey Sneddon
756d745cb8 Enable CSS tests for all modules we partially support.
(Except Fonts, on the assumption #8374 will land first.)
2016-01-04 21:46:52 +00:00
bors-servo
ea4206e362 Auto merge of #9150 - mbrubeck:run-android, r=larsbergstrom
Make "mach run" use --android by default when enabled in .servobuild

When `.servobuild` has `android = true`, the `mach run` command should default to running on Android.

r? @larsbergstrom

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9150)
<!-- Reviewable:end -->
2016-01-05 01:22:53 +05:30
Matt Brubeck
b9fc84ddcc Use build.android config in mach run 2016-01-04 10:52:06 -08:00
bors-servo
f3075d1319 Auto merge of #9110 - bholley:hoist_matching_and_animation, r=SimonSapin
Hoist the rest of css/matching.rs (and its dependencies) into style/.

Slowly but surely.

This goes on top of #9103.

Fixes #9103.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9110)
<!-- Reviewable:end -->
2016-01-04 23:34:35 +05:30
Bobby Holley
513a75d86a Hoist the rest of css/matching.rs into style/. 2016-01-04 09:41:31 -08:00
Bobby Holley
94b0789a5f Remove the dependency of css/matching.rs on SharedLayoutContext. 2016-01-04 09:38:04 -08:00
Bobby Holley
08f2a24552 Remove the dependency of css/matching.rs on concrete RestyleDamage.
We can make this easier by inlining helper method implementations in the traits themselves,
which makes the code more compact as a nice side-effect.
2016-01-04 09:38:04 -08:00
Bobby Holley
27987c3bb4 Introduce a concept of restyle damage to the style system.
We can't hoist RestyleDamage itself, because it's very layout-dependent. But this
should be enough to let us hoist the things we need.
2016-01-04 09:38:04 -08:00
Bobby Holley
947134949a Refactor parallel dom traversal to be agnostic to the processing steps themselves. 2016-01-04 09:38:04 -08:00
Emilio Cobos Álvarez
d30f05554b conversions: Make a is_type_compatible method for ArrayBufferViews. 2016-01-04 15:51:01 +01:00
Emilio Cobos Álvarez
43d395a682 conversions: Add a checked version to array_buffer_view_data 2016-01-04 15:29:24 +01:00
Adrian Heine
4069645729 Don't shadow family_name
The argument to `local()` is not a font family, but the name of a
»single font face within a larger family« according to
http://www.w3.org/TR/css3-fonts/#src-desc.

The previous implementation of `FontCache::run` would panic if the argument to
`local()` would not be the name of a font family present in `self.web_families`.
That happened since `FontCacheTask` tried to use the argument to `local()` as a
key for `self.web_families`, although it previously correctly initialized a
value for the `family` field of the `AddWebFont` command.
2016-01-04 15:23:27 +01:00
Adrian Heine
3f61d63e72 Add failing test for FontCacheTask::add_web_font
This test tries to add a web font to the `FontCacheTask`. The added web font
corresponds to the following CSS font definition:

```
@font-face {
  font-family: "test family";
  src: local(test font face);
}
```

This test fails, since `FontCacheTask` tries to get the value for the key
"test font face" from `self.web_families`, but previously initialized
a value for the key "test family".
2016-01-04 15:23:26 +01:00
Emilio Cobos Álvarez
9ad49c8aa1 conversion: Extrapolate array_buffer_view_data
And use it instead of the raw jsapi calls.
2016-01-04 15:19:29 +01:00
Emilio Cobos Álvarez
4d32444bf7 webgl: Unify accessing ArrayBufferView objects
This fixes an invalid length being reported from
float32_array_to_slice (which used the byte length), and also to
generalize getting data from a JS array buffer view, to reduce code
duplication.

The pending type safety issues, like where we could send a UInt16Array
where we expect a Float32 one, should be solved by IDL bindings in
some cases, like uniform[n]fv or vertexAttrib[n]fv, and with extra
checks in others, like in the pending texImage2D(..., ArrayBufferView).
2016-01-04 13:58:49 +01:00
bors-servo
5ad9207a99 Auto merge of #9076 - Ms2ger:viewport, r=jgraham
Add support for viewport size adjustments in wptrunner.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9076)
<!-- Reviewable:end -->
2016-01-04 14:46:16 +05:30
Ms2ger
796c77df33 Move viewport_percentage_vw_vh_a.html and viewport_percentage_vw_vh_b.html to wpt reftests. 2016-01-04 10:15:59 +01:00
Ms2ger
797b21eaf1 Move viewport_percentage_vmin_vmax_a.html and viewport_percentage_vmin_vmax_b.html to wpt reftests. 2016-01-04 09:58:15 +01:00
Ms2ger
1619a908b3 Add support for viewport size adjustments in wptrunner. 2016-01-04 09:58:14 +01:00
bors-servo
ee6c5ae2fb Auto merge of #9138 - nox:enable-canvas-linestyles-linecap-001, r=frewsxcv
Enable /canvas_linestyles_linecap_001.htm (fixes #5714)

CanvasDrawingStyles.lineCap was implemented and the test passes now.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9138)
<!-- Reviewable:end -->
2016-01-04 02:30:15 +05:30
bors-servo
1096183cce Auto merge of #9146 - GuillaumeGomez:generate_partial_eq, r=nox
Generate PartialEq automatically

r? @nox

Fixes #8804

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9146)
<!-- Reviewable:end -->
2016-01-04 00:37:10 +05:30
Guillaume Gomez
99ac1a81cc Generate PartialEq automatically 2016-01-03 18:07:09 +01:00
bors-servo
99e0cc76d7 Auto merge of #9125 - nox:enable-type-change-state, r=jdm
Enable again /the-input-element/type-change-state.html.ini (fixes #3657)

It doesn't seem to intermittently timeout anymore.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9125)
<!-- Reviewable:end -->
2016-01-03 21:26:02 +05:30
bors-servo
efb145e744 Auto merge of #9139 - nox:enable-canvas-compositing-globalcompositeoperation-001, r=frewsxcv
Enable canvas_compositing_globalcompositeoperation_001.htm (fixes #5803)

This always pass now.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9139)
<!-- Reviewable:end -->
2016-01-03 20:32:34 +05:30