Noissue - Dead code removal

src/plugin/PluginClass.cpp

Index: src/plugin/PluginClass.cpp
===================================================================
--- a/src/plugin/PluginClass.cpp
+++ b/src/plugin/PluginClass.cpp
@@ -42,7 +42,6 @@
 CComAutoCriticalSection CPluginClass::s_criticalSectionWindow;
 
 CComQIPtr<IWebBrowser2> CPluginClass::s_asyncWebBrowser2;
-std::map<UINT,CString> CPluginClass::s_menuDomains;
 
 /*
  * Without namespace declaration, the identifier "Rectangle" is ambiguous
@@ -1382,13 +1381,6 @@
   DEBUG_GENERAL("SetMenuBar");
 
   std::wstring ctext;
-
-  s_criticalSectionLocal.Lock();
-  {
-    s_menuDomains.clear();
-  }
-  s_criticalSectionLocal.Unlock();
-
   Dictionary* dictionary = Dictionary::GetInstance();
 
   MENUITEMINFOW fmii;
@@ -1631,10 +1623,7 @@
   if (tab)
   {
     CPluginClient* client = CPluginClient::GetInstance();
-    if (!CPluginSettings::GetInstance()->IsPluginEnabled())
-    {
-    }
-    else if (client->IsWhitelistedUrl(std::wstring(url)))
+    if (client->IsWhitelistedUrl(std::wstring(url)))

[Oleksandr, 2014/10/03 08:19:59]

This should still run only if plugin is enabled. I must've miss communicated
what can be removed. Only the empty 'if' block is weird here. Should be 
if (CPluginSettings::GetInstance()->IsPluginEnabled())
{
  if (client->IsWhitelistedUrl(...))
  {
  }
  else
   ...
}

[Eric, 2014/10/07 15:57:49]

Done.

     {
       hIcon = GetIcon(ICON_PLUGIN_DISABLED);
     }