Add a histogram for tracking web store promo

chrome/browser/dom_ui/app_launcher_handler.h

 // 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.
 
 #ifndef CHROME_BROWSER_DOM_UI_APP_LAUNCHER_HANDLER_H_
 #define CHROME_BROWSER_DOM_UI_APP_LAUNCHER_HANDLER_H_
 #pragma once
 
 #include "base/scoped_ptr.h"
 #include "chrome/browser/dom_ui/dom_ui.h"
@@ skipping 46 matching lines @@
 
   // Callback for the "setLaunchType" message.
   void HandleSetLaunchType(const ListValue* args);
 
   // Callback for the "uninstallApp" message.
   void HandleUninstallApp(const ListValue* args);
 
   // Callback for the "hideAppPromo" message.
   void HandleHideAppsPromo(const ListValue* args);
 
+  // Callback for pings related to launching apps on the NTP.
+  static bool HandlePing(const std::string& path);

[Aaron Boodman, 2010/11/12 19:01:02]

We seem to usually put the statics methods together at the top of the method
list in Chromium code (just after the constructor and destructor). Can you fix
CreateAppInfo too?

+
  private:
   // ExtensionInstallUI::Delegate implementation, used for receiving
   // notification about uninstall confirmation dialog selections.
   virtual void InstallUIProceed();
   virtual void InstallUIAbort();
 
   // Returns the ExtensionInstallUI object for this class, creating it if
   // needed.
   ExtensionInstallUI* GetExtensionInstallUI();
 
   // Starts the animation of the app icon.
   void AnimateAppIcon(const Extension* extension, const gfx::Rect& rect);
 
+  // Records a web store launch in the appropriate histograms. |promo_active|
+  // specifies if the web store promotion was active.
+  static void RecordWebStoreLaunch(bool promo_active);

[Aaron Boodman, 2010/11/12 19:01:02]

Same here.

+
+  // Records an app launch in the appropriate histograms. |promo_active|
+  // specifies if the web store promotion was active.
+  static void RecordAppLaunch(bool promo_active);
+
   // The apps are represented in the extensions model.
   scoped_refptr<ExtensionsService> extensions_service_;
 
   // We monitor changes to the extension system so that we can reload the apps
   // when necessary.
   NotificationRegistrar registrar_;
 
   // Monitor extension preference changes so that the DOM UI can be notified.
   PrefChangeRegistrar pref_change_registrar_;
 
   // Used to show confirmation UI for uninstalling/enabling extensions in
   // incognito mode.
   scoped_ptr<ExtensionInstallUI> install_ui_;
 
   // The id of the extension we are prompting the user about.
   std::string extension_id_prompting_;
 
+  // Whether the promo is currently being shown.
+  bool promo_active_;
+
   DISALLOW_COPY_AND_ASSIGN(AppLauncherHandler);
 };
 
 #endif  // CHROME_BROWSER_DOM_UI_APP_LAUNCHER_HANDLER_H_