Builder linux-rel-css Build #3443

Results:

Build successful

SourceStamp:

BuildSlave:

Reason:

The AnyBranchScheduler scheduler named 'servo-auto' triggered this build

Steps and Logfiles:

1. git update ( 3 secs )
   1. stdio
2. generic generic ( 0 secs )
   1. - no logs -
3. generic_1 generic_1 ( 0 secs )
   1. - no logs -
4. shell 'pkill -x ...' ( 0 secs )
   1. stdio
5. compile ran build --release --with-debug-assertions warnings ( 14 mins, 17 secs )
   1. warnings (3)
6. test ran test-css --release --processes 16 --always-succeed ( 11 mins, 19 secs )
   1. css-errorsummary.log
7. shell__1 './mach filter-intermittents ...' ( 0 secs )
   1. stdio
   2. filtered-css-errorsummary.log
   3. intermittents.log
8. compile__1 ran build-cef --release --with-debug-assertions ( 36 secs )
   1. stdio
9. compile__2 ran build-geckolib --release ( 4 mins, 21 secs )
   1. stdio
10. test__1 ran test-stylo --release ( 5 mins, 31 secs )
    1. - no logs -
11. shell__2 'bash ./etc/ci/lockfile_changed.sh' ( 0 secs )
    1. stdio
12. shell__3 'bash ./etc/ci/manifest_changed.sh' ( 0 secs )
    1. stdio

Build Properties:

Forced Build Properties:

Responsible Users:

1. bors-servo

   lbergstrom+borsohnoyoudont@mozilla.com

Timing:

All Changes:

:

1. Change #43330

   Category	None
   Changed by	bors-servo <lbergstrom+borsohnoyoudont@mozilla.com>
   Changed at	Sun 16 Apr 2017 13:28:08
   Repository	https://github.com/servo/servo
   Project	servo/servo
   Branch	auto
   Revision	9667090e0b7c714dd938d08d19d31e46a2dd2291

   Comments

   Auto merge of #16472 - cu1t:#14095-fix-xml-doc-namespaces, r=nox
   
   Fix namespaces of elements created in XML documents
   
   Correctly implement following step of [Dom Document Spec](https://dom.spec.whatwg.org/#dom-document-createelement):
   > Let namespace be the HTML namespace, if the context object is an HTML document or context object’s content type is "application/xhtml+xml", and null otherwise.
   
   Note, this will make following test in `tests/wpt/web-platform-tests/dom/nodes/Document-constructor.html` to fail, so related .ini file added to mark it as such:
   ```
   test(function() {
     var doc = new Document();
     var a = doc.createElement("a");
     // In UTF-8: 0xC3 0xA4
     a.href = "http://example.org/?\u00E4";
     assert_equals(a.href, "http://example.org/?%C3%A4");
   }, "new Document(): URL parsing")
   ```
   I'm not very familiar with specs, but from quick look at it, I'm doubtfull that it is valid in the first place. This is an "application/xml" document, so I don't see why it should encode a.href. Firefox also fails that.
   
   ---
   <!-- 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 #14095 (github issue number if applicable).
   
   <!-- Either: -->
   - [ ] There are tests for these changes OR
   - [X] These changes do not require tests because because there are already tests which were being ignored
   
   <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->
   
   <!-- 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/16472)
   <!-- Reviewable:end -->

   Changed files

   • components/script/dom/document.rs
   • tests/wpt/metadata/dom/nodes/Document-constructor.html.ini
   • tests/wpt/metadata/dom/nodes/Node-properties.html.ini