Handle remote closed andrewvy#37

Author: aus70

lib/page_session.ex

@@ -119,6 +119,7 @@ defmodule ChromeRemoteInterface.PageSession do
   # ---
 
   def init(url) do
+    Process.flag(:trap_exit, true)
     {:ok, socket} = ChromeRemoteInterface.Websocket.start_link(url)
 
     state = %__MODULE__{
@@ -282,8 +283,17 @@ defmodule ChromeRemoteInterface.PageSession do
     |> Enum.each(&send(&1, event))
   end
 
-  def terminate(_reason, state) do
-    Process.exit(state.socket, :kill)
+  # handle the trapped exit call
+  def handle_info({:EXIT, from, reason}, %{socket: socket} = state) do
+    cleanup(reason, state)
+    # see GenServer docs for other return types
+    {:stop, reason, state}
+  end
+
+  def terminate(reason, state) do
+    cleanup(reason, state)
     :stop
   end
+
+  def cleanup(reason, state), do: Process.exit(state.socket, reason)
 end

lib/websocket.ex

@@ -11,4 +11,13 @@ defmodule ChromeRemoteInterface.Websocket do
     send(state, {:message, frame_data})
     {:ok, state}
   end
+
+  def handle_disconnect(_status, state) do
+    Process.exit(state, :remote_closed)
+    {:ok, state}
+  end
+
+  def terminate({:remote, :closed}, _state) do
+    :stop
+  end
 end