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

Combine script profiling with profile crates. Fixes #7514. #7547

Merged
merged 1 commit into from Sep 8, 2015

Conversation

@connorimes
Copy link
Contributor

connorimes commented Sep 4, 2015

The script crate had its own built-in profiling which was basically doing the same thing as the profile crate. This wraps the internal profiling around the main profile functionality. Script-related tasks are now added to the ProfilerCategory enum.

Review on Reviewable

@jdm jdm self-assigned this Sep 4, 2015
@jdm
Copy link
Member

jdm commented Sep 8, 2015

@bors-servo: r+
Seems reasonable. Thanks for doing this!

@bors-servo
Copy link
Contributor

bors-servo commented Sep 8, 2015

📌 Commit d746835 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Sep 8, 2015

Testing commit d746835 with merge 25727ad...

bors-servo pushed a commit that referenced this pull request Sep 8, 2015
Combine script profiling with profile crates. Fixes #7514.

The script crate had its own built-in profiling which was basically doing the same thing as the profile crate.  This wraps the internal profiling around the main profile functionality.  Script-related tasks are now added to the ProfilerCategory enum.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7547)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 8, 2015

💔 Test failed - linux-rel

@jdm
Copy link
Member

jdm commented Sep 8, 2015

bors-servo pushed a commit that referenced this pull request Sep 8, 2015
Combine script profiling with profile crates. Fixes #7514.

The script crate had its own built-in profiling which was basically doing the same thing as the profile crate.  This wraps the internal profiling around the main profile functionality.  Script-related tasks are now added to the ProfilerCategory enum.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7547)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 8, 2015

Testing commit d746835 with merge ca36779...

@bors-servo
Copy link
Contributor

bors-servo commented Sep 8, 2015

@bors-servo bors-servo merged commit d746835 into servo:master Sep 8, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
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

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