diff options
author | Cameron Zwarich <zwarich@mozilla.com> | 2014-09-19 01:32:30 -0700 |
---|---|---|
committer | Cameron Zwarich <zwarich@mozilla.com> | 2014-09-20 11:54:10 -0700 |
commit | 2c8d51a37c84fb5de531d00c45de9c0020930b11 (patch) | |
tree | 9d65c2f2141edf9bd8b47bb785b7e948e092f831 /components/script/dom/treewalker.rs | |
parent | 2adc594e5d8babaadbe1a4e05a8f7d808313728f (diff) | |
download | servo-2c8d51a37c84fb5de531d00c45de9c0020930b11.tar.gz servo-2c8d51a37c84fb5de531d00c45de9c0020930b11.zip |
More progress in the &JSRef -> JSRef conversion
Change all of the <Class>Methods traits to take `self` instead of
`&self`.
Diffstat (limited to 'components/script/dom/treewalker.rs')
-rw-r--r-- | components/script/dom/treewalker.rs | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/components/script/dom/treewalker.rs b/components/script/dom/treewalker.rs index aa09af45c7c..e1d49be408f 100644 --- a/components/script/dom/treewalker.rs +++ b/components/script/dom/treewalker.rs @@ -69,15 +69,15 @@ impl TreeWalker { } impl<'a> TreeWalkerMethods for JSRef<'a, TreeWalker> { - fn Root(&self) -> Temporary<Node> { + fn Root(self) -> Temporary<Node> { Temporary::new(self.root_node) } - fn WhatToShow(&self) -> u32 { + fn WhatToShow(self) -> u32 { self.what_to_show } - fn GetFilter(&self) -> Option<NodeFilter> { + fn GetFilter(self) -> Option<NodeFilter> { match self.filter { FilterNone => None, FilterJS(nf) => Some(nf), @@ -85,41 +85,41 @@ impl<'a> TreeWalkerMethods for JSRef<'a, TreeWalker> { } } - fn CurrentNode(&self) -> Temporary<Node> { + fn CurrentNode(self) -> Temporary<Node> { Temporary::new(self.current_node.get()) } - fn SetCurrentNode(&self, node: JSRef<Node>) -> ErrorResult { + fn SetCurrentNode(self, node: JSRef<Node>) -> ErrorResult { // XXX Future: check_same_origin(root_node, node) (throws) self.current_node.set(JS::from_rooted(node)); Ok(()) } - fn ParentNode(&self) -> Fallible<Option<Temporary<Node>>> { + fn ParentNode(self) -> Fallible<Option<Temporary<Node>>> { self.parent_node() } - fn FirstChild(&self) -> Fallible<Option<Temporary<Node>>> { + fn FirstChild(self) -> Fallible<Option<Temporary<Node>>> { self.first_child() } - fn LastChild(&self) -> Fallible<Option<Temporary<Node>>> { + fn LastChild(self) -> Fallible<Option<Temporary<Node>>> { self.last_child() } - fn PreviousSibling(&self) -> Fallible<Option<Temporary<Node>>> { + fn PreviousSibling(self) -> Fallible<Option<Temporary<Node>>> { self.prev_sibling() } - fn NextSibling(&self) -> Fallible<Option<Temporary<Node>>> { + fn NextSibling(self) -> Fallible<Option<Temporary<Node>>> { self.next_sibling() } - fn PreviousNode(&self) -> Fallible<Option<Temporary<Node>>> { + fn PreviousNode(self) -> Fallible<Option<Temporary<Node>>> { self.prev_node() } - fn NextNode(&self) -> Fallible<Option<Temporary<Node>>> { + fn NextNode(self) -> Fallible<Option<Temporary<Node>>> { self.next_node() } } |