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

Report memory usage from LayoutThreadData Stylist [#7038] #10088

Conversation

@pkondzior
Copy link
Contributor

pkondzior commented Mar 19, 2016

@jdm PTAL I'm not sure what is the approach of updating cargo components here, I've made a pull request servo/heapsize#54 but it has to be landed first before merge and version bump.


This change is Reviewable

@highfive
Copy link

highfive commented Mar 19, 2016

Heads up! This PR modifies the following files:

  • @bholley: components/style/selector_matching.rs, components/style/custom_properties.rs, components/style/restyle_hints.rs, components/style/properties.mako.rs, components/style/values.rs
@highfive
Copy link

highfive commented Mar 19, 2016

warning Warning warning

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

emilio commented Mar 19, 2016

You should be able to update heapsize using:

./mach cargo-update -p heapsize

Hopefully you don't run into any version number conflicts.

@pkondzior pkondzior force-pushed the pkondzior:7038-report-memory-usage-in-layout-thread-data-stylist branch from de4adaf to db2e10a Mar 19, 2016
@pkondzior
Copy link
Contributor Author

pkondzior commented Mar 19, 2016

@ecoal95 fixed, thanks!

@emilio emilio assigned emilio and unassigned metajack Mar 21, 2016
@emilio
Copy link
Member

emilio commented Mar 21, 2016

This looks good to me :)

@bors-servo: r+

@bors-servo
Copy link
Contributor

bors-servo commented Mar 21, 2016

📌 Commit db2e10a has been approved by ecoal95

@bors-servo
Copy link
Contributor

bors-servo commented Mar 21, 2016

Testing commit db2e10a with merge 96dc0f0...

bors-servo added a commit that referenced this pull request Mar 21, 2016
…hread-data-stylist, r=ecoal95

Report memory usage from LayoutThreadData Stylist [#7038]

@jdm PTAL I'm not sure what is the approach of updating cargo components here, I've made a pull request servo/heapsize#54 but it has to be landed first before merge and version bump.

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

bors-servo commented Mar 21, 2016

💔 Test failed - gonk

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Mar 22, 2016

@bors-servo retry

  • infra
@bors-servo
Copy link
Contributor

bors-servo commented Mar 22, 2016

Testing commit db2e10a with merge 9a8ba23...

bors-servo added a commit that referenced this pull request Mar 22, 2016
…hread-data-stylist, r=ecoal95

Report memory usage from LayoutThreadData Stylist [#7038]

@jdm PTAL I'm not sure what is the approach of updating cargo components here, I've made a pull request servo/heapsize#54 but it has to be landed first before merge and version bump.

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

bors-servo commented Mar 22, 2016

@bors-servo bors-servo merged commit db2e10a into servo:master Mar 22, 2016
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
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

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