Detect when chrome is shutting down and don't SendFinacialPing

chrome/browser/rlz/rlz.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_RLZ_RLZ_H_
 #define CHROME_BROWSER_RLZ_RLZ_H_
 
 #include "build/build_config.h"
 
 #if defined(ENABLE_RLZ)
@@ skipping 69 matching lines @@
 
 #if defined(OS_CHROMEOS)
   // Clears all product state. Should be called when turning RLZ off. On other
   // platforms, this is done by product uninstaller.
   static void ClearRlzState();
 #endif
 
   // This method is public for use by the Singleton class.
   static RLZTracker* GetInstance();
 
+  // Called when Chrome is shutting down.
+  static void PostMainMessageLoopRun();

[Roger Tawa OOO till Jul 10th, 2013/04/29 16:56:45]

If we do need this method, better to rename this something like Shutdown(). 
PostMainMessageLoopRun() is descriptive of where this is currently called from
the chrome code today, not really what the function does.

[bcwhite, 2013/04/29 19:21:34]

I was trying to keep the name consistent with every other shutdown method, but
it doesn't matter as I've gone with the existing method.

+
   // Enables zero delay for InitRlzFromProfileDelayed. For testing only.
   static void EnableZeroDelayForTesting();
 
   // The following methods are made protected so that they can be used for
   // testing purposes. Production code should never need to call these.
  protected:
   RLZTracker();
   virtual ~RLZTracker();
 
   // Called by InitRlzFromProfileDelayed with values taken from |profile|.
@@ skipping 95 matching lines @@
   bool send_ping_immediately_;
   bool is_google_default_search_;
   bool is_google_homepage_;
   bool is_google_in_startpages_;
 
   // Unique sequence token so that tasks posted by RLZTracker are executed
   // sequentially in the blocking pool.
   base::SequencedWorkerPool::SequenceToken worker_pool_token_;
 
   // URLRequestContextGetter used by RLZ library.
   net::URLRequestContextGetter* url_request_context_;

[Roger Tawa OOO till Jul 10th, 2013/04/29 16:56:45]

This member is unused.  Lets delete it?

[bcwhite, 2013/04/29 19:21:34]

Done.

 
   // Keeps track if the RLZ tracker has already performed its delayed
   // initialization.
   bool already_ran_;
 
   // Keeps a cache of RLZ access point strings, since they rarely change.
   // The cache must be protected by a lock since it may be accessed from
   // the UI thread for reading and the IO thread for reading and/or writing.
   base::Lock cache_lock_;
   std::map<rlz_lib::AccessPoint, string16> rlz_cache_;
@@ skipping 10 matching lines @@
 
   // Minimum delay before sending financial ping after initialization.
   base::TimeDelta min_init_delay_;
 
   DISALLOW_COPY_AND_ASSIGN(RLZTracker);
 };
 
 #endif  // defined(ENABLE_RLZ)
 
 #endif  // CHROME_BROWSER_RLZ_RLZ_H_