Make purify happy by fixing memory leak that shows up during shutdown.

chrome/browser/metrics/metrics_service.cc

 // Copyright (c) 2006-2008 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.
 
 
 
 //------------------------------------------------------------------------------
 // Description of the life cycle of a instance of MetricsService.
 //
 //  OVERVIEW
@@ skipping 331 matching lines @@
       logged_samples_(),
       interlog_duration_(TimeDelta::FromSeconds(kInitialInterlogDuration)),
       log_event_limit_(kInitialEventLimit),
       timer_pending_(false) {
   DCHECK(IsSingleThreaded());
   InitializeMetricsState();
 }
 
 MetricsService::~MetricsService() {
   SetRecording(false);
+  if (pending_log_) {

[darin (slow to review), 2009/01/28 09:18:27]

nit: there's no need to null-check a pointer before calling delete on it. 
"delete NULL" is a no-op.

+    delete pending_log_;
+    pending_log_ = NULL;
+  }
+  if (current_log_) {
+    delete current_log_;
+    current_log_ = NULL;
+  }
 }
 
 void MetricsService::SetUserPermitsUpload(bool enabled) {
   HandleIdleSinceLastTransmission(false);
   user_permits_upload_ = enabled;
 }
 
 void MetricsService::Start() {
   SetRecording(true);
   SetReporting(true);
@@ skipping 1391 matching lines @@
       L"." + key;
   prof_prefs->SetInteger(pref_key.c_str(), value);
 }
 
 static bool IsSingleThreaded() {
   static int thread_id = 0;
   if (!thread_id)
     thread_id = GetCurrentThreadId();
   return GetCurrentThreadId() == thread_id;
 }