Commit graph

14795 commits

Author SHA1 Message Date
Ms2ger
0cc75b731c Remove some unused imports. 2015-12-20 22:26:07 +01:00
bors-servo
5b91daa923 Auto merge of #9025 - k-cross:master, r=nox
touchlist dom api #8225

Sorry for the wait, I just finished my finals.

Fixes #8225

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9025)
<!-- Reviewable:end -->
2015-12-21 01:19:41 +05:30
bors-servo
62b11a28fa Auto merge of #9027 - vwvww:cmrf2, r=jdm
Moved ScriptToCompositorMsg enum and EventResult enum to script_traits

Moved ScriptToCompositorMsg enum and EventResult enum to script_traits
resolving issue #8835.

(Need to be checked)
Variants in ScriptToCompositorMsg enum and EventResult enum had no doc
comment. I found some with grep command and copied it, and others I
wrote some doc comments manually to pass the build system. It needs to
be checked whether the doc comment is proper or not.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9027)
<!-- Reviewable:end -->
2015-12-21 00:30:03 +05:30
Beomjin Kim
82f39d5a1c Moved ScriptToCompositorMsg enum and EventResult enum to script_traits
Moved ScriptToCompositorMsg enum and EventResult enum to script_traits

(rebased)
2015-12-20 11:53:41 +09:00
k-cross
ae8d3f67c8 touchlist dom api #8225 2015-12-19 12:10:18 -08:00
bors-servo
95eabdb4db Auto merge of #9022 - mskrzypkows:PaintMsg_refactoring, r=jdm
Move PaintMsg enum to gfx_traits #8844

re-PR of https://github.com/servo/servo/pull/9007, because homu seems stuck

Fixes #8844.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9022)
<!-- Reviewable:end -->
2015-12-20 01:34:26 +05:30
bors-servo
8c87477138 Auto merge of #9026 - Manishearth:attack-of-the-clones, r=Ms2ger
Remove clone in collect_text_contents

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9026)
<!-- Reviewable:end -->
2015-12-20 00:14:02 +05:30
Beomjin Kim
2637d30c2a Moved ScriptToCompositorMsg enum and EventResult enum to script_traits
Moved ScriptToCompositorMsg enum and EventResult enum to script_traits
resolving issue #8835.
2015-12-20 00:31:33 +09:00
bors-servo
2a3a7a73b5 Auto merge of #9020 - mbrubeck:mach-run, r=larsbergstrom
Update `mach run` to use android_params file

Currently it sets the SERVO_URL variable, which Servo no longer supports.

r? @larsbergstrom

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9020)
<!-- Reviewable:end -->
2015-12-19 03:08:28 +05:30
bors-servo
f9141ef2db Auto merge of #9019 - magopian:9002-use-url.join, r=SimonSapin
Use Url.join instead of UrlParser.base_url(...).parse (#9002)

Fix issue #9002

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9019)
<!-- Reviewable:end -->
2015-12-19 01:58:56 +05:30
Manish Goregaokar
ef5de267d9 Remove clone in collect_text_contents 2015-12-19 01:06:16 +05:30
bors-servo
21277b1fa0 Auto merge of #8959 - nox:domtokenlist-stringifier, r=Ms2ger
Fix DOMStringMap's stringifier behaviour according to the spec

It should just return its associated attribute's value, if any.

https://github.com/whatwg/dom/issues/105

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8959)
<!-- Reviewable:end -->
2015-12-19 00:51:17 +05:30
Matt Brubeck
d7ba536ad6 Update mach run to use android_params file 2015-12-18 10:09:48 -08:00
bors-servo
6ba4ef22fa Auto merge of #8882 - GuillaumeGomez:dom_quad, r=nox
Add DOMQuad element

cc  @nox

Part of #8821

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8882)
<!-- Reviewable:end -->
2015-12-18 23:37:12 +05:30
Guillaume Gomez
823e1b96c3 Add DOMQuad element 2015-12-18 17:37:10 +01:00
Anthony Ramine
e7a9f44df9 Fix DOMStringMap's stringifier behaviour according to the spec
It should just return its associated attribute's value, if any.

https://github.com/whatwg/dom/issues/105
2015-12-18 17:03:49 +01:00
Mathieu Agopian
11234f5370 Use Url.join instead of UrlParser.base_url(...).parse (#9002) 2015-12-18 17:02:41 +01:00
bors-servo
6764cf0ee9 Auto merge of #9017 - jinwooahn:spec, r=jdm
Fix a logical operator to match the spec (#9016)

Fix issue #9016

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9017)
<!-- Reviewable:end -->
2015-12-18 20:06:40 +05:30
Jinwoo Ahn
2eac20861d Fix a logical operator to match the spec 2015-12-18 21:25:04 +09:00
bors-servo
22e1ccae37 Auto merge of #9010 - Ms2ger:warnings, r=jdm
Silence warnings in htmlformelement.rs.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9010)
<!-- Reviewable:end -->
2015-12-18 15:16:32 +05:30
bors-servo
6490d1e1c5 Auto merge of #9013 - antrik:debug-fontgroup, r=Ms2ger
Derive Debug for FontGroup and Homu is a bully

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9013)
<!-- Reviewable:end -->
2015-12-18 14:23:15 +05:30
bors-servo
2ef972b53b Auto merge of #8886 - paulrouget:freeze-pipeline-iframe, r=mbrubeck,jdm
Freeze old pipeline in iframes

Fixes https://github.com/servo/servo/issues/8673 and part of https://github.com/servo/servo/issues/8674

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8886)
<!-- Reviewable:end -->
2015-12-18 13:37:25 +05:30
Maciej Skrzypkowski
3f7f323556 Move PaintMsg enum to gfx_traits #8844 2015-12-18 08:33:32 +01:00
Paul Rouget
f0b25e16f7 delay page freeze/thaw until it's created 2015-12-18 08:27:01 +01:00
Paul Rouget
abc127ac23 basic iframe.goBack() test 2015-12-18 08:26:59 +01:00
Paul Rouget
e7449d541f Freeze old pipeline in iframes 2015-12-18 08:26:57 +01:00
bors-servo
f3e921a584 Auto merge of #9011 - asajeffrey:add_valgrind_suppression_file, r=metajack
Add a valgrind suppression file.

Adding a suppression file reduces the number of false positives from memcheck. Run with:
```
valgrind --suppressions=etc/valgrind-memcheck.supp servo ...
```
For the moment, this just switches off the warnings generated by jemalloc.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9011)
<!-- Reviewable:end -->
2015-12-18 08:40:47 +05:30
bors-servo
6afed16565 Auto merge of #9005 - frewsxcv:servo-egl, r=jdm
Update Cargo.{lock,toml} to reference servo-egl instead of egl

The crate name was changed in this commit:

ebbd1cff89

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9005)
<!-- Reviewable:end -->
2015-12-18 07:07:30 +05:30
bors-servo
8dc4b8dcb9 Auto merge of #8993 - nox:small-codegen-slimming, r=Ms2ger
Slightly reduce the output of codegen

Interfaces which we know are never instantiated can generate less code.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8993)
<!-- Reviewable:end -->
2015-12-18 06:35:15 +05:30
bors-servo
44588cdc98 Auto merge of #8967 - jinwooahn:comments, r=jdm
preflight_fetch steps comments out of date with spec #8941

Fix comments. Resolves #8941

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8967)
<!-- Reviewable:end -->
2015-12-18 05:55:23 +05:30
bors-servo
63923bc7c9 Auto merge of #8825 - jmr0:master, r=jdm
adding initial support for websocket subprotocol negotation

Addresses #8177

I also noticed some bugs/gaps (and at least one of my TODO's can be an E-Easy)

cc @jdm

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8825)
<!-- Reviewable:end -->
2015-12-18 05:23:15 +05:30
bors-servo
82e73e7a5f Auto merge of #8723 - dzbarsky:origin_clean, r=dzbarsky
Add origin-clean flag tracking for canvas

The resulting failures are because we aren't doing origin-comparison correctly (I bet the image ends up with an empty hostname or something). We could probably hack around that if we care, but it should just go away when we get the origin checking right.

r? @jdm

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8723)
<!-- Reviewable:end -->
2015-12-18 04:26:59 +05:30
David Zbarsky
b8e9064fe6 Add origin-clean flag tracking for canvas 2015-12-17 11:40:38 -08:00
Alan Jeffrey
5ca06b5dcb Add a valgrind suppression file.
Adding a suppression file reduces the number of false positives from memcheck. Run with:
```
valgrind --suppressions=etc/valgrind-memcheck.supp servo ...
```
For the moment, this just switches off the warnings generated by jemalloc.
2015-12-17 10:36:03 -06:00
Ms2ger
50ff54c37c Silence warnings in htmlformelement.rs. 2015-12-17 16:03:22 +01:00
jmr0
7d0bede8ba adding initial support for websocket subprotocol negotation 2015-12-17 09:17:15 -05:00
bors-servo
9570b51565 Auto merge of #8618 - paulrouget:reload, r=jdm
Browser API: implement iframe.reload()

fixes #8575

The implementation is naive, and doesn't support the `hardreload` parameter.
And for the test, I'm not sure how else I can test the reload.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8618)
<!-- Reviewable:end -->
2015-12-17 19:45:44 +05:30
bors-servo
c6ae32abdd Auto merge of #8612 - glennw:pending-frames, r=jdm
Add test to constellation to avoid writing reftest image if there are pending frames.

This changes several tests that contain <iframe></iframe> from FAIL to TIMEOUT. This is correct
since there is a bug that prevents these iframes from ever rendering.

~~~There are also a few previous FAILs that changed to OK. These may be intermittents or they
may genuinely be fixed by this change.~~~

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8612)
<!-- Reviewable:end -->
2015-12-17 16:19:16 +05:30
jinwoo.ahn
316a619172 preflight_fetch steps comments out of date with spec #8941
Fixed the comments according to the spec.
2015-12-17 15:05:51 +09:00
bors-servo
338f66003e Auto merge of #8995 - pcwalton:measure-text-shaping, r=mbrubeck
gfx: Measure text shaping time in the time profiler.

r? @mbrubeck

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8995)
<!-- Reviewable:end -->
2015-12-17 10:57:29 +05:30
bors-servo
67c3cb3707 Auto merge of #8990 - pcwalton:advance-for-range-optzns, r=mbrubeck
Optimize `TextRun::advance_for_range`.

The combined effects of these optimizations move `advance_for_range` from #1 in the layout profile on all sites I tested to #2, #3, or #4, depending on the site.

r? @mbrubeck

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8990)
<!-- Reviewable:end -->
2015-12-17 09:47:03 +05:30
bors-servo
8c09154d59 Auto merge of #8977 - frewsxcv:immediate-tidy-errors, r=jdm
Print tidy errors as they happen instead of all-at-once

Related to #8969

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8977)
<!-- Reviewable:end -->
2015-12-17 09:09:59 +05:30
bors-servo
0e4ae508e2 Auto merge of #8975 - mbrubeck:touch-refactor, r=glennw
Add pinch zoom gesture for touch-screen devices

r? @glennw

Notes:
* Our layer system doesn't yet support zooming `<iframe mozbrowser>` separately, so this can only zoom the root window.
* There are some issues that cause some tiles to disappear while zooming. I hope this will be fixed by WebRender.
* Sorry this is mostly in one giant commit. I did the refactoring and adding new functionality in tandem, and didn't really end up with logical or compilable intermediate commits. Hopefully the new code is easier to read than the old stuff.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8975)
<!-- Reviewable:end -->
2015-12-17 08:17:38 +05:30
Glenn Watson
b670430cb2 Add test to constellation to avoid writing reftest image if there are pending frames.
Also change when pipelines become active.

This makes the constellation activate a pipeline as the current frame
when it is ready to do initial reflow, rather than when it is ready
to paint.

This fixes a number of intermittent failures that could previously occur
if an iframe was not visible - which would mean it was never moved from
a pending frame in the constellation to an active frame.

(It happens that webrender exposes these intermittents as permanent failures).
2015-12-17 12:30:19 +10:00
Corey Farwell
2cf29cbcdb Update Cargo.{lock,toml} to reference servo-egl instead of egl
The crate name was changed in this commit:

ebbd1cff89
2015-12-16 20:39:55 -05:00
bors-servo
d9bf1ba616 Auto merge of #8747 - KiChjang:form-interactive-validation, r=Manishearth
Implement basic framework for static and interactive validation on forms

This came along when I was trying to do #3648. I was about to do form validation as well, but then I saw it was actually intended to be a student project. Hopefully I didn't step on too many toes on this one; I did leave most of the important bits out (the fun bits I believe, e.g. interactive validation doesn't actually interactively validate anything right now).

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8747)
<!-- Reviewable:end -->
2015-12-17 05:41:45 +05:30
Keith Yeung
5ed03a5466 Remove big blob of commented code 2015-12-16 10:36:13 -08:00
Matt Brubeck
40776b9492 Add queuing of pinch zoom events 2015-12-16 10:03:53 -08:00
Matt Brubeck
eb2a37d41c Refactor touch handling and add pinch zoom gesture 2015-12-16 10:03:49 -08:00
bors-servo
bc7158310c Auto merge of #8903 - KiChjang:layout-reporter-redirect, r=nox
fix for Layout memory reporter uses pre-redirect url

Rebase of #7612.

Fixes #6872.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8903)
<!-- Reviewable:end -->
2015-12-16 23:18:30 +05:30