Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure reference frame is popped in layout 2020 #26072

Merged
merged 1 commit into from Mar 31, 2020

Conversation

@dralley
Copy link
Contributor

dralley commented Mar 31, 2020

closes: #26066

Should be equivalent to #26063 for layout2020


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #26066 (GitHub issue number if applicable)
  • There are tests for these changes OR
  • These changes do not require tests because ___
@cbrewster
Copy link
Member

cbrewster commented Mar 31, 2020

LGTM, does this look okay @mrobinson?

@mrobinson
Copy link
Member

mrobinson commented Mar 31, 2020

@bors-servo
Copy link
Contributor

bors-servo commented Mar 31, 2020

📌 Commit 814478b has been approved by mrobinson

@bors-servo
Copy link
Contributor

bors-servo commented Mar 31, 2020

Testing commit 814478b with merge 4ccbb13...

bors-servo added a commit that referenced this pull request Mar 31, 2020
Ensure reference frame is popped in layout 2020

closes: #26066

Should be equivalent to #26063 for layout2020

---
<!-- 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
- [ ] `./mach test-tidy` does not report any errors
- [x] These changes fix #26066 (GitHub issue number if applicable)

<!-- Either: -->
- [ ] There are tests for these changes OR
- [ ] These changes do not require tests because ___
@bors-servo
Copy link
Contributor

bors-servo commented Mar 31, 2020

💔 Test failed - status-taskcluster

@dralley
Copy link
Contributor Author

dralley commented Mar 31, 2020

  • 3 tests timed out unexpectedly
  • 1 tests unexpectedly okay
  • 4 tests had unexpected subtest results
@jdm
Copy link
Member

jdm commented Mar 31, 2020

@bors-servo
Copy link
Contributor

bors-servo commented Mar 31, 2020

Testing commit 814478b with merge 148c24c...

@bors-servo
Copy link
Contributor

bors-servo commented Mar 31, 2020

☀️ Test successful - status-taskcluster
Approved by: mrobinson
Pushing 148c24c to master...

@bors-servo bors-servo merged commit 148c24c into servo:master Mar 31, 2020
2 checks passed
2 checks passed
Community-TC (pull_request) TaskGroup: success
Details
homu Test successful
Details
@dralley dralley deleted the dralley:fix-assertion-layout2020 branch Mar 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

7 participants
You can’t perform that action at this time.