Start and stop crash reporting outside of the loader lock.

chrome_frame/vtable_patch_manager.cc

 // Copyright (c) 2011 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.
 
 #include "chrome_frame/vtable_patch_manager.h"
 
 #include <atlcomcli.h>
 
 #include <algorithm>
 
 #include "base/atomicops.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/synchronization/lock.h"
 #include "chrome_frame/function_stub.h"
-#include "chrome_frame/utils.h"
+#include "chrome_frame/pin_module.h"
 
 namespace vtable_patch {
 
 // The number of times we retry a patch/unpatch operation in case of
 // VM races with other 3rd party software trying to patch the same thing.
 const int kMaxRetries = 3;
 
 // We hold a lock over all patching operations to make sure that we don't
 // e.g. race on VM operations to the same patches, or to physical pages
 // shared across different VTABLEs.
@@ skipping 110 matching lines @@
 
     if (!succeeded) {
       FunctionStub::Destroy(stub);
       stub = NULL;
 
       DLOG(ERROR) << "Failed to patch VTable.";
       return E_FAIL;
     } else {
       // Success, save the stub we created.
       it->stub_ = stub;
-      PinModule();
+      chrome_frame::PinModule();
     }
   }
 
   return S_OK;
 }
 
 HRESULT UnpatchInterfaceMethods(MethodPatchInfo* patches) {
   base::AutoLock lock(patch_lock_);
 
   for (MethodPatchInfo* it = patches; it->index_ != -1; ++it) {
@@ skipping 77 matching lines @@
   }
   patch_list_.clear();
   patch_list_lock_.Release();
 
   return true;
 }
 
 #endif  // disabled DynamicPatchManager
 
 }  // namespace vtable_patch