From 3781567c199512ce2f257d1b2ccf465e634e12ec Mon Sep 17 00:00:00 2001
From: Ms2ger <Ms2ger@gmail.com>
Date: Sun, 13 Sep 2015 16:47:06 +0200
Subject: [PATCH] Stop implementing Reflectable for JS<T> (fixes #2285).

It's not generally safe to expose the contents of a JS<T>.
---
 components/script/dom/bindings/js.rs    | 13 ++++++-------
 components/script/dom/bindings/trace.rs |  6 ------
 2 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/components/script/dom/bindings/js.rs b/components/script/dom/bindings/js.rs
index 5345d08ee9e..e50bf68e478 100644
--- a/components/script/dom/bindings/js.rs
+++ b/components/script/dom/bindings/js.rs
@@ -88,6 +88,12 @@ impl<T: Reflectable> JS<T> {
     }
 }
 
+impl<T: Reflectable> JSTraceable for JS<T> {
+    fn trace(&self, trc: *mut JSTracer) {
+        trace_reflector(trc, "", unsafe { (**self.ptr).reflector() });
+    }
+}
+
 /// An unrooted reference to a DOM object for use in layout. `Layout*Helpers`
 /// traits must be implemented on this.
 #[allow_unrooted_interior]
@@ -148,13 +154,6 @@ impl LayoutJS<Node> {
     }
 }
 
-impl<T: Reflectable> Reflectable for JS<T> {
-    fn reflector(&self) -> &Reflector {
-        unsafe {
-            (**self.ptr).reflector()
-        }
-    }
-}
 
 /// A trait to be implemented for JS-managed types that can be stored in
 /// mutable member fields.
diff --git a/components/script/dom/bindings/trace.rs b/components/script/dom/bindings/trace.rs
index ed16d5a2515..5a9aa5b2abb 100644
--- a/components/script/dom/bindings/trace.rs
+++ b/components/script/dom/bindings/trace.rs
@@ -88,12 +88,6 @@ pub trait JSTraceable {
     fn trace(&self, trc: *mut JSTracer);
 }
 
-impl<T: Reflectable> JSTraceable for JS<T> {
-    fn trace(&self, trc: *mut JSTracer) {
-        trace_reflector(trc, "", self.reflector());
-    }
-}
-
 no_jsmanaged_fields!(EncodingRef);
 
 no_jsmanaged_fields!(Reflector);