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 (ellipse border-radius, preserve-3d, rayon thread pool). #16801

Closed
wants to merge 1 commit into from

Conversation

@glennw
Copy link
Member

glennw commented May 10, 2017

This change is Reviewable

@highfive
Copy link

highfive commented May 10, 2017

Heads up! This PR modifies the following files:

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

highfive commented May 10, 2017

warning Warning warning

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

mbrubeck commented May 10, 2017

@bors-servo
Copy link
Contributor

bors-servo commented May 10, 2017

📌 Commit 247cb33 has been approved by mbrubeck

@glennw glennw force-pushed the glennw:update-wr-ellipse-radii branch from 247cb33 to cc324ff May 10, 2017
@glennw
Copy link
Member Author

glennw commented May 10, 2017

@bors-servo r=mbrubeck

  • Added some new test passes.
@bors-servo
Copy link
Contributor

bors-servo commented May 10, 2017

📌 Commit cc324ff has been approved by mbrubeck

@bors-servo
Copy link
Contributor

bors-servo commented May 10, 2017

Testing commit cc324ff with merge 2c1e658...

bors-servo added a commit that referenced this pull request May 10, 2017
Update WR (ellipse border-radius, preserve-3d, rayon thread pool).

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

bors-servo commented May 11, 2017

💔 Test failed - linux-rel-wpt

@jdm
Copy link
Member

jdm commented May 11, 2017

@bors-servo
Copy link
Contributor

bors-servo commented May 11, 2017

Testing commit cc324ff with merge bddb359...

bors-servo added a commit that referenced this pull request May 11, 2017
Update WR (ellipse border-radius, preserve-3d, rayon thread pool).

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

bors-servo commented May 11, 2017

💔 Test failed - mac-rel-css

@jdm
Copy link
Member

jdm commented May 11, 2017

  ▶ TIMEOUT [expected FAIL] /css-backgrounds-3_dev/html4/border-radius-clipping.htm
  │ 
  │ VMware, Inc.
  │ Gallium 0.4 on softpipe
  │ 3.3 (Core Profile) Mesa 12.0.1
  └ Shutting down the Constellation after generating an output file or exit flag specified
@glennw
Copy link
Member Author

glennw commented May 12, 2017

@bors-servo try

  • Testing if the timeouts are consistent.
@kvark kvark mentioned this pull request May 13, 2017
3 of 5 tasks complete
bors-servo added a commit that referenced this pull request May 14, 2017
WR update: preserve-3d support

<!-- Please describe your changes on the following line: -->

This is WR update to servo/webrender@d335555 having new features:
  - limited "preserve-3d" support (servo/webrender#1169, servo/webrender#1208)
  - rayon thread pool (servo/webrender#1202)
  - further border rendering improvements

Edit: the references to bincode serialization and border styles are removed from here, since they are already integrated into Servo.
Edit2: this is alternative/similar to  #16801, based on @mrobinson code (see first commit).

---
<!-- 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
- [ ] These changes fix #__ (github issue number if applicable).

Related to #9087
Note that I'm unlocking a few tests as well as changing some related to `preserve-3d`. The changes come from common sense and comparison to Chromium. I'm ready to discuss them on a individual basis.

<!-- Either: -->
- [x] There are tests for these changes OR
- [ ] These changes do not require tests because _____

There is still an investigation to do with regards to the differences of preserve3d logic between Blink and Gecko.

<!-- 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/16775)
<!-- Reviewable:end -->
@glennw
Copy link
Member Author

glennw commented May 14, 2017

Superseded by #16775

@glennw glennw closed this May 14, 2017
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

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