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

webgl: Unlock WebRender by default by adding a readback based fallback to WebGL context creation. #11744

Merged
merged 1 commit into from Jul 11, 2016

Conversation

@emilio
Copy link
Member

emilio commented Jun 14, 2016


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes do not require tests because refactoring.

This should give us the chance to use WebRender by default in OSX.

r? @pcwalton

cc @glennw @larsbergstrom


This change is Reviewable

…k to WebGL context creation.

This should give us the chance to use WebRender by default in OSX.
emilio added a commit to emilio/servo that referenced this pull request Jun 15, 2016
@emilio emilio mentioned this pull request Jun 15, 2016
4 of 4 tasks complete
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Jun 27, 2016

@pcwalton @glennw Is this ready to go? I'd really like to get WebRender on by default before the June release or people enlisting in and building Servo will be quite confused.

cc @metajack

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Jul 9, 2016

What's next on this PR? @glennw @pcwalton

@glennw
Copy link
Member

glennw commented Jul 10, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jul 10, 2016

📌 Commit 4af508a has been approved by glennw

@highfive highfive assigned glennw and unassigned pcwalton Jul 10, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Jul 11, 2016

Testing commit 4af508a with merge 68aca87...

bors-servo added a commit that referenced this pull request Jul 11, 2016
webgl: Unlock WebRender by default by adding a readback based fallback to WebGL context creation.

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

---
<!-- 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

<!-- Either: -->
- [x] These changes do not require tests because refactoring.

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

This should give us the chance to use WebRender by default in OSX.

r? @pcwalton

cc @glennw @larsbergstrom

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11744)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 11, 2016

💔 Test failed - linux-rel

@emilio
Copy link
Member Author

emilio commented Jul 11, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jul 11, 2016

Previous build results for android, arm32, arm64, linux-dev, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows are reusable. Rebuilding only linux-rel...

@bors-servo
Copy link
Contributor

bors-servo commented Jul 11, 2016

@bors-servo bors-servo merged commit 4af508a into servo:master Jul 11, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
emilio added a commit to emilio/servo that referenced this pull request Jul 11, 2016
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

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