Appear fast for IE

src/plugin/PluginTabBase.cpp

Index: src/plugin/PluginTabBase.cpp
===================================================================
--- a/src/plugin/PluginTabBase.cpp
+++ b/src/plugin/PluginTabBase.cpp
@@ -29,6 +29,7 @@
 CPluginTabBase::CPluginTabBase(CPluginClass* plugin) : m_plugin(plugin), m_isActivated(false)
 {
   m_filter = std::auto_ptr<CPluginFilter>(new CPluginFilter());
+  m_filter->hideFiltersLoadedEvent = CreateEvent(NULL, TRUE, FALSE, NULL);

[Felix Dahlke, 2013/08/07 09:34:49]

Since this is C++, we should use true and false instead of the defines.

 
   CPluginClient* client = CPluginClient::GetInstance();
   if (client->GetIEVersion() < 10)
@@ -82,6 +83,13 @@
   m_isActivated = true;
 }
 
+DWORD WINAPI FilterLoader(void* thisPtrVoid)
+{
+  CPluginTabBase* thisPtr = (CPluginTabBase*)thisPtrVoid;
+  thisPtr->m_filter->LoadHideFilters(CPluginClient::GetInstance()->GetElementHidingSelectors(thisPtr->GetDocumentDomain().GetString()));
+  SetEvent(thisPtr->m_filter->hideFiltersLoadedEvent);
+  return 0;
+}
 
 void CPluginTabBase::OnNavigate(const CString& url)
 {
@@ -93,7 +101,8 @@
 #endif
 
   std::wstring domainString = GetDocumentDomain();
-  m_filter->LoadHideFilters(CPluginClient::GetInstance()->GetElementHidingSelectors(domainString));
+  ResetEvent(m_filter->hideFiltersLoadedEvent);
+  CreateThread(NULL, NULL, &FilterLoader, this, NULL, NULL);

[Felix Dahlke, 2013/08/07 13:23:03]

How I see it, this is the relevant change, right? The one that makes IE think
we're fast although we're not. I think it's worth adding a comment here on why
filters are being loaded in a separate thread.

[Oleksandr, 2013/08/07 16:07:40]

This is not the only relevant change. This is just done for IE not to show us as
a bottleneck of the navigation slowdown (the second value in the Menage Addons).
The first value (the load time) was influenced by us initializing the settings
in the main thread (SetSite essentially). So more like the relevant change is
the "always registering the mime filter, and only unregistering it if it was
disabled". That way we can initialize settings singleton elsewhere (since we
don't need to call the GetPref("enabled"..) in the main thread anymore.
On 2013/08/07 13:23:03, Felix H. Dahlke wrote:

 
 #ifdef SUPPORT_DOM_TRAVERSER
   m_traverser->ClearCache();