Skip to content

Add API for tweaking debug options. #526

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

Merged
merged 1 commit into from
Nov 4, 2016
Merged

Conversation

glennw
Copy link
Member

@glennw glennw commented Nov 4, 2016

Right now, it's possible to toggle the profiler. In the future
this can include options for subpixel AA, shader hot reload etc.


This change is Reviewable

@pcwalton
Copy link
Contributor

pcwalton commented Nov 4, 2016

This is an uber-nit, but do you think that toggling is really the best API here, as opposed to set on/off? It requires clients to know the state and doesn't allow multiple clients to interact with the system unless they coordinate…

Used by Servo to tweak whether profiler is currently running.
@pcwalton
Copy link
Contributor

pcwalton commented Nov 4, 2016

@bors-servo: r+

@bors-servo
Copy link
Contributor

📌 Commit 0c238e4 has been approved by pcwalton

@glennw
Copy link
Member Author

glennw commented Nov 4, 2016

@bors-servo r=pcwalton

@bors-servo
Copy link
Contributor

💡 This pull request was already approved, no need to approve it again.

@bors-servo
Copy link
Contributor

📌 Commit 0c238e4 has been approved by pcwalton

@bors-servo
Copy link
Contributor

⌛ Testing commit 0c238e4 with merge 8b53081...

bors-servo pushed a commit that referenced this pull request Nov 4, 2016
Add API for tweaking debug options.

Right now, it's possible to toggle the profiler. In the future
this can include options for subpixel AA, shader hot reload etc.

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

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit 0c238e4 into servo:master Nov 4, 2016
@glennw glennw deleted the debug-opts branch December 12, 2016 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants