From ad22b29b2d426fa39e66a66f8be9130bdf1fc0a1 Mon Sep 17 00:00:00 2001
From: Tetsuharu OHZEKI <saneyuki.snyk@gmail.com>
Date: Sat, 31 May 2014 23:40:24 +0900
Subject: [PATCH] Remove the implementation OptionalSettable for Option<JS<T>>
 to remove needless '&mut self'.

---
 src/components/script/dom/bindings/js.rs | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/src/components/script/dom/bindings/js.rs b/src/components/script/dom/bindings/js.rs
index c5d39959c8a..805adcf1ff3 100644
--- a/src/components/script/dom/bindings/js.rs
+++ b/src/components/script/dom/bindings/js.rs
@@ -260,20 +260,12 @@ impl<T: Reflectable> Assignable<T> for Temporary<T> {
 /// Assign an optional rootable value (either of JS<T> or Temporary<T>) to an optional
 /// field of a DOM type (ie. Option<JS<T>>)
 pub trait OptionalSettable<T> {
-    fn assign(&mut self, val: Option<T>);
-}
-
-impl<T: Assignable<U>, U: Reflectable> OptionalSettable<T> for Option<JS<U>> {
-    fn assign(&mut self, val: Option<T>) {
-        *self = val.map(|val| unsafe { val.get_js() });
-    }
+    fn assign(&self, val: Option<T>);
 }
 
 impl<T: Assignable<U>, U: Reflectable> OptionalSettable<T> for Cell<Option<JS<U>>> {
-    fn assign(&mut self, val: Option<T>) {
-        let mut item = self.get();
-        item.assign(val);
-        self.set(item);
+    fn assign(&self, val: Option<T>) {
+        self.set(val.map(|val| unsafe { val.get_js() }));
     }
 }