Remove Reflectable::GetParentObject.

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)
-            }
         }
     )
 )

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> {

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>

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)
-    }
 }

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)
-    }
 }

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)
-    }
 }

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)
-    }
 }

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 {

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)
-    }
 }

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)]

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)
-        }
-    }
 }

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)
-        }
-    }
 }

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)
-    }
 }

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)
-    }
 }

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 {

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)
-    }
 }

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)
-        }
-    }
 }

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

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)
-    }
 }

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)
-    }
 }

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)
-    }
 }

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 {

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)
-        }
-    }
 }