Auto merge of #7401 - nox:rm-helpers, r=Manishearth

Remove helper traits

Now that `JSRef<T>` is gone, there is no need to have helper traits.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7401)
<!-- Reviewable:end -->

1 files changed, 3 insertions, 8 deletions

diff --git a/components/script/dom/storage.rs b/components/script/dom/storage.rs
index 8d5d78d56a6..29872ab068a 100644
--- a/components/script/dom/storage.rs
+++ b/components/script/dom/storage.rs
@@ -9,10 +9,9 @@ use dom::bindings::global::{GlobalRef, GlobalField};
 use dom::bindings::js::{Root, RootedReference};
 use dom::bindings::refcounted::Trusted;
 use dom::bindings::utils::{Reflector, reflect_dom_object};
-use dom::event::{EventHelpers, EventBubbles, EventCancelable};
+use dom::event::{EventBubbles, EventCancelable};
 use dom::storageevent::StorageEvent;
 use dom::urlhelper::UrlHelper;
-use dom::window::WindowHelpers;
 use ipc_channel::ipc;
 use net_traits::storage_task::{StorageTask, StorageTaskMsg, StorageType};
 use page::IterablePage;
@@ -140,14 +139,10 @@ impl<'a> StorageMethods for &'a Storage {
     }
 }
 
-trait PrivateStorageHelpers {
-    fn broadcast_change_notification(self, key: Option<DOMString>, old_value: Option<DOMString>,
-                                     new_value: Option<DOMString>);
-}
 
-impl<'a> PrivateStorageHelpers for &'a Storage {
+impl Storage {
     /// https://html.spec.whatwg.org/multipage/#send-a-storage-notification
-    fn broadcast_change_notification(self, key: Option<DOMString>, old_value: Option<DOMString>,
+    fn broadcast_change_notification(&self, key: Option<DOMString>, old_value: Option<DOMString>,
                                      new_value: Option<DOMString>) {
         let global_root = self.global.root();
         let global_ref = global_root.r();