Pin the Module after we register a hook....

ceee/ie/plugin/bho/executor.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // CeeeExecutor implementation
 //
 // We use interfaces named ITabWindowManager and ITabWindow
 // (documented at
 // http://www.geoffchappell.com/viewer.htm?doc=studies/windows/ie/ieframe/interfaces/itabwindowmanager.htm
 // and
@@ skipping 83 matching lines @@
 
   hook_ = ::SetWindowsHookEx(WH_GETMESSAGE, GetMsgProc,
       static_cast<HINSTANCE>(_AtlBaseModule.GetModuleInstance()), thread_id);
   if (hook_ == NULL) {
     LOG(ERROR) << "Couldn't hook into thread: " << thread_id << " " <<
         com::LogWe();
     current_thread_id_ = 0;
     return E_FAIL;
   }
 
+  // We have seen a case where a hook may call us in the Broker process
+  // but the DLL is not loaded anymore... This is weird... But to make sure
+  // this doesn't cause a crash when it happens, we pin ourselves here to make
+  // sure we won't be called when unloaded.
+  PinModule();
+
   // We unfortunately can't Send a synchronous message here.
   // If we do, any calls back to the broker fail with the following error:
   // "An outgoing call cannot be made since the application is dispatching an
   //  input synchronous call."
   BOOL success = ::PostThreadMessage(thread_id, kCreateWindowExecutorMessage,
                                      0, static_cast<LPARAM>(window));
   if (success)
     return S_OK;
   else
     return HRESULT_FROM_WIN32(::GetLastError());
@@ skipping 905 matching lines @@
 
 STDMETHODIMP CeeeExecutor::OnTopFrameBeforeNavigate(BSTR url) {
   DCHECK(infobar_manager_ != NULL) << "infobar_manager_ is not initialized";
   if (infobar_manager_ == NULL)
     return E_FAIL;
 
   // According to the specification, tab navigation closes the infobar.
   infobar_manager_->HideAll();
   return S_OK;
 }