aboutsummaryrefslogtreecommitdiffstats
path: root/components/script/devtools.rs
diff options
context:
space:
mode:
Diffstat (limited to 'components/script/devtools.rs')
-rw-r--r--components/script/devtools.rs67
1 files changed, 34 insertions, 33 deletions
diff --git a/components/script/devtools.rs b/components/script/devtools.rs
index f14e19999a5..ea79a3850a1 100644
--- a/components/script/devtools.rs
+++ b/components/script/devtools.rs
@@ -6,18 +6,19 @@ use devtools_traits::{CachedConsoleMessage, CachedConsoleMessageTypes, PAGE_ERRO
use devtools_traits::{EvaluateJSReply, NodeInfo, Modification, TimelineMarker, TimelineMarkerType};
use dom::bindings::conversions::FromJSValConvertible;
use dom::bindings::conversions::StringificationBehavior;
-use dom::bindings::js::{JSRef, OptionalRootable, Rootable, Temporary};
+use dom::bindings::js::Root;
use dom::bindings::codegen::InheritTypes::{NodeCast, ElementCast};
use dom::bindings::codegen::Bindings::DocumentBinding::DocumentMethods;
use dom::bindings::codegen::Bindings::DOMRectBinding::{DOMRectMethods};
use dom::bindings::codegen::Bindings::ElementBinding::{ElementMethods};
use dom::node::{Node, NodeHelpers};
use dom::window::{WindowHelpers, ScriptHelpers};
-use dom::element::Element;
use dom::document::DocumentHelpers;
use page::{IterablePage, Page};
use msg::constellation_msg::PipelineId;
use script_task::{get_page, ScriptTask};
+use js::jsapi::RootedValue;
+use js::jsval::UndefinedValue;
use std::sync::mpsc::Sender;
use std::rc::Rc;
@@ -25,53 +26,54 @@ use std::rc::Rc;
pub fn handle_evaluate_js(page: &Rc<Page>, pipeline: PipelineId, eval: String, reply: Sender<EvaluateJSReply>){
let page = get_page(&*page, pipeline);
- let window = page.window().root();
+ let window = page.window();
let cx = window.r().get_cx();
- let rval = window.r().evaluate_js_on_global_with_result(&eval);
+ let mut rval = RootedValue::new(cx, UndefinedValue());
+ window.r().evaluate_js_on_global_with_result(&eval, rval.handle_mut());
- reply.send(if rval.is_undefined() {
+ reply.send(if rval.ptr.is_undefined() {
EvaluateJSReply::VoidValue
- } else if rval.is_boolean() {
- EvaluateJSReply::BooleanValue(rval.to_boolean())
- } else if rval.is_double() {
- EvaluateJSReply::NumberValue(FromJSValConvertible::from_jsval(cx, rval, ()).unwrap())
- } else if rval.is_string() {
+ } else if rval.ptr.is_boolean() {
+ EvaluateJSReply::BooleanValue(rval.ptr.to_boolean())
+ } else if rval.ptr.is_double() {
+ EvaluateJSReply::NumberValue(FromJSValConvertible::from_jsval(cx, rval.handle(), ()).unwrap())
+ } else if rval.ptr.is_string() {
//FIXME: use jsstring_to_str when jsval grows to_jsstring
EvaluateJSReply::StringValue(
- FromJSValConvertible::from_jsval(cx, rval, StringificationBehavior::Default).unwrap())
- } else if rval.is_null() {
+ FromJSValConvertible::from_jsval(cx, rval.handle(), StringificationBehavior::Default).unwrap())
+ } else if rval.ptr.is_null() {
EvaluateJSReply::NullValue
} else {
//FIXME: jsvals don't have an is_int32/is_number yet
- assert!(rval.is_object());
+ assert!(rval.ptr.is_object());
panic!("object values unimplemented")
}).unwrap();
}
pub fn handle_get_root_node(page: &Rc<Page>, pipeline: PipelineId, reply: Sender<NodeInfo>) {
let page = get_page(&*page, pipeline);
- let document = page.document().root();
+ let document = page.document();
- let node: JSRef<Node> = NodeCast::from_ref(document.r());
+ let node = NodeCast::from_ref(document.r());
reply.send(node.summarize()).unwrap();
}
pub fn handle_get_document_element(page: &Rc<Page>, pipeline: PipelineId, reply: Sender<NodeInfo>) {
let page = get_page(&*page, pipeline);
- let document = page.document().root();
- let document_element = document.r().GetDocumentElement().root().unwrap();
+ let document = page.document();
+ let document_element = document.r().GetDocumentElement().unwrap();
- let node: JSRef<Node> = NodeCast::from_ref(document_element.r());
+ let node = NodeCast::from_ref(document_element.r());
reply.send(node.summarize()).unwrap();
}
-fn find_node_by_unique_id(page: &Rc<Page>, pipeline: PipelineId, node_id: String) -> Temporary<Node> {
+fn find_node_by_unique_id(page: &Rc<Page>, pipeline: PipelineId, node_id: String) -> Root<Node> {
let page = get_page(&*page, pipeline);
- let document = page.document().root();
- let node: JSRef<Node> = NodeCast::from_ref(document.r());
+ let document = page.document();
+ let node = NodeCast::from_ref(document.r());
for candidate in node.traverse_preorder() {
- if candidate.root().r().get_unique_id() == node_id {
+ if candidate.r().get_unique_id() == node_id {
return candidate;
}
}
@@ -80,18 +82,17 @@ fn find_node_by_unique_id(page: &Rc<Page>, pipeline: PipelineId, node_id: String
}
pub fn handle_get_children(page: &Rc<Page>, pipeline: PipelineId, node_id: String, reply: Sender<Vec<NodeInfo>>) {
- let parent = find_node_by_unique_id(&*page, pipeline, node_id).root();
+ let parent = find_node_by_unique_id(&*page, pipeline, node_id);
let children = parent.r().children().map(|child| {
- let child = child.root();
child.r().summarize()
}).collect();
reply.send(children).unwrap();
}
pub fn handle_get_layout(page: &Rc<Page>, pipeline: PipelineId, node_id: String, reply: Sender<(f32, f32)>) {
- let node = find_node_by_unique_id(&*page, pipeline, node_id).root();
- let elem: JSRef<Element> = ElementCast::to_ref(node.r()).expect("should be getting layout of element");
- let rect = elem.GetBoundingClientRect().root();
+ let node = find_node_by_unique_id(&*page, pipeline, node_id);
+ let elem = ElementCast::to_ref(node.r()).expect("should be getting layout of element");
+ let rect = elem.GetBoundingClientRect();
let width = *rect.r().Width();
let height = *rect.r().Height();
reply.send((width, height)).unwrap();
@@ -141,8 +142,8 @@ pub fn handle_modify_attribute(page: &Rc<Page>,
pipeline: PipelineId,
node_id: String,
modifications: Vec<Modification>) {
- let node = find_node_by_unique_id(&*page, pipeline, node_id).root();
- let elem: JSRef<Element> = ElementCast::to_ref(node.r()).expect("should be getting layout of element");
+ let node = find_node_by_unique_id(&*page, pipeline, node_id);
+ let elem = ElementCast::to_ref(node.r()).expect("should be getting layout of element");
for modification in modifications.iter(){
match modification.newValue {
@@ -156,7 +157,7 @@ pub fn handle_modify_attribute(page: &Rc<Page>,
pub fn handle_wants_live_notifications(page: &Rc<Page>, pipeline_id: PipelineId, send_notifications: bool) {
let page = get_page(&*page, pipeline_id);
- let window = page.window().root();
+ let window = page.window();
window.r().set_devtools_wants_updates(send_notifications);
}
@@ -167,7 +168,7 @@ pub fn handle_set_timeline_markers(page: &Rc<Page>,
for marker_type in &marker_types {
match *marker_type {
TimelineMarkerType::Reflow => {
- let window = page.window().root();
+ let window = page.window();
window.r().set_devtools_timeline_marker(TimelineMarkerType::Reflow, reply.clone());
}
TimelineMarkerType::DOMEvent => {
@@ -180,7 +181,7 @@ pub fn handle_set_timeline_markers(page: &Rc<Page>,
pub fn handle_drop_timeline_markers(page: &Rc<Page>,
script_task: &ScriptTask,
marker_types: Vec<TimelineMarkerType>) {
- let window = page.window().root();
+ let window = page.window();
for marker_type in &marker_types {
match *marker_type {
TimelineMarkerType::Reflow => {
@@ -195,6 +196,6 @@ pub fn handle_drop_timeline_markers(page: &Rc<Page>,
pub fn handle_request_animation_frame(page: &Rc<Page>, id: PipelineId, callback: Box<Fn(f64, )>) {
let page = page.find(id).expect("There is no such page");
- let doc = page.document().root();
+ let doc = page.document();
doc.r().request_animation_frame(callback);
}