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

Update WR (box shadows + per-corner radii, elliptical clips). #18956

Merged
merged 1 commit into from Oct 20, 2017

Conversation

@glennw
Copy link
Member

glennw commented Oct 19, 2017

This change is Reviewable

@highfive
Copy link

highfive commented Oct 19, 2017

Heads up! This PR modifies the following files:

  • @emilio: components/layout/webrender_helpers.rs, components/layout/display_list_builder.rs
@highfive
Copy link

highfive commented Oct 19, 2017

warning Warning warning

  • These commits modify gfx and layout code, but no tests are modified. Please consider adding a test!
@glennw
Copy link
Member Author

glennw commented Oct 19, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Oct 19, 2017

Trying commit ae831ea with merge bcfaa77...

bors-servo added a commit that referenced this pull request Oct 19, 2017
Update WR (box shadows + per-corner radii, elliptical clips).
@bors-servo
Copy link
Contributor

bors-servo commented Oct 19, 2017

💔 Test failed - linux-rel-css

@glennw
Copy link
Member Author

glennw commented Oct 19, 2017

@bors-servo retry

  • infra issue?
@bors-servo
Copy link
Contributor

bors-servo commented Oct 19, 2017

Trying commit ae831ea with merge 6774bc3...

bors-servo added a commit that referenced this pull request Oct 19, 2017
Update WR (box shadows + per-corner radii, elliptical clips).

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

bors-servo commented Oct 19, 2017

The build was interrupted to prioritize another pull request.

@bors-servo
Copy link
Contributor

bors-servo commented Oct 19, 2017

Trying commit ae831ea with merge 57e411b...

bors-servo added a commit that referenced this pull request Oct 19, 2017
Update WR (box shadows + per-corner radii, elliptical clips).

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

bors-servo commented Oct 19, 2017

The build was interrupted to prioritize another pull request.

@bors-servo
Copy link
Contributor

bors-servo commented Oct 19, 2017

Trying commit ae831ea with merge 31575d3...

bors-servo added a commit that referenced this pull request Oct 19, 2017
Update WR (box shadows + per-corner radii, elliptical clips).

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

bors-servo commented Oct 20, 2017

💔 Test failed - arm64

@jdm
Copy link
Member

jdm commented Oct 20, 2017

That problem will be solved by #18961.

@jdm
Copy link
Member

jdm commented Oct 20, 2017

  ▶ PASS [expected FAIL] /css-backgrounds-3_dev/html4/css-box-shadow-001.htm
@glennw glennw force-pushed the glennw:update-wr-ellipse-bs branch from ae831ea to 8da7cc4 Oct 20, 2017
@glennw
Copy link
Member Author

glennw commented Oct 20, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Oct 20, 2017

Trying commit 8da7cc4 with merge 576cdfe...

bors-servo added a commit that referenced this pull request Oct 20, 2017
Update WR (box shadows + per-corner radii, elliptical clips).

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

bors-servo commented Oct 20, 2017

💔 Test failed - linux-dev

@glennw glennw force-pushed the glennw:update-wr-ellipse-bs branch from 8da7cc4 to aaaf644 Oct 20, 2017
@glennw
Copy link
Member Author

glennw commented Oct 20, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Oct 20, 2017

Trying commit aaaf644 with merge c24c6e8...

bors-servo added a commit that referenced this pull request Oct 20, 2017
Update WR (box shadows + per-corner radii, elliptical clips).

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

bors-servo commented Oct 20, 2017

💔 Test failed - linux-rel-wpt

@jdm
Copy link
Member

jdm commented Oct 20, 2017

That's #13509. Everything else passed.

@jdm
Copy link
Member

jdm commented Oct 20, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Oct 20, 2017

📌 Commit aaaf644 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Oct 20, 2017

Testing commit aaaf644 with merge fef468e...

bors-servo added a commit that referenced this pull request Oct 20, 2017
Update WR (box shadows + per-corner radii, elliptical clips).

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

bors-servo commented Oct 20, 2017

💔 Test failed - linux-rel-wpt

@jdm
Copy link
Member

jdm commented Oct 20, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Oct 20, 2017

Testing commit aaaf644 with merge d604270...

bors-servo added a commit that referenced this pull request Oct 20, 2017
Update WR (box shadows + per-corner radii, elliptical clips).

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

bors-servo commented Oct 20, 2017

@bors-servo bors-servo merged commit aaaf644 into servo:master Oct 20, 2017
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@bors-servo bors-servo mentioned this pull request Oct 20, 2017
3 of 5 tasks complete
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.

None yet

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