Skip to content

Commit

Permalink
Fix leak
Browse files Browse the repository at this point in the history
Introduced by yours-truly in 7212c35

Oopsie.
  • Loading branch information
brendanzab committed Jun 6, 2014
1 parent eae9b94 commit bf21217
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions src/components/script/dom/node.rs
Expand Up @@ -126,6 +126,15 @@ impl NodeFlags {
}
}

#[unsafe_destructor]
impl Drop for Node {
fn drop(&mut self) {
unsafe {
self.reap_layout_data();
}
}
}

/// suppress observers flag
/// http://dom.spec.whatwg.org/#concept-node-insert
/// http://dom.spec.whatwg.org/#concept-node-remove
Expand Down

5 comments on commit bf21217

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from jdm
at brendanzab@bf21217

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging bjz/servo/leeeeeak = bf21217 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bjz/servo/leeeeeak = bf21217 merged ok, testing candidate = f1194fc

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = f1194fc

Please sign in to comment.