Improve API for getting current window

fxfactorial · Nov 3, 2015 · edc3a19 · edc3a19
1 parent 4344e94
commit edc3a19
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/electron_main.ml b/src/electron_main.ml
@@ -257,7 +257,7 @@ module Browser_window = struct
 
   end
 
-  class browser_window ?(remote=false) ?opts ?existing = object
+  class browser_window ?(remote=false) ?opts existing = object
 
     val raw_js =
       let result = match opts with

diff --git a/src/electron_renderer.ml b/src/electron_renderer.ml
@@ -18,13 +18,11 @@ module Remote = struct
     val raw_js = require_module "remote"
 
     method browser_window (o : b_opts) =
-      new B.browser_window ~remote:true ~existing:None ~opts:o
+      new B.browser_window ~remote:true ~opts:o None
 
     method get_current_window =
-      new B.browser_window
-        ~remote:false
-        ~existing:(m raw_js "getCurrentWindow" [||])
-        ?opts:None
+      (Some (m raw_js "getCurrentWindow" [||]))
+      |> new B.browser_window ~remote:false ?opts:None
 
     (* method get_current_web_contents  *)