Make the traits for the IDL interfaces take &self

1 files changed, 12 insertions, 12 deletions

diff --git a/components/script/dom/storage.rs b/components/script/dom/storage.rs
index 29872ab068a..a749663d945 100644
--- a/components/script/dom/storage.rs
+++ b/components/script/dom/storage.rs
@@ -55,9 +55,9 @@ impl Storage {
 
 }
 
-impl<'a> StorageMethods for &'a Storage {
+impl StorageMethods for Storage {
     // https://html.spec.whatwg.org/multipage/#dom-storage-length
-    fn Length(self) -> u32 {
+    fn Length(&self) -> u32 {
         let (sender, receiver) = ipc::channel().unwrap();
 
         self.get_storage_task().send(StorageTaskMsg::Length(sender, self.get_url(), self.storage_type)).unwrap();
@@ -65,7 +65,7 @@ impl<'a> StorageMethods for &'a Storage {
     }
 
     // https://html.spec.whatwg.org/multipage/#dom-storage-key
-    fn Key(self, index: u32) -> Option<DOMString> {
+    fn Key(&self, index: u32) -> Option<DOMString> {
         let (sender, receiver) = ipc::channel().unwrap();
 
         self.get_storage_task().send(StorageTaskMsg::Key(sender, self.get_url(), self.storage_type, index)).unwrap();
@@ -73,7 +73,7 @@ impl<'a> StorageMethods for &'a Storage {
     }
 
     // https://html.spec.whatwg.org/multipage/#dom-storage-getitem
-    fn GetItem(self, name: DOMString) -> Option<DOMString> {
+    fn GetItem(&self, name: DOMString) -> Option<DOMString> {
         let (sender, receiver) = ipc::channel().unwrap();
 
         let msg = StorageTaskMsg::GetItem(sender, self.get_url(), self.storage_type, name);
@@ -82,7 +82,7 @@ impl<'a> StorageMethods for &'a Storage {
     }
 
     // https://html.spec.whatwg.org/multipage/#dom-storage-setitem
-    fn SetItem(self, name: DOMString, value: DOMString) {
+    fn SetItem(&self, name: DOMString, value: DOMString) {
         let (sender, receiver) = ipc::channel().unwrap();
 
         let msg = StorageTaskMsg::SetItem(sender, self.get_url(), self.storage_type, name.clone(), value.clone());
@@ -94,7 +94,7 @@ impl<'a> StorageMethods for &'a Storage {
     }
 
     // https://html.spec.whatwg.org/multipage/#dom-storage-removeitem
-    fn RemoveItem(self, name: DOMString) {
+    fn RemoveItem(&self, name: DOMString) {
         let (sender, receiver) = ipc::channel().unwrap();
 
         let msg = StorageTaskMsg::RemoveItem(sender, self.get_url(), self.storage_type, name.clone());
@@ -105,7 +105,7 @@ impl<'a> StorageMethods for &'a Storage {
     }
 
     // https://html.spec.whatwg.org/multipage/#dom-storage-clear
-    fn Clear(self) {
+    fn Clear(&self) {
         let (sender, receiver) = ipc::channel().unwrap();
 
         self.get_storage_task().send(StorageTaskMsg::Clear(sender, self.get_url(), self.storage_type)).unwrap();
@@ -115,25 +115,25 @@ impl<'a> StorageMethods for &'a Storage {
     }
 
     // check-tidy: no specs after this line
-    fn NamedGetter(self, name: DOMString, found: &mut bool) -> Option<DOMString> {
+    fn NamedGetter(&self, name: DOMString, found: &mut bool) -> Option<DOMString> {
         let item = self.GetItem(name);
         *found = item.is_some();
         item
     }
 
-    fn NamedSetter(self, name: DOMString, value: DOMString) {
+    fn NamedSetter(&self, name: DOMString, value: DOMString) {
         self.SetItem(name, value);
     }
 
-    fn NamedCreator(self, name: DOMString, value: DOMString) {
+    fn NamedCreator(&self, name: DOMString, value: DOMString) {
         self.SetItem(name, value);
     }
 
-    fn NamedDeleter(self, name: DOMString) {
+    fn NamedDeleter(&self, name: DOMString) {
         self.RemoveItem(name);
     }
 
-    fn SupportedPropertyNames(self) -> Vec<DOMString> {
+    fn SupportedPropertyNames(&self) -> Vec<DOMString> {
         // FIXME: unimplemented (https://github.com/servo/servo/issues/7273)
         vec![]
     }