Some cleanup regarding Siggi's comments on http://codereview.chromium.org/173...

chrome_frame/crash_reporting/crash_report.cc

 
 // Copyright (c) 2009 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.
 
 // crash_report.cc : Implementation crash reporting.
 #include "chrome_frame/crash_reporting/crash_report.h"
 
 #include "base/basictypes.h"
 #include "base/lock.h"
 #include "breakpad/src/client/windows/handler/exception_handler.h"
 
 // TODO(joshia): factor out common code with chrome used for crash reporting
 const wchar_t kGoogleUpdatePipeName[] = L"\\\\.\\pipe\\GoogleCrashServices\\";
-static google_breakpad::ExceptionHandler * g_breakpad = NULL;
+
+// This lock protects against concurrent access to g_breakpad.
 static Lock g_breakpad_lock;
+static google_breakpad::ExceptionHandler* g_breakpad = NULL;
 
-// These minidump flag combinations have been tested safe agains the
+// These minidump flag combinations have been tested safe against the
 // DbgHelp.dll version that ships with Windows XP SP2.
 const MINIDUMP_TYPE kSmallDumpType = static_cast<MINIDUMP_TYPE>(
     MiniDumpWithProcessThreadData |  // Get PEB and TEB.
     MiniDumpWithUnloadedModules);  // Get unloaded modules when available.
 
 const MINIDUMP_TYPE kLargerDumpType = static_cast<MINIDUMP_TYPE>(
     MiniDumpWithProcessThreadData |  // Get PEB and TEB.
     MiniDumpWithUnloadedModules |  // Get unloaded modules when available.
     MiniDumpWithIndirectlyReferencedMemory);  // Get memory referenced by stack.
 
@@ skipping 167 matching lines @@
 }
 
 bool WriteMinidumpForException(EXCEPTION_POINTERS* p) {
   AutoLock lock(g_breakpad_lock);
   bool success = false;
   if (g_breakpad) {
     success = g_breakpad->WriteMinidumpForException(p);
   }
   return success;
 }