Remove Reflectable::GetParentObject.

23 files changed, 3 insertions, 154 deletions

diff --git a/src/components/script/dom/bindings/element.rs b/src/components/script/dom/bindings/element.rs
index 8ad84023789..a6b0d37481b 100644
--- a/src/components/script/dom/bindings/element.rs
+++ b/src/components/script/dom/bindings/element.rs
@@ -5,7 +5,7 @@
 use dom::types::*;
 use dom::bindings::utils::{Reflectable, Reflector, Traceable};
 
-use js::jsapi::{JSContext, JSTracer};
+use js::jsapi::JSTracer;
 
 // generate_cacheable_wrapper
 macro_rules! generate_cacheable_wrapper(
@@ -48,10 +48,6 @@ macro_rules! generate_cacheable_wrapper_base(
             fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
                 self.$parent.mut_reflector()
             }
-
-            fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-                self.$parent.GetParentObject(cx)
-            }
         }
     )
 )
diff --git a/src/components/script/dom/bindings/node.rs b/src/components/script/dom/bindings/node.rs
index 1ad5c3b782a..561e22cc637 100644
--- a/src/components/script/dom/bindings/node.rs
+++ b/src/components/script/dom/bindings/node.rs
@@ -9,7 +9,7 @@ use dom::node::{AbstractNode, ScriptView};
 use std::cast;
 use std::libc;
 use std::ptr;
-use js::jsapi::{JSContext, JSTracer, JSTRACE_OBJECT, JS_CallTracer};
+use js::jsapi::{JSTracer, JSTRACE_OBJECT, JS_CallTracer};
 use servo_util::tree::TreeNodeRef;
 
 impl Reflectable for AbstractNode<ScriptView> {
@@ -20,10 +20,6 @@ impl Reflectable for AbstractNode<ScriptView> {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.mut_node().mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.node().GetParentObject(cx)
-    }
 }
 
 impl Traceable for Node<ScriptView> {
diff --git a/src/components/script/dom/bindings/utils.rs b/src/components/script/dom/bindings/utils.rs
index 9f258c2424f..e75e9d5198f 100644
--- a/src/components/script/dom/bindings/utils.rs
+++ b/src/components/script/dom/bindings/utils.rs
@@ -540,7 +540,6 @@ pub fn initialize_global(global: *JSObject) {
 pub trait Reflectable {
     fn reflector<'a>(&'a self) -> &'a Reflector;
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector;
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable>;
 }
 
 pub fn reflect_dom_object<T: Reflectable>
diff --git a/src/components/script/dom/blob.rs b/src/components/script/dom/blob.rs
index 90dcfc8a0ac..4b7d37aa7e9 100644
--- a/src/components/script/dom/blob.rs
+++ b/src/components/script/dom/blob.rs
@@ -7,8 +7,6 @@ use dom::bindings::utils::Fallible;
 use dom::bindings::codegen::BlobBinding;
 use dom::window::Window;
 
-use js::jsapi::JSContext;
-
 pub struct Blob {
     reflector_: Reflector,
     window: @mut Window,
@@ -41,8 +39,4 @@ impl Reflectable for Blob {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.window as @mut Reflectable)
-    }
 }
diff --git a/src/components/script/dom/characterdata.rs b/src/components/script/dom/characterdata.rs
index 4434bdcabf9..f84ebbbdb50 100644
--- a/src/components/script/dom/characterdata.rs
+++ b/src/components/script/dom/characterdata.rs
@@ -9,8 +9,6 @@ use dom::bindings::utils::{Reflectable, Reflector};
 use dom::document::AbstractDocument;
 use dom::node::{Node, NodeTypeId, ScriptView};
 
-use js::jsapi::JSContext;
-
 pub struct CharacterData {
     node: Node<ScriptView>,
     data: ~str
@@ -67,8 +65,4 @@ impl Reflectable for CharacterData {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.node.mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.node.GetParentObject(cx)
-    }
 }
diff --git a/src/components/script/dom/clientrect.rs b/src/components/script/dom/clientrect.rs
index 9704c8ec4c5..1f8014f2ac1 100644
--- a/src/components/script/dom/clientrect.rs
+++ b/src/components/script/dom/clientrect.rs
@@ -6,8 +6,6 @@ use dom::bindings::codegen::ClientRectBinding;
 use dom::bindings::utils::{Reflectable, Reflector, reflect_dom_object};
 use dom::window::Window;
 
-use js::jsapi::JSContext;
-
 pub struct ClientRect {
     reflector_: Reflector,
     top: f32,
@@ -71,8 +69,4 @@ impl Reflectable for ClientRect {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.window as @mut Reflectable)
-    }
 }
diff --git a/src/components/script/dom/clientrectlist.rs b/src/components/script/dom/clientrectlist.rs
index 9f615294ae4..7d221754fa3 100644
--- a/src/components/script/dom/clientrectlist.rs
+++ b/src/components/script/dom/clientrectlist.rs
@@ -7,8 +7,6 @@ use dom::bindings::utils::{Reflectable, Reflector, reflect_dom_object};
 use dom::clientrect::ClientRect;
 use dom::window::Window;
 
-use js::jsapi::JSContext;
-
 pub struct ClientRectList {
     reflector_: Reflector,
     rects: ~[@mut ClientRect],
@@ -57,8 +55,4 @@ impl Reflectable for ClientRectList {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.window as @mut Reflectable)
-    }
 }
diff --git a/src/components/script/dom/document.rs b/src/components/script/dom/document.rs
index c828e7f24d9..9a978027b00 100644
--- a/src/components/script/dom/document.rs
+++ b/src/components/script/dom/document.rs
@@ -158,10 +158,6 @@ impl Reflectable for AbstractDocument {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.mut_document().mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.document().GetParentObject(cx)
-    }
 }
 
 impl Reflectable for Document {
@@ -172,10 +168,6 @@ impl Reflectable for Document {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.node.mut_reflector()
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.window as @mut Reflectable)
-    }
 }
 
 impl Document {
diff --git a/src/components/script/dom/domparser.rs b/src/components/script/dom/domparser.rs
index df0b0d4f148..7fc874d05d1 100644
--- a/src/components/script/dom/domparser.rs
+++ b/src/components/script/dom/domparser.rs
@@ -10,8 +10,6 @@ use dom::document::{AbstractDocument, Document, XML};
 use dom::htmldocument::HTMLDocument;
 use dom::window::Window;
 
-use js::jsapi::JSContext;
-
 pub struct DOMParser {
     owner: @mut Window, //XXXjdm Document instead?
     reflector_: Reflector
@@ -60,8 +58,4 @@ impl Reflectable for DOMParser {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.owner as @mut Reflectable)
-    }
 }
diff --git a/src/components/script/dom/element.rs b/src/components/script/dom/element.rs
index 3cfddaa698b..24644e514af 100644
--- a/src/components/script/dom/element.rs
+++ b/src/components/script/dom/element.rs
@@ -16,8 +16,6 @@ use layout_interface::{ContentBoxesResponse};
 use style;
 use servo_util::tree::{TreeNodeRef, ElementLike};
 
-use js::jsapi::JSContext;
-
 use std::comm;
 use std::hashmap::HashMap;
 use std::ascii::StrAsciiExt;
@@ -38,10 +36,6 @@ impl Reflectable for Element {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.node.mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.node.GetParentObject(cx)
-    }
 }
 
 #[deriving(Eq)]
diff --git a/src/components/script/dom/event.rs b/src/components/script/dom/event.rs
index 3683ec757ac..aa70ade8f5c 100644
--- a/src/components/script/dom/event.rs
+++ b/src/components/script/dom/event.rs
@@ -11,9 +11,6 @@ use dom::mouseevent::MouseEvent;
 use dom::uievent::UIEvent;
 
 use geom::point::Point2D;
-use js::jsapi::JSContext;
-
-use script_task::page_from_context;
 
 use std::cast;
 use std::unstable::raw::Box;
@@ -119,10 +116,6 @@ impl Reflectable for AbstractEvent {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.mut_event().mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.event().GetParentObject(cx)
-    }
 }
 
 #[deriving(Eq)]
@@ -263,11 +256,4 @@ impl Reflectable for Event {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        let page = page_from_context(cx);
-        unsafe {
-            Some((*page).frame.get_ref().window as @mut Reflectable)
-        }
-    }
 }
diff --git a/src/components/script/dom/eventtarget.rs b/src/components/script/dom/eventtarget.rs
index ddcdda44b16..26df47ef585 100644
--- a/src/components/script/dom/eventtarget.rs
+++ b/src/components/script/dom/eventtarget.rs
@@ -8,9 +8,6 @@ use dom::bindings::codegen::EventListenerBinding::EventListener;
 use dom::event::AbstractEvent;
 use dom::eventdispatcher::dispatch_event;
 use dom::node::{AbstractNode, ScriptView};
-use script_task::page_from_context;
-
-use js::jsapi::JSContext;
 
 use std::cast;
 use std::hashmap::HashMap;
@@ -104,10 +101,6 @@ impl Reflectable for AbstractEventTarget {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.mut_eventtarget().mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.eventtarget().GetParentObject(cx)
-    }
 }
 
 impl EventTarget {
@@ -186,12 +179,4 @@ impl Reflectable for EventTarget {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        let page = page_from_context(cx);
-        // TODO(tkuehn): This only handles top-level pages. Needs to handle subframes.
-        unsafe {
-            Some((*page).frame.get_ref().window as @mut Reflectable)
-        }
-    }
 }
diff --git a/src/components/script/dom/formdata.rs b/src/components/script/dom/formdata.rs
index e5b7d9d4881..de34386bcac 100644
--- a/src/components/script/dom/formdata.rs
+++ b/src/components/script/dom/formdata.rs
@@ -8,8 +8,6 @@ use dom::bindings::codegen::FormDataBinding;
 use dom::blob::Blob;
 use dom::window::Window;
 
-use js::jsapi::JSContext;
-
 use std::hashmap::HashMap;
 
 enum FormDatum {
@@ -57,8 +55,4 @@ impl Reflectable for FormData {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.window as @mut Reflectable)
-    }
 }
diff --git a/src/components/script/dom/htmlcollection.rs b/src/components/script/dom/htmlcollection.rs
index 67e5560e1f0..03a5ebac878 100644
--- a/src/components/script/dom/htmlcollection.rs
+++ b/src/components/script/dom/htmlcollection.rs
@@ -68,8 +68,4 @@ impl Reflectable for HTMLCollection {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.window as @mut Reflectable)
-    }
 }
diff --git a/src/components/script/dom/htmldocument.rs b/src/components/script/dom/htmldocument.rs
index dc53c9f4700..6751d88fe61 100644
--- a/src/components/script/dom/htmldocument.rs
+++ b/src/components/script/dom/htmldocument.rs
@@ -10,7 +10,7 @@ use dom::htmlcollection::HTMLCollection;
 use dom::node::{AbstractNode, ScriptView, ElementNodeTypeId};
 use dom::window::Window;
 
-use js::jsapi::{JSContext, JSTracer};
+use js::jsapi::JSTracer;
 
 use servo_util::tree::{TreeNodeRef, ElementLike};
 
@@ -88,10 +88,6 @@ impl Reflectable for HTMLDocument {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.parent.mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.parent.GetParentObject(cx)
-    }
 }
 
 impl Traceable for HTMLDocument {
diff --git a/src/components/script/dom/mouseevent.rs b/src/components/script/dom/mouseevent.rs
index 4bf4a4ae069..30dae36521e 100644
--- a/src/components/script/dom/mouseevent.rs
+++ b/src/components/script/dom/mouseevent.rs
@@ -11,8 +11,6 @@ use dom::uievent::UIEvent;
 use dom::window::Window;
 use dom::windowproxy::WindowProxy;
 
-use js::jsapi::JSContext;
-
 pub struct MouseEvent {
     parent: UIEvent,
     screen_x: i32,
@@ -151,8 +149,4 @@ impl Reflectable for MouseEvent {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.parent.mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.parent.GetParentObject(cx)
-    }
 }
diff --git a/src/components/script/dom/navigator.rs b/src/components/script/dom/navigator.rs
index fa256a35990..7dbd81959b5 100644
--- a/src/components/script/dom/navigator.rs
+++ b/src/components/script/dom/navigator.rs
@@ -6,9 +6,6 @@ use dom::bindings::utils::{Reflectable, Reflector, reflect_dom_object};
 use dom::bindings::utils::{DOMString, Fallible};
 use dom::bindings::codegen::NavigatorBinding;
 use dom::window::Window;
-use script_task::{page_from_context};
-
-use js::jsapi::JSContext;
 
 pub struct Navigator {
     reflector_: Reflector //XXXjdm cycle: window->navigator->window
@@ -98,11 +95,4 @@ impl Reflectable for Navigator {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        let page = page_from_context(cx);
-        unsafe {
-            Some((*page).frame.get_ref().window as @mut Reflectable)
-        }
-    }
 }
diff --git a/src/components/script/dom/node.rs b/src/components/script/dom/node.rs
index e2db42afe80..a6cf77d0c19 100644
--- a/src/components/script/dom/node.rs
+++ b/src/components/script/dom/node.rs
@@ -1056,13 +1056,6 @@ impl Reflectable for Node<ScriptView> {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.eventtarget.mut_reflector()
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        match self.parent_node {
-            Some(node) => Some(unsafe {node.as_cacheable_wrapper()}),
-            None => None
-        }
-    }
 }
 
 // This stuff is notionally private to layout, but we put it here because it needs
diff --git a/src/components/script/dom/nodelist.rs b/src/components/script/dom/nodelist.rs
index 2500f106f15..ce44222a604 100644
--- a/src/components/script/dom/nodelist.rs
+++ b/src/components/script/dom/nodelist.rs
@@ -7,8 +7,6 @@ use dom::bindings::utils::{Reflectable, Reflector, reflect_dom_object};
 use dom::node::{AbstractNode, ScriptView};
 use dom::window::Window;
 
-use js::jsapi::JSContext;
-
 enum NodeListType {
     Simple(~[AbstractNode<ScriptView>]),
     Children(AbstractNode<ScriptView>)
@@ -74,8 +72,4 @@ impl Reflectable for NodeList {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.window as @mut Reflectable)
-    }
 }
diff --git a/src/components/script/dom/uievent.rs b/src/components/script/dom/uievent.rs
index 97c193c846f..b3a48c69d2b 100644
--- a/src/components/script/dom/uievent.rs
+++ b/src/components/script/dom/uievent.rs
@@ -10,8 +10,6 @@ use dom::event::{AbstractEvent, Event, EventTypeId, UIEventTypeId};
 use dom::window::Window;
 use dom::windowproxy::WindowProxy;
 
-use js::jsapi::JSContext;
-
 pub struct UIEvent {
     parent: Event,
     view: Option<@mut WindowProxy>,
@@ -120,8 +118,4 @@ impl Reflectable for UIEvent {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.parent.mut_reflector()
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        self.parent.GetParentObject(cx)
-    }
 }
diff --git a/src/components/script/dom/validitystate.rs b/src/components/script/dom/validitystate.rs
index 471964c0737..562b58a16c6 100644
--- a/src/components/script/dom/validitystate.rs
+++ b/src/components/script/dom/validitystate.rs
@@ -6,8 +6,6 @@ use dom::bindings::utils::{Reflectable, Reflector, reflect_dom_object};
 use dom::bindings::codegen::ValidityStateBinding;
 use dom::window::Window;
 
-use js::jsapi::JSContext;
-
 pub struct ValidityState {
     reflector_: Reflector,
     window: @mut Window,
@@ -75,8 +73,4 @@ impl Reflectable for ValidityState {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        Some(self.window as @mut Reflectable)
-    }
 }
diff --git a/src/components/script/dom/window.rs b/src/components/script/dom/window.rs
index e148cd8a2ad..b4a804464f5 100644
--- a/src/components/script/dom/window.rs
+++ b/src/components/script/dom/window.rs
@@ -147,10 +147,6 @@ impl Reflectable for Window {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         self.eventtarget.mut_reflector()
     }
-
-    fn GetParentObject(&self, _cx: *JSContext) -> Option<@mut Reflectable> {
-        None
-    }
 }
 
 impl Window {
diff --git a/src/components/script/dom/windowproxy.rs b/src/components/script/dom/windowproxy.rs
index fada574cb8b..bae5c5b933f 100644
--- a/src/components/script/dom/windowproxy.rs
+++ b/src/components/script/dom/windowproxy.rs
@@ -3,9 +3,6 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 use dom::bindings::utils::{Reflectable, Reflector};
-use script_task::page_from_context;
-
-use js::jsapi::JSContext;
 
 pub struct WindowProxy {
     reflector_: Reflector
@@ -27,11 +24,4 @@ impl Reflectable for WindowProxy {
     fn mut_reflector<'a>(&'a mut self) -> &'a mut Reflector {
         &mut self.reflector_
     }
-
-    fn GetParentObject(&self, cx: *JSContext) -> Option<@mut Reflectable> {
-        let page = page_from_context(cx);
-        unsafe {
-            Some((*page).frame.get_ref().window as @mut Reflectable)
-        }
-    }
 }