Issue 1201 - Enabling/disabling ABP everywhere applies only in the next opened tab

src/plugin/Plugin.cpp

Index: src/plugin/Plugin.cpp
===================================================================
--- a/src/plugin/Plugin.cpp
+++ b/src/plugin/Plugin.cpp
@@ -31,6 +31,7 @@
 #include "PluginMimeFilterClient.h"
 #include "Msiquery.h"
 #include "PluginFilter.h"
+#include "PluginClientFactory.h"
 
 #include "../shared/Dictionary.h"
 
@@ -85,12 +86,8 @@
       delete CPluginSettings::s_instance;
     }
 
-    if (CPluginClass::s_mimeFilter != NULL)
-    {
-      CPluginClass::s_mimeFilter->Unregister();
-      CPluginClass::s_mimeFilter = NULL;
-    }
-
+    CPluginClientFactory::ReleaseMimeFilterClientInstance();

[Eric, 2015/03/27 13:35:50]

I've seen a crash recently during shutdown when 's_mimeFilter' was being
referenced after COM had already destroyed the instance. I'm guessing that any
reference to COM objects here is defective. That includes a reference to
's_mimeFilterInstance' through 'ReleaseMimeFilterClientInstance'. The problem is
that even though 's_mimeFilterInstance' is not 'nullptr', its value has been
invalidated by COM.

We need to manage the life cycle of the mime-filter correctly, but I don't think
this is the way to do it.

+ 
     _CrtDumpMemoryLeaks();
   }
   return (_Module.GetLockCount() == 0) ? S_OK : S_FALSE;