Added ActivityLog tests and associated bugfixes/extra logging.

chrome/browser/extensions/activity_log.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_EXTENSIONS_ACTIVITY_LOG_H_
 #define CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 182 matching lines @@
   // A map of extensions to activity observers for that extension.
   ObserverMap observers_;
 
   // The database wrapper that does the actual database I/O.
   scoped_refptr<extensions::ActivityDatabase> db_;
 
   // Whether to log activity to stdout or the UI. These are set by switches.
   bool log_activity_to_stdout_;
   bool log_activity_to_ui_;
 
-  // log_arguments controls whether to log API call arguments. By default, we
+  // testing_mode_ controls whether to log API call arguments. By default, we
   // don't log most arguments to avoid saving too much data. In testing mode,
   // argument collection is enabled. We also whitelist some arguments for
   // collection regardless of whether this bool is true.
-  bool log_arguments_;
+  bool testing_mode_;
   base::hash_set<std::string> arg_whitelist_api_;
 
   DISALLOW_COPY_AND_ASSIGN(ActivityLog);
 };
 
 // Each profile has different extensions, so we keep a different database for
 // each profile.
 class ActivityLogFactory : public ProfileKeyedServiceFactory {
  public:
   static ActivityLog* GetForProfile(Profile* profile) {
@@ skipping 15 matching lines @@
 
   virtual bool ServiceRedirectedInIncognito() const OVERRIDE;
 
   DISALLOW_COPY_AND_ASSIGN(ActivityLogFactory);
 };
 
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_H_