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

Regenerate bindings #14268

Merged
merged 2 commits into from Nov 21, 2016
Merged

Regenerate bindings #14268

merged 2 commits into from Nov 21, 2016

Conversation

@Manishearth
Copy link
Member

Manishearth commented Nov 18, 2016

Don't merge yet


This change is Reviewable

@highfive
Copy link

highfive commented Nov 18, 2016

Heads up! This PR modifies the following files:

  • @bholley: components/style/gecko_bindings/structs_debug.rs, components/style/gecko_string_cache/atom_macro.rs, components/style/gecko_bindings/structs_release.rs
  • @emilio: components/style/gecko_bindings/structs_debug.rs, components/style/gecko_string_cache/atom_macro.rs, components/style/gecko_bindings/structs_release.rs
@highfive
Copy link

highfive commented Nov 18, 2016

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!
  • These commits modify style code, but no tests are modified. Please consider adding a test!
@Manishearth
Copy link
Member Author

Manishearth commented Nov 18, 2016

I'm getting this segfault on wikipedia:

thread '<unnamed>' panicked at 'Calling previous_styles without having gathered it', /Users/manishearth/mozilla/s-i/servo/components/style/data.rs:172
stack backtrace:
   1:        0x114b01138 - std::sys::backtrace::tracing::imp::write::h22f199c1dbb72ba2
   2:        0x114b0eacf - std::panicking::default_hook::{{closure}}::h9a389c462b6a22dd
   3:        0x114b0bf9f - std::panicking::default_hook::h852b4223c1c00c59
   4:        0x114b0c5c6 - std::panicking::rust_panic_with_hook::hcd9d05f53fa0dafc
   5:        0x1147af61a - std::panicking::begin_panic::hc03e2830c2c89a5f
   6:        0x11487c01a - style::data::ElementData::previous_styles_mut::h3209ac372e2e86dd
   7:        0x11488af2d - style::matching::MatchMethods::cascade_node::hc5639905206ebb4b
   8:        0x114892dba - style::traversal::recalc_style_at::h0e01fc41d4f2f294
   9:        0x11487dc79 - <style::gecko::traversal::RecalcStyleOnly<'lc> as style::traversal::DomTraversalContext<style::gecko::wrapper::GeckoNode<'ln>>>::process_preorder::ha33cdaf57d18a3a9
  10:        0x1146f71b1 - style::parallel::top_down_dom::{{closure}}::h5d9a5fa5b76c3f58
  11:        0x1146f70c3 - rayon::scope::Scope::execute_job_closure::{{closure}}::h1f98fccacfa09a33
  12:        0x1146ec797 - <std::panic::AssertUnwindSafe<F> as core::ops::FnOnce<()>>::call_once::h415c352b9b94e4d7
  13:        0x114693a67 - std::panicking::try::do_call::h3f93514d15f1526b

Trypush at https://treeherder.mozilla.org/#/jobs?repo=try&revision=dee8796be7b8a0b9811c683bf3b5079bc5019af0&selectedJob=31377430

@bholley , know anything about this? Seems to be related to your cascade changes?

@wafflespeanut
Copy link
Member

wafflespeanut commented Nov 18, 2016

Note: There's another #14255

@bholley
Copy link
Contributor

bholley commented Nov 18, 2016

@bors-servo delegate+

@bors-servo
Copy link
Contributor

bors-servo commented Nov 18, 2016

✌️ @Manishearth can now approve this pull request

@Manishearth
Copy link
Member Author

Manishearth commented Nov 18, 2016

@bors-servo r=bholley

@bors-servo
Copy link
Contributor

bors-servo commented Nov 18, 2016

📌 Commit 1c35faf has been approved by bholley

@highfive highfive assigned bholley and unassigned emilio Nov 18, 2016
@Manishearth Manishearth mentioned this pull request Nov 18, 2016
0 of 5 tasks complete
@bors-servo
Copy link
Contributor

bors-servo commented Nov 19, 2016

Testing commit 1c35faf with merge 51d9ef9...

bors-servo added a commit that referenced this pull request Nov 19, 2016
Regenerate bindings

Don't merge yet

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

bors-servo commented Nov 19, 2016

💔 Test failed - linux-dev

@Manishearth Manishearth force-pushed the Manishearth:regen branch from 1c35faf to 3d5b26a Nov 19, 2016
@Manishearth
Copy link
Member Author

Manishearth commented Nov 19, 2016

@bors-servo r=bholley

@bors-servo
Copy link
Contributor

bors-servo commented Nov 19, 2016

📌 Commit 3d5b26a has been approved by bholley

@bors-servo
Copy link
Contributor

bors-servo commented Nov 19, 2016

Testing commit 3d5b26a with merge 41d0d01...

@bors-servo
Copy link
Contributor

bors-servo commented Nov 20, 2016

Testing commit 3d5b26a with merge 8c60848...

bors-servo added a commit that referenced this pull request Nov 20, 2016
Regenerate bindings

Don't merge yet

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

bors-servo commented Nov 20, 2016

💔 Test failed - linux-rel-wpt

@KiChjang
Copy link
Member

KiChjang commented Nov 20, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Nov 20, 2016

Testing commit 3d5b26a with merge 5de74cc...

bors-servo added a commit that referenced this pull request Nov 20, 2016
Regenerate bindings

Don't merge yet

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

bors-servo commented Nov 20, 2016

💔 Test failed - mac-rel-wpt1

@KiChjang
Copy link
Member

KiChjang commented Nov 21, 2016

@heycam
Copy link
Member

heycam commented Nov 21, 2016

@heycam
Copy link
Member

heycam commented Nov 21, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Nov 21, 2016

Testing commit 3d5b26a with merge 95bc7ec...

bors-servo added a commit that referenced this pull request Nov 21, 2016
Regenerate bindings

Don't merge yet

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

bors-servo commented Nov 21, 2016

💔 Test failed - mac-rel-wpt1

@Manishearth
Copy link
Member Author

Manishearth commented Nov 21, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Nov 21, 2016

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

@bors-servo
Copy link
Contributor

bors-servo commented Nov 21, 2016

@bors-servo bors-servo merged commit 3d5b26a into servo:master Nov 21, 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

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