Cleanup

src/plugin/PluginClass.cpp

 #include "PluginStdAfx.h"
 
 #include "PluginClass.h"
 #include "PluginSettings.h"
 #include "PluginSystem.h"
 #ifdef SUPPORT_FILTER
 #include "PluginFilter.h"
 #endif
 #include "PluginMimeFilterClient.h"
 #include "PluginClient.h"
@@ skipping 51 matching lines @@
   m_hTabWnd = NULL;
   m_hStatusBarWnd = NULL;
   m_hPaneWnd = NULL;
   m_nPaneWidth = 0;
   m_pWndProcStatus = NULL;
   m_hTheme = NULL;
   m_isInitializedOk = false;
 
 
   m_tab = new CPluginTab(this);
-
-  // Load settings (fetch the available subscriptions, if still not fetched)
-  CPluginSettings* settings = CPluginSettings::GetInstance();
 
   CPluginSystem* system = CPluginSystem::GetInstance();
 
   std::wstring locale((LPCWSTR)system->GetBrowserLanguage());
   Dictionary::Create(locale);
 }
 
 CPluginClass::~CPluginClass()
 {
   delete m_tab;
@@ skipping 336 matching lines @@
     }
   }
   return trueth == 1;
 }
 
 void CPluginClass::ShowStatusBar()
 {
   VARIANT_BOOL isVisible;
 
   CPluginSettings* settings = CPluginSettings::GetInstance();
-
-  //TODO: Remove the message box
-  MessageBox(NULL, L"", L"", MB_OK);

[Wladimir Palant, 2013/07/11 12:53:10]

Please address that TODO comment ;)

 
   CComQIPtr<IWebBrowser2> browser = GetAsyncBrowser();
   if (browser)
   {
     HRESULT hr = S_OK;
     hr = browser->get_StatusBar(&isVisible);
     if (SUCCEEDED(hr))
     {
       if (!isVisible)
       {
@@ skipping 1522 matching lines @@
         }
       }
     }
 
     hTabWnd = ::GetWindow(hTabWnd, GW_HWNDNEXT);
   }
 
   return hTabWnd;
 
 }