Add a lock to the cookie accountant to prevent race conditions in patching...

ceee/ie/plugin/bho/cookie_accountant.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.
 //
 // CookieAccountant implementation.
 #include "ceee/ie/plugin/bho/cookie_accountant.h"
 
 #include <atlbase.h>
 #include <wininet.h>
 
@@ skipping 204 matching lines @@
       continue;  // Skip non-cookie headers.
     std::string cookie_data = header_line.substr(
         name_pos + std::string(kSetCookieHeaderName).size());
     // TODO(cindylau@chromium.org): Get the URL for the HTTP request from
     // IHttpNegotiate::BeginningTransaction.
     RecordCookie(std::string(), cookie_data, current_time);
   }
 }
 
 void CookieAccountant::PatchWininetFunctions() {
+  {
+    AutoLock lock(lock_);
+    if (patching_wininet_functions_)
+      return;
+    patching_wininet_functions_ = true;
+  }
   if (!internet_set_cookie_ex_a_patch_.is_patched()) {
     DWORD error = internet_set_cookie_ex_a_patch_.Patch(
         kMsHtmlModuleName, kWinInetModuleName,
         kInternetSetCookieExAFunctionName, InternetSetCookieExAPatch);
     DCHECK(error == NO_ERROR || !internet_set_cookie_ex_a_patch_.is_patched());
   }
   if (!internet_set_cookie_ex_w_patch_.is_patched()) {
     DWORD error = internet_set_cookie_ex_w_patch_.Patch(
         kMsHtmlModuleName, kWinInetModuleName,
         kInternetSetCookieExWFunctionName, InternetSetCookieExWPatch);
     DCHECK(error == NO_ERROR || !internet_set_cookie_ex_w_patch_.is_patched());
   }
   DCHECK(internet_set_cookie_ex_a_patch_.is_patched() ||
          internet_set_cookie_ex_w_patch_.is_patched());
+  {
+    AutoLock lock(lock_);
+    patching_wininet_functions_ = false;
+  }
 }