diff options
author | Corey Farwell <coreyf@rwell.org> | 2015-07-14 09:24:53 +0900 |
---|---|---|
committer | Corey Farwell <coreyf@rwell.org> | 2015-07-14 09:24:53 +0900 |
commit | 78d2fe6e9b08e41f23a9767b29317896ddb90689 (patch) | |
tree | 4f2ef754fe722977052da64e6c6274380dd745b8 | |
parent | 12195a5c4ac08f1e6eeaf8888c279fea8e5be48a (diff) | |
download | servo-78d2fe6e9b08e41f23a9767b29317896ddb90689.tar.gz servo-78d2fe6e9b08e41f23a9767b29317896ddb90689.zip |
Complete FIXMEs related to UFCS
-rw-r--r-- | components/script/dom/element.rs | 24 | ||||
-rw-r--r-- | components/script/dom/node.rs | 8 |
2 files changed, 4 insertions, 28 deletions
diff --git a/components/script/dom/element.rs b/components/script/dom/element.rs index 30b92f0d49b..e60946193ff 100644 --- a/components/script/dom/element.rs +++ b/components/script/dom/element.rs @@ -1645,22 +1645,10 @@ impl<'a> ::selectors::Element for &'a Element { } fn get_local_name<'b>(&'b self) -> &'b Atom { - // FIXME(zwarich): Remove this when UFCS lands and there is a better way - // of disambiguating methods. - fn get_local_name<'a, T: ElementHelpers<'a>>(this: T) -> &'a Atom { - this.local_name() - } - - get_local_name(*self) + ElementHelpers::local_name(*self) } fn get_namespace<'b>(&'b self) -> &'b Namespace { - // FIXME(zwarich): Remove this when UFCS lands and there is a better way - // of disambiguating methods. - fn get_namespace<'a, T: ElementHelpers<'a>>(this: T) -> &'a Namespace { - this.namespace() - } - - get_namespace(*self) + self.namespace() } fn get_hover_state(&self) -> bool { let node = NodeCast::from_ref(*self); @@ -1707,13 +1695,7 @@ impl<'a> ::selectors::Element for &'a Element { } } fn has_class(&self, name: &Atom) -> bool { - // FIXME(zwarich): Remove this when UFCS lands and there is a better way - // of disambiguating methods. - fn has_class<T: AttributeHandlers>(this: T, name: &Atom) -> bool { - this.has_class(name) - } - - has_class(*self, name) + AttributeHandlers::has_class(*self, name) } fn each_class<F>(&self, mut callback: F) where F: FnMut(&Atom) diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs index 21c427d07b1..7c3b6b2906c 100644 --- a/components/script/dom/node.rs +++ b/components/script/dom/node.rs @@ -2606,13 +2606,7 @@ impl<'a> ::selectors::Node<&'a Element> for &'a Node { } fn is_document(&self) -> bool { - // FIXME(zwarich): Remove this when UFCS lands and there is a better way - // of disambiguating methods. - fn is_document<'a, T: DocumentDerived>(this: &T) -> bool { - this.is_document() - } - - is_document(*self) + DocumentDerived::is_document(*self) } fn as_element(&self) -> Option<&'a Element> { |