Home - Waterfall Grid T-Grid Console Builders Recent Builds Buildslaves Changesources - JSON API - About

Builder linux-rel-wpt Build #10066

Results:

Failed shell__4

SourceStamp:

Projectservo/servo
Repositoryhttps://github.com/servo/servo
Branchtry
Revisioncfd9608a29de2a4b6a9979bcdd5e506a2230f139
Got Revisioncfd9608a29de2a4b6a9979bcdd5e506a2230f139
Changes1 change

BuildSlave:

servo-linux2

Reason:

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

Steps and Logfiles:

  1. git update ( 34 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. shell__1 './mach clean-nightlies ...' ( 11 secs )
    1. stdio
  6. shell__2 './mach clean-cargo-cache ...' ( 0 secs )
    1. stdio
  7. shell__3 './etc/ci/clean_build_artifacts.sh' ( 0 secs )
    1. stdio
  8. compile ran build --release --with-debug-assertions -p servo warnings ( 9 mins, 40 secs )
    1. warnings (1)
  9. test ran test-wpt-failure ( 0 secs )
    1. stdio
  10. test__1 ran test-wpt --release --processes 24 --total-chunks 2 --this-chunk 1 --always-succeed ( 16 mins, 8 secs )
    1. - no logs -
  11. shell__4 './mach filter-intermittents ...' failed ( 1 secs )
    1. stdio
    2. filtered-wpt-errorsummary.log
  12. test__2 ran test-wpt --release --binary-arg=--multiprocess --processes 24 eventsource ( 21 secs )
    1. wpt-mp-errorsummary.log
  13. shell__5 './etc/ci/clean_build_artifacts.sh' ( 0 secs )
    1. stdio

Build Properties:

NameValueSource
branch try Build
builddir /home/servo/buildbot/slave/linux-rel-wpt slave
buildername linux-rel-wpt Builder
buildnumber 10066 Build
codebase Build
got_revision cfd9608a29de2a4b6a9979bcdd5e506a2230f139 Git
project servo/servo Build
repository https://github.com/servo/servo Build
revision cfd9608a29de2a4b6a9979bcdd5e506a2230f139 Build
scheduler servo-auto Scheduler
slavename servo-linux2 BuildSlave
warnings-count 1 WarningCountingShellCommand
workdir /home/servo/buildbot/slave/linux-rel-wpt slave (deprecated)

Forced Build Properties:

NameLabelValue

Responsible Users:

  1. bors-servo

Timing:

StartSun Oct 7 09:50:18 2018
EndSun Oct 7 10:17:18 2018
Elapsed27 mins, 0 secs

All Changes:

:

  1. Change #97818

    Category None
    Changed by bors-servo <lbergstrom+borsohnoyoudont@mozilla.com>
    Changed at Sun 07 Oct 2018 09:50:16
    Repository https://github.com/servo/servo
    Project servo/servo
    Branch try
    Revision cfd9608a29de2a4b6a9979bcdd5e506a2230f139

    Comments

    Auto merge of #21882 - dguenther:update-document-open, r=<try>
    
    Update document.open to latest spec
    
    <!-- Please describe your changes on the following line: -->
    
    This is one of my first contributions, so I might need some direction cleaning it up -- I ran web-platform-tests locally, but the suite has several intermittent passes/failures for me. Thanks!
    
    Few notes:
    
    * I may have been wrong to eliminate all of the resets listed in what was formerly Step 18 (starts with `self.implementation.set(None);`). It's not clear to me that they're still needed or if so, what step they would fall under, but I didn't notice any web platform tests break as a result.
    * If I'm reading the spec right, there's a discrepancy in the error returned by the three-parameter overload of Document.open between the spec and web-platform-tests/implementations in other browsers. As written, I favored the spec, but it causes one web-platform-test to fail.
    * I'm not 100% certain that tests pass as expected, I had several intermittent failures that disappeared when re-run.
    
    ---
    <!-- 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 #21445
    
    <!-- Either: -->
    - [x] There are tests for these changes (existing web-platform-tests)
    
    <!-- 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/21882)
    <!-- Reviewable:end -->

    Changed files

    • components/script/dom/document.rs
    • components/script/dom/webidls/Document.webidl
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/001.html.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/005.html.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/008.html.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/009.https.html.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/document.open-02.html.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/document.open-03.html.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/event-listeners.window.js.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/quirks.window.js.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/readiness.window.js.ini
    • tests/wpt/metadata/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/url-fragment.window.js.ini