aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Update WR (shutdown api, clip mask segment optimization, render target sizing).Glenn Watson2017-01-121-2/+2
|
* Auto merge of #14963 - jdm:script_current_line, r=asajeffreybors-servo2017-01-1111-22/+61
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Report meaningful line numbers for inline script errors Rebased from #14661. --- - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12744 and partially #9604 - [X] There are tests for these changes <!-- 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/14963) <!-- Reviewable:end -->
| * Store parser's current line when script elements are created.karenher2017-01-1111-22/+61
| | | | | | | | | | | | Use the newly stored line as the starting line number when evaluating JS. This ensures that inline scripts will report errors with meaningful line numbers.
* | Auto merge of #14964 - canaltinova:ok-try, r=Wafflespeanutbors-servo2017-01-111-4/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove unnecessary try!'s <!-- Please describe your changes on the following line: --> These try!'s are unnecessary here. We can remove them. --- <!-- 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's just refactoring <!-- 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/14964) <!-- Reviewable:end -->
| * | Remove unnecessary try!'sNazım Can Altınova2017-01-111-4/+4
| | |
* | | Auto merge of #14291 - fiji-flo:text-input-select, r=pcwaltonbors-servo2017-01-119-50/+219
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Position insertion point in input field with mouse <!-- Please describe your changes on the following line: --> Implements cursor positioning in input elements (text/password) via mouse. The related issue is #10083 but is only covered partly. This PR does **not** cover: * positioning in textarea elements via mouse * text selection in input/textarea elements via mouse --- <!-- 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 #10083 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because I can't think of a way to test mouse interaction in the current test pipeline. <!-- 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/14291) <!-- Reviewable:end -->
| * | | Position insertion point in input field with mouseFlorian Merz2017-01-119-50/+219
| | | |
* | | | Auto merge of #14924 - ioctaptceb:webgl-blendFunc-tests, r=jdmbors-servo2017-01-112-48/+33
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Webgl blend func tests <!-- Please describe your changes on the following line: --> Add a function that validates the conditions for [Constant Color Blend](https://www.khronos.org/registry/webgl/specs/latest/1.0/#CONSTANT_COLOR_BLEND) in the WebGL specs. ./mach run -r tests/wpt/web-platform-tests/webgl/conformance-1.0.3/conformance/misc/webgl-specific.html --- <!-- 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: --> - [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/14924) <!-- Reviewable:end -->
| * | | | Add a check to BlendFunc for invalid constantioctaptceb2017-01-112-48/+33
| | |_|/ | |/| | | | | | | | | | combinations
* | | | Auto merge of #13972 - shravan-achar:master, r=Manishearth,emilio,jdmbors-servo2017-01-118-2/+466
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ImageMaps: Implemented support for parsing coord attribute to a shape… <!-- Please describe your changes on the following line: --> Image Maps: (Part 1) Implemented support for parsing coord attribute to a shape object. Implemented a hit_test method to see if a point is within the area. Tests for constructors and hit_test included --- <!-- 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. --> … object in HTMLAreaElement <!-- 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/13972) <!-- Reviewable:end -->
| * | | | Make area structs accessible from area testsManish Goregaokar2017-01-112-2/+5
| | | | |
| * | | | HTML image map supportbachar2017-01-117-2/+463
| | |/ / | |/| |
* | | | Auto merge of #14961 - Ms2ger:DevicePixel, r=noxbors-servo2017-01-1113-27/+13
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move DevicePixel to script_traits. <!-- 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/14961) <!-- Reviewable:end -->
| * | | | Move DevicePixel to script_traits.Ms2ger2017-01-1113-27/+13
| | |_|/ | |/| |
* | | | Auto merge of #14911 - charlesvdv:alternate-stylesheet, r=cbrewsterbors-servo2017-01-115-16/+23
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handle properly alternate stylesheet <!-- Please describe your changes on the following line: --> Alternate stylesheet are now fetched and then desactivated instead of ignoring them. I don't know if tests are required to check if the stylesheet is correctly loaded and disabled ? <!-- 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 #14881 (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="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/14911) <!-- Reviewable:end -->
| * | | Update metadata testsCharles Vandevoorde2017-01-111-5/+2
| | | |
| * | | Modify test to check if @import stylesheet get applied in a alternate stylesheetcharlesvdv2017-01-102-0/+5
| | | |
| * | | Handle properly alternate stylesheetcharlesvdv2017-01-102-11/+16
| | | |
* | | | Auto merge of #14959 - nox:pending-script, r=Ms2gerbors-servo2017-01-118-94/+205
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce PendingScript <!-- 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/14959) <!-- Reviewable:end -->
| * | | | Introduce PendingScriptAnthony Ramine2017-01-118-82/+193
| | | | | | | | | | | | | | | | | | | | | | | | | This moves scripts' loading results in Document, instead of maintaining them behind a DOMRefCell in each HTMLScriptElement.
| * | | | Rename ScriptOrigin to ClassicScriptAnthony Ramine2017-01-111-10/+10
| | | | |
| * | | | Introduce Document::has_pending_parsing_blocking_scriptAnthony Ramine2017-01-112-5/+5
| | | | |
| * | | | Do not pass an Option to Document::set_pending_parsing_blocking_scriptAnthony Ramine2017-01-102-5/+5
| | |_|/ | |/| |
* | | | Auto merge of #14956 - upsuper:geometry-boxes, r=Manishearthbors-servo2017-01-114-4/+24
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend supported geometry box value for geckolib <!-- Please describe your changes on the following line: --> 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 - [ ] 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="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/14956) <!-- Reviewable:end -->
| * | | | Extend supported geometry box value for geckolibXidorn Quan2017-01-114-4/+24
| | |_|/ | |/| |
* | | | Auto merge of #14929 - servo:update-image, r=mbrubeckbors-servo2017-01-116-21/+43
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | Update image. <!-- 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/14929) <!-- Reviewable:end -->
| * | | Update image.Ms2ger2017-01-116-21/+43
| | | |
* | | | Auto merge of #14939 - upsuper:add-aliases, r=emilio,Manishearthbors-servo2017-01-1020-41/+101
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add aliases for geckolib for supported properties <!-- Please describe your changes on the following line: --> This fixes ~7.5k failures in style system mochitests. 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 - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [x] These changes do not require tests because it is for geckolib only <!-- 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/14939) <!-- Reviewable:end -->
| * | | Add other property aliasesXidorn Quan2017-01-1116-34/+68
| | | |
| * | | Add gecko leagcy logical properties aliasesXidorn Quan2017-01-114-3/+16
| | | |
| * | | Add prefixes parameter for propertiesXidorn Quan2017-01-111-2/+13
| | | |
| * | | Fix alias handling in properties.makoXidorn Quan2017-01-111-2/+4
|/ / /
* | | Auto merge of #14943 - nox:external-script-kind, r=jdmbors-servo2017-01-101-66/+74
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce ExternalScriptKind to simplify HTMLScriptElement::prepare <!-- 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/14943) <!-- Reviewable:end -->
| * | | Introduce ExternalScriptKind to simplify HTMLScriptElement::prepareAnthony Ramine2017-01-101-43/+44
| | | |
| * | | Fix steps numbering of HTMLScriptExecute::prepareAnthony Ramine2017-01-101-35/+42
| | | |
* | | | Auto merge of #14942 - servo:warnings, r=noxbors-servo2017-01-102-4/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix some warnings. <!-- 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/14942) <!-- Reviewable:end -->
| * | | | Fix some warnings.Ms2ger2017-01-102-4/+1
| | | | |
* | | | | Auto merge of #14938 - servo:gfx-traits, r=noxbors-servo2017-01-109-99/+30
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Various cleanup around gfx_traits. <!-- 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/14938) <!-- Reviewable:end -->
| * | | | Remove the rustc-serialize dependency from gfx_traits.Ms2ger2017-01-103-5/+2
| | | | |
| * | | | Remove the rustc-serialize dependency from gfx.Ms2ger2017-01-104-5/+2
| | | | |
| * | | | Replace gfx_traits::ScrollPolicy by webrender_traits::ScrollPolicy.Ms2ger2017-01-104-22/+8
| | | | |
| * | | | Remove unused LayerKind and LayerType enums.Ms2ger2017-01-101-18/+0
| | | | |
| * | | | Move LayerPixel to compositing.Ms2ger2017-01-102-8/+8
| | | | |
| * | | | Move FrameTreeId to compositing.Ms2ger2017-01-102-10/+10
| | | | |
| * | | | Remove some unused StackingContextId methods.Ms2ger2017-01-101-26/+0
| | | | |
| * | | | Remove unused ScrollRootId::to_stacking_context_id.Ms2ger2017-01-101-5/+0
| | |/ / | |/| |
* | | | Auto merge of #14603 - mrobinson:collect-scoll-roots, r=emiliobors-servo2017-01-1017-272/+224
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rework the way scroll roots are collected <!-- 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 - [x] These changes do not require tests because they should not change behavior. <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Collect scroll roots during the collect_stacking_context phase instead of during display list construction. This will be useful in order to collect containing block scroll roots as well as to give scroll roots sequential ids in the future. This change also pulls stacking context children out of the StackingContext struct itself, which should reduce very slightly the memory used by the finished display list. This also simplifies the DisplayListBuilder because it no longer has to maintain a stack of ScrollRootIds and StackingContextIds and can instead just rely on the program stack. <!-- 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/14603) <!-- Reviewable:end -->
| * | | | Rework the way scroll roots are collectedMartin Robinson2017-01-1017-272/+224
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Collect scroll roots during the collect_stacking_context phase instead of during display list construction. This will be useful in order to collect containing block scroll roots as well as to give scroll roots sequential ids in the future. This change also pulls stacking context children out of the StackingContext struct itself, which should reduce very slightly the memory used by the finished display list. This also simplifies the DisplayListBuilder because it no longer has to maintain a stack of ScrollRootIds and StackingContextIds and can instead just rely on the program stack.
* | | | Auto merge of #14928 - nox:insert, r=SimonSapinbors-servo2017-01-104-30/+27
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unify text insertion when parsing HTML and XML <!-- 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/14928) <!-- Reviewable:end -->
| * | | | Unify text insertion when parsing HTML and XMLAnthony Ramine2017-01-104-30/+27
| | | | |