Implement window.openURLInDefaultBrowser() (fixes #11292)

components/script/Cargo.toml

@@ -43,6 +43,7 @@ msg = {path = "../msg"}
 net_traits = {path = "../net_traits"}
 num-traits = "0.1.32"
 offscreen_gl_context = "0.1.2"
+open = "1.1.1"
 phf = "0.7.13"
 phf_macros = "0.7.13"
 plugins = {path = "../plugins"}

components/script/dom/webidls/BrowserElement.webidl

@@ -146,16 +146,16 @@ interface BrowserElementPrivileged {
   //                    unsigned long count,
   //                    unsigned long modifiers);
 
-  [Func="Window::global_is_mozbrowser", Throws]
+  [Throws]
   void goBack();
 
-  [Func="Window::global_is_mozbrowser", Throws]
+  [Throws]
   void goForward();
 
-  [Func="Window::global_is_mozbrowser", Throws]
+  [Throws]
   void reload(optional boolean hardReload = false);
 
-  [Func="Window::global_is_mozbrowser", Throws]
+  [Throws]
   void stop();
 
   //[Throws,

components/script/dom/webidls/Window.webidl

@@ -159,6 +159,8 @@ partial interface Window {
   void debug(DOMString arg);
   void gc();
   void trap();
+  [Func="Window::global_is_mozbrowser", Throws]
+  void openURLInDefaultBrowser(DOMString href);
 };
 
 // WebDriver extensions

components/script/dom/window.rs

@@ -13,7 +13,7 @@ use dom::bindings::codegen::Bindings::EventHandlerBinding::OnErrorEventHandlerNo
 use dom::bindings::codegen::Bindings::FunctionBinding::Function;
 use dom::bindings::codegen::Bindings::WindowBinding::{ScrollBehavior, ScrollToOptions};
 use dom::bindings::codegen::Bindings::WindowBinding::{self, FrameRequestCallback, WindowMethods};
-use dom::bindings::error::{Error, Fallible, report_pending_exception};
+use dom::bindings::error::{Error, ErrorResult, Fallible, report_pending_exception};
 use dom::bindings::global::{GlobalRef, global_root_from_object};
 use dom::bindings::inheritance::Castable;
 use dom::bindings::js::{JS, MutNullableHeap, Root};
@@ -52,6 +52,7 @@ use net_traits::image_cache_thread::{ImageCacheChan, ImageCacheThread};
 use net_traits::storage_thread::StorageType;
 use net_traits::{ResourceThreads, CustomResponseSender};
 use num_traits::ToPrimitive;
+use open;
 use profile_traits::mem;
 use profile_traits::time::{ProfilerCategory, TimerMetadata, TimerMetadataFrameType};
 use profile_traits::time::{ProfilerChan, TimerMetadataReflowType, profile};
@@ -839,6 +840,17 @@ impl WindowMethods for Window {
     fn SetStatus(&self, status: DOMString) {
         *self.status.borrow_mut() = status
     }
+
+    // check-tidy: no specs after this line
+    fn OpenURLInDefaultBrowser(&self, href: DOMString) -> ErrorResult {
+        let url = try!(Url::parse(&href).map_err(|e| {
+            Error::Type(format!("Couldn't parse URL: {}", e))
+        }));
+        match open::that(url.as_str()) {
+            Ok(_) => Ok(()),
+            Err(e) => Err(Error::Type(format!("Couldn't open URL: {}", e))),
+        }
+    }
 }
 
 pub trait ScriptHelpers {

components/script/lib.rs

@@ -59,6 +59,7 @@ extern crate msg;
 extern crate net_traits;
 extern crate num_traits;
 extern crate offscreen_gl_context;
+extern crate open;
 extern crate phf;
 #[macro_use]
 extern crate profile_traits;

components/servo/Cargo.lock

@@ -1565,6 +1565,11 @@ dependencies = [
  "x11 2.5.1 (registry+https://github.com/rust-lang/crates.io-index)",
 ]
 
+[[package]]
+name = "open"
+version = "1.1.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+
 [[package]]
 name = "openssl"
 version = "0.7.13"
@@ -1877,6 +1882,7 @@ dependencies = [
  "net_traits 0.0.1",
  "num-traits 0.1.32 (registry+https://github.com/rust-lang/crates.io-index)",
  "offscreen_gl_context 0.1.5 (registry+https://github.com/rust-lang/crates.io-index)",
+ "open 1.1.1 (registry+https://github.com/rust-lang/crates.io-index)",
  "phf 0.7.14 (registry+https://github.com/rust-lang/crates.io-index)",
  "phf_macros 0.7.14 (registry+https://github.com/rust-lang/crates.io-index)",
  "plugins 0.0.1",

ports/cef/Cargo.lock

@@ -1443,6 +1443,11 @@ dependencies = [
  "x11 2.5.1 (registry+https://github.com/rust-lang/crates.io-index)",
 ]
 
+[[package]]
+name = "open"
+version = "1.1.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+
 [[package]]
 name = "openssl"
 version = "0.7.13"
@@ -1735,6 +1740,7 @@ dependencies = [
  "net_traits 0.0.1",
  "num-traits 0.1.32 (registry+https://github.com/rust-lang/crates.io-index)",
  "offscreen_gl_context 0.1.5 (registry+https://github.com/rust-lang/crates.io-index)",
+ "open 1.1.1 (registry+https://github.com/rust-lang/crates.io-index)",
  "phf 0.7.14 (registry+https://github.com/rust-lang/crates.io-index)",
  "phf_macros 0.7.14 (registry+https://github.com/rust-lang/crates.io-index)",
  "plugins 0.0.1",

ports/gonk/Cargo.lock

@@ -1431,6 +1431,11 @@ dependencies = [
  "x11 2.5.1 (registry+https://github.com/rust-lang/crates.io-index)",
 ]
 
+[[package]]
+name = "open"
+version = "1.1.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+
 [[package]]
 name = "openssl"
 version = "0.7.13"
@@ -1723,6 +1728,7 @@ dependencies = [
  "net_traits 0.0.1",
  "num-traits 0.1.32 (registry+https://github.com/rust-lang/crates.io-index)",
  "offscreen_gl_context 0.1.5 (registry+https://github.com/rust-lang/crates.io-index)",
+ "open 1.1.1 (registry+https://github.com/rust-lang/crates.io-index)",
  "phf 0.7.14 (registry+https://github.com/rust-lang/crates.io-index)",
  "phf_macros 0.7.14 (registry+https://github.com/rust-lang/crates.io-index)",
  "plugins 0.0.1",