Skip to content

Commit

Permalink
Made Aardvark exit when one of its processes crashes
Browse files Browse the repository at this point in the history
Somewhere down the road it might make sense for some of these processes to restart, but for now it's better to keep people from needing to go to task manager.
  • Loading branch information
JoeLudwig committed Aug 23, 2020
1 parent 00650ec commit 4f6b6f4
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 0 deletions.
11 changes: 11 additions & 0 deletions src/avrenderer/av_cef_handler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -436,3 +436,14 @@ void CAardvarkCefHandler::windowUpdate( CefRefPtr<CefListValue> windowInfo )
m_browser->GetFocusedFrame()->SendProcessMessage( PID_RENDERER, msg );
}


// -----------------------------------------------------------------------------------------------------
// Purpose: Called when a window info changes for a subscribed window
// -----------------------------------------------------------------------------------------------------
void CAardvarkCefHandler::OnRenderProcessTerminated( CefRefPtr<CefBrowser> browser,
TerminationStatus status )
{
LOG( ERROR ) << "Exiting because process for " << this->m_params.uri << " crashed.";
m_application->quitRequested();
}

2 changes: 2 additions & 0 deletions src/avrenderer/av_cef_handler.h
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,8 @@ class CAardvarkCefHandler : public CefClient,
{
return m_resourceRequestHandler;
}
virtual void OnRenderProcessTerminated( CefRefPtr<CefBrowser> browser,
TerminationStatus status ) override;

bool IsClosing() const { return m_isClosing; }

Expand Down

0 comments on commit 4f6b6f4

Please sign in to comment.