Remove dead error handling code.

servo · Sep 5, 2016 · 59904e6 · 59904e6
1 parent b305085
commit 59904e6
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 76 deletions.
diff --git a/components/script/dom/abstractworker.rs b/components/script/dom/abstractworker.rs
@@ -4,7 +4,6 @@
 
 use dom::bindings::refcounted::Trusted;
 use dom::bindings::reflector::Reflectable;
-use dom::bindings::str::DOMString;
 use dom::bindings::structuredclone::StructuredCloneData;
 use js::jsapi::{JSRuntime, JS_RequestInterruptCallback};
 use js::rust::Runtime;
@@ -30,27 +29,6 @@ impl<T: Reflectable> SimpleWorkerErrorHandler<T> {
     }
 }
 
-pub struct WorkerErrorHandler<T: Reflectable> {
-    pub addr: Trusted<T>,
-    pub msg: DOMString,
-    pub file_name: DOMString,
-    pub line_num: u32,
-    pub col_num: u32,
-}
-
-impl<T: Reflectable> WorkerErrorHandler<T> {
-    pub fn new(addr: Trusted<T>, msg: DOMString, file_name: DOMString, line_num: u32, col_num: u32)
-            -> WorkerErrorHandler<T> {
-        WorkerErrorHandler {
-            addr: addr,
-            msg: msg,
-            file_name: file_name,
-            line_num: line_num,
-            col_num: col_num,
-        }
-    }
-}
-
 #[derive(Copy, Clone)]
 pub struct SharedRt {
     pub rt: *mut JSRuntime

diff --git a/components/script/dom/serviceworker.rs b/components/script/dom/serviceworker.rs
@@ -2,21 +2,17 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
-use dom::abstractworker::WorkerScriptMsg;
-use dom::abstractworker::{SimpleWorkerErrorHandler, WorkerErrorHandler};
+use dom::abstractworker::{SimpleWorkerErrorHandler, WorkerScriptMsg};
 use dom::bindings::cell::DOMRefCell;
 use dom::bindings::codegen::Bindings::EventHandlerBinding::EventHandlerNonNull;
 use dom::bindings::codegen::Bindings::ServiceWorkerBinding::{ServiceWorkerMethods, ServiceWorkerState, Wrap};
 use dom::bindings::global::GlobalRef;
 use dom::bindings::inheritance::Castable;
 use dom::bindings::js::Root;
 use dom::bindings::refcounted::Trusted;
-use dom::bindings::reflector::{Reflectable, reflect_dom_object};
-use dom::bindings::str::{DOMString, USVString};
-use dom::errorevent::ErrorEvent;
-use dom::event::{Event, EventBubbles, EventCancelable};
+use dom::bindings::reflector::reflect_dom_object;
+use dom::bindings::str::USVString;
 use dom::eventtarget::EventTarget;
-use js::jsval::UndefinedValue;
 use script_thread::Runnable;
 use std::cell::Cell;
 use std::sync::mpsc::Sender;
@@ -66,18 +62,6 @@ impl ServiceWorker {
         Url::parse(&self.script_url.borrow().clone()).unwrap()
     }
 
-    pub fn handle_error_message(address: TrustedServiceWorkerAddress, message: DOMString,
-                                filename: DOMString, lineno: u32, colno: u32) {
-        let worker = address.root();
-
-        let global = worker.r().global();
-        rooted!(in(global.r().get_cx()) let error = UndefinedValue());
-        let errorevent = ErrorEvent::new(global.r(), atom!("error"),
-                                         EventBubbles::Bubbles, EventCancelable::Cancelable,
-                                         message, filename, lineno, colno, error.handle());
-        errorevent.upcast::<Event>().fire(worker.upcast());
-    }
-
     pub fn install_serviceworker(global: GlobalRef,
                                  script_url: Url,
                                  skip_waiting: bool) -> Root<ServiceWorker> {
@@ -112,11 +96,3 @@ impl Runnable for SimpleWorkerErrorHandler<ServiceWorker> {
         ServiceWorker::dispatch_simple_error(this.addr);
     }
 }
-
-impl Runnable for WorkerErrorHandler<ServiceWorker> {
-    #[allow(unrooted_must_root)]
-    fn handler(self: Box<WorkerErrorHandler<ServiceWorker>>) {
-        let this = *self;
-        ServiceWorker::handle_error_message(this.addr, this.msg, this.file_name, this.line_num, this.col_num);
-    }
-}
diff --git a/components/script/dom/worker.rs b/components/script/dom/worker.rs
@@ -4,7 +4,7 @@
 
 use devtools_traits::{ScriptToDevtoolsControlMsg, DevtoolsPageInfo};
 use dom::abstractworker::WorkerScriptMsg;
-use dom::abstractworker::{SimpleWorkerErrorHandler, SharedRt, WorkerErrorHandler};
+use dom::abstractworker::{SimpleWorkerErrorHandler, SharedRt};
 use dom::bindings::codegen::Bindings::EventHandlerBinding::EventHandlerNonNull;
 use dom::bindings::codegen::Bindings::WorkerBinding;
 use dom::bindings::codegen::Bindings::WorkerBinding::WorkerMethods;
@@ -17,8 +17,6 @@ use dom::bindings::reflector::{Reflectable, reflect_dom_object};
 use dom::bindings::str::DOMString;
 use dom::bindings::structuredclone::StructuredCloneData;
 use dom::dedicatedworkerglobalscope::DedicatedWorkerGlobalScope;
-use dom::errorevent::ErrorEvent;
-use dom::event::{Event, EventBubbles, EventCancelable};
 use dom::eventtarget::EventTarget;
 use dom::messageevent::MessageEvent;
 use dom::workerglobalscope::prepare_workerscope_init;
@@ -139,22 +137,6 @@ impl Worker {
         let worker = address.root();
         worker.upcast().fire_simple_event("error");
     }
-
-    pub fn handle_error_message(address: TrustedWorkerAddress, message: DOMString,
-                                filename: DOMString, lineno: u32, colno: u32) {
-        let worker = address.root();
-
-        if worker.is_terminated() {
-            return;
-        }
-
-        let global = worker.r().global();
-        rooted!(in(global.r().get_cx()) let error = UndefinedValue());
-        let errorevent = ErrorEvent::new(global.r(), atom!("error"),
-                                         EventBubbles::Bubbles, EventCancelable::Cancelable,
-                                         message, filename, lineno, colno, error.handle());
-        errorevent.upcast::<Event>().fire(worker.upcast());
-    }
 }
 
 impl WorkerMethods for Worker {
@@ -220,11 +202,3 @@ impl Runnable for SimpleWorkerErrorHandler<Worker> {
         Worker::dispatch_simple_error(this.addr);
     }
 }
-
-impl Runnable for WorkerErrorHandler<Worker> {
-    #[allow(unrooted_must_root)]
-    fn handler(self: Box<WorkerErrorHandler<Worker>>) {
-        let this = *self;
-        Worker::handle_error_message(this.addr, this.msg, this.file_name, this.line_num, this.col_num);
-    }
-}