Fix crash on exit when multiple windows are open

Controllers would delete their Views even if they failed to pop them from the window stack.

Have I mentioned that I really hate our MVC?

src/gui/console/ConsoleController.cpp

@@ -71,8 +71,10 @@ ConsoleView * ConsoleController::GetView()
 
 ConsoleController::~ConsoleController()
 {
-	consoleView->CloseActiveWindow();
 	delete consoleModel;
-	delete consoleView;
+	if (consoleView->CloseActiveWindow())
+	{
+		delete consoleView;
+	}
 }
 

src/gui/localbrowser/LocalBrowserController.cpp

@@ -141,8 +141,10 @@ void LocalBrowserController::Exit()
 
 LocalBrowserController::~LocalBrowserController()
 {
-	browserView->CloseActiveWindow();
 	delete browserModel;
-	delete browserView;
+	if (browserView->CloseActiveWindow())
+	{
+		delete browserView;
+	}
 }
 

src/gui/login/LoginController.cpp

@@ -39,8 +39,10 @@ void LoginController::Exit()
 
 LoginController::~LoginController()
 {
-	loginView->CloseActiveWindow();
 	delete loginModel;
-	delete loginView;
+	if (loginView->CloseActiveWindow())
+	{
+		delete loginView;
+	}
 }
 

src/gui/options/OptionsController.cpp

@@ -149,8 +149,10 @@ void OptionsController::Exit()
 
 OptionsController::~OptionsController()
 {
-	view->CloseActiveWindow();
 	delete model;
-	delete view;
+	if (view->CloseActiveWindow())
+	{
+		delete view;
+	}
 }
 

src/gui/preview/PreviewController.cpp

@@ -171,8 +171,10 @@ void PreviewController::Exit()
 
 PreviewController::~PreviewController()
 {
-	previewView->CloseActiveWindow();
 	Client::Ref().RemoveListener(this);
 	delete previewModel;
-	delete previewView;
+	if (previewView->CloseActiveWindow())
+	{
+		delete previewView;
+	}
 }

src/gui/render/RenderController.cpp

@@ -58,8 +58,10 @@ void RenderController::Exit()
 
 RenderController::~RenderController()
 {
-	renderView->CloseActiveWindow();
 	delete renderModel;
-	delete renderView;
+	if (renderView->CloseActiveWindow())
+	{
+		delete renderView;
+	}
 }
 

src/gui/search/SearchController.cpp

@@ -85,9 +85,11 @@ void SearchController::Exit()
 SearchController::~SearchController()
 {
 	delete activePreview;
-	searchView->CloseActiveWindow();
 	delete searchModel;
-	delete searchView;
+	if (searchView->CloseActiveWindow())
+	{
+		delete searchView;
+	}
 }
 
 void SearchController::DoSearch(String query, bool now)

src/gui/tags/TagsController.cpp

@@ -46,8 +46,10 @@ void TagsController::Exit()
 
 TagsController::~TagsController()
 {
-	tagsView->CloseActiveWindow();
 	delete tagsModel;
-	delete tagsView;
+	if (tagsView->CloseActiveWindow())
+	{
+		delete tagsView;
+	}
 }