Issues #1173 - Add exception handler to entry point FilterLoader

src/plugin/PluginTabBase.cpp

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 76 matching lines @@
   // Entry Point
   void FilterLoader(CPluginTabBase* tabBase)
   {
     try
     {
       tabBase->m_filter->LoadHideFilters(CPluginClient::GetInstance()->GetElementHidingSelectors(tabBase->GetDocumentDomain()));
       SetEvent(tabBase->m_filter->hideFiltersLoadedEvent);
     }
     catch (...)
     {
-      // There's presently a race condition between loading filters and terminating the appplication; see #1654.
-      // In this particular case, it's OK to simply suppress the exception.

[sergei, 2015/02/13 14:41:40]

This comment is not relevant as well.

[Eric, 2015/02/13 15:13:52]

This wording is indeed not right. I've changed it to something generic.

+      // As a thread-main function, we truncate any C++ exception.
     }
   }
 }
 
 void CPluginTabBase::OnNavigate(const std::wstring& url)
 {
   SetDocumentUrl(url);
   ClearFrameCache(GetDocumentDomain());
   std::wstring domainString = GetDocumentDomain();
   ResetEvent(m_filter->hideFiltersLoadedEvent);
@@ skipping 266 matching lines @@
           CPluginClient::LogPluginError(pluginError.GetErrorCode(), pluginError.GetErrorId(), pluginError.GetErrorSubid(), pluginError.GetErrorDescription(), true, pluginError.GetProcessId(), pluginError.GetThreadId());
         }
 
         // Non-hanging sleep
         Sleep(50);
       }
 
       tabLoopIteration++;
   }
 }