Issue 1490 - Enable pre-configurable properties in Internet Explorer

src/plugin/PluginClass.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 897 matching lines @@
     
   HDC hdc = GetWindowDC(m_hStatusBarWnd);
   SendMessage(m_hStatusBarWnd, WM_PAINT, (WPARAM)hdc, 0);
   ReleaseDC(m_hStatusBarWnd, hdc);
   
   return true;
 }
 
 void CPluginClass::FirstRunThread()
 {
+  // Just return if the First Run Page should be suppressed
+  if (CPluginClient::GetInstance()->GetPref(L"suppress_first_run_page", false))

[Eric, 2015/06/19 14:51:41]

I dislike the way that default values for preferences are spread throughout the
code rather than localized in 'libadblockplus'. It's really a design defect,
because it means that each client of 'libadblockplus' has to get the defaults
consistent with each other.

If that design defect were fixed, there would be no need for the second argument
here.

I understand that this might be considered out of scope. But since we're working
on preferences generally, now would be a good time to correct the flaw.

[Oleksandr, 2015/06/22 08:27:53]

I agree, but yes, I think it's out of scope here.

+    return;
+
   CoInitialize(NULL);
   VARIANT vFlags;
   vFlags.vt = VT_I4;
   vFlags.intVal = navOpenInNewTab;
 
   CComBSTR navigatePath = CComBSTR(FirstRunPageFileUrl().c_str());
   
   HRESULT hr = GetAsyncBrowser()->Navigate(navigatePath, &vFlags, NULL, NULL, NULL);
   if (FAILED(hr))
   {
@@ skipping 800 matching lines @@
           s_criticalSectionLocal.Unlock();
 
         }
       }
     }
 
     hTabWnd = ::GetWindow(hTabWnd, GW_HWNDNEXT);
   }
   return hTabWnd;
 }