Skip to content

Comments

Sync WPT with upstream (08-10-2019)#24394

Closed
servo-wpt-sync wants to merge 2 commits intoservo:masterfrom
servo-wpt-sync:wpt_update_08-10-2019
Closed

Sync WPT with upstream (08-10-2019)#24394
servo-wpt-sync wants to merge 2 commits intoservo:masterfrom
servo-wpt-sync:wpt_update_08-10-2019

Conversation

@servo-wpt-sync
Copy link
Collaborator

@servo-wpt-sync servo-wpt-sync commented Oct 8, 2019

Automated downstream sync of changes from upstream as of 08-10-2019.
[no-wpt-sync]
r? @servo-wpt-sync


This change is Reviewable

@servo-wpt-sync
Copy link
Collaborator Author

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 4a65c0d has been approved by servo-wpt-sync

@highfive highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 8, 2019
@highfive
Copy link

highfive commented Oct 8, 2019

Heads up! This PR modifies the following files:

  • @jgraham: tests/wpt/webgl/meta/conformance2/rendering/canvas-resizing-with-pbo-bound.html.ini

@highfive highfive added S-awaiting-review There is new code that needs to be reviewed. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Oct 8, 2019
@highfive
Copy link

highfive commented Oct 8, 2019

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!

@jdm
Copy link
Member

jdm commented Oct 8, 2019

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

⌛ Trying commit adaf292 with merge 9d0086f...

bors-servo pushed a commit that referenced this pull request Oct 8, 2019
Sync WPT with upstream (08-10-2019)

Automated downstream sync of changes from upstream as of 08-10-2019.
[no-wpt-sync]
r? @servo-wpt-sync

<!-- 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/24394)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

💔 Test failed - linux-rel-wpt

@highfive highfive added the S-tests-failed The changes caused existing tests to fail. label Oct 8, 2019
@jdm jdm force-pushed the wpt_update_08-10-2019 branch from adaf292 to 6764635 Compare October 9, 2019 02:01
@highfive highfive removed the S-tests-failed The changes caused existing tests to fail. label Oct 9, 2019
@jdm
Copy link
Member

jdm commented Oct 9, 2019

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 6764635 has been approved by jdm

@highfive highfive assigned jdm and unassigned servo-wpt-sync Oct 9, 2019
@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Oct 9, 2019
@bors-servo
Copy link
Contributor

⌛ Testing commit 6764635 with merge 8afd5da...

bors-servo pushed a commit that referenced this pull request Oct 9, 2019
Sync WPT with upstream (08-10-2019)

Automated downstream sync of changes from upstream as of 08-10-2019.
[no-wpt-sync]
r? @servo-wpt-sync

<!-- 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/24394)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

💔 Test failed - linux-rel-css

@highfive highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Oct 9, 2019
@jdm
Copy link
Member

jdm commented Oct 9, 2019

@bors-servo
Copy link
Contributor

⌛ Testing commit 6764635 with merge 23413fe...

bors-servo pushed a commit that referenced this pull request Oct 9, 2019
Sync WPT with upstream (08-10-2019)

Automated downstream sync of changes from upstream as of 08-10-2019.
[no-wpt-sync]
r? @servo-wpt-sync

<!-- 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/24394)
<!-- Reviewable:end -->
@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Oct 9, 2019
@bors-servo
Copy link
Contributor

💔 Test failed - status-taskcluster

@highfive highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Oct 9, 2019
@CYBAI
Copy link
Member

CYBAI commented Oct 9, 2019

Interesting. These 2 tests are passed in mac build (5/6) now 🤔

{
    "status": "PASS", 
    "group": "default", 
    "message": null, 
    "stack": null, 
    "subtest": "Autofocus elements in top-level browsing context's documents with URI fragments should be skipped.", 
    "test": "/html/interaction/focus/the-autofocus-attribute/skip-document-with-fragment.html", 
    "line": 71369, 
    "action": "test_result", 
    "expected": "TIMEOUT"
}
{
    "status": "OK", 
    "group": "default", 
    "message": null, 
    "stack": null, 
    "subtest": null, 
    "test": "/html/interaction/focus/the-autofocus-attribute/skip-document-with-fragment.html", 
    "line": 71370, 
    "action": "test_result", 
    "expected": "TIMEOUT"
}

and another mac build (2/6) failure is #21067

@jdm
Copy link
Member

jdm commented Oct 9, 2019

#24407

@jdm jdm closed this Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-tests-failed The changes caused existing tests to fail.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants