Add a histogram for tracking web store promo

chrome/browser/dom_ui/new_tab_ui.cc

 // 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.
 
 #include "build/build_config.h"
 
 #include "chrome/browser/dom_ui/new_tab_ui.h"
 
 #include <set>
 
 #include "app/l10n_util.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/i18n/rtl.h"
 #include "base/metrics/histogram.h"
 #include "base/singleton.h"
 #include "base/string_number_conversions.h"
+#include "base/string_split.h"
 #include "base/thread.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_thread.h"
 #include "chrome/browser/dom_ui/app_launcher_handler.h"
 #include "chrome/browser/dom_ui/dom_ui_theme_source.h"
 #include "chrome/browser/dom_ui/foreign_session_handler.h"
 #include "chrome/browser/dom_ui/most_visited_handler.h"
 #include "chrome/browser/dom_ui/new_tab_page_sync_handler.h"
 #include "chrome/browser/dom_ui/ntp_login_handler.h"
 #include "chrome/browser/dom_ui/ntp_resource_cache.h"
 #include "chrome/browser/dom_ui/shown_sections_handler.h"
 #include "chrome/browser/dom_ui/tips_handler.h"
 #include "chrome/browser/metrics/user_metrics.h"
 #include "chrome/browser/themes/browser_theme_provider.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/sessions/session_types.h"
 #include "chrome/browser/sessions/tab_restore_service.h"
 #include "chrome/browser/sessions/tab_restore_service_observer.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/extension.h"
+#include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 
 namespace {
 
 // The number of recent bookmarks we show.
 const int kRecentBookmarks = 9;
@@ skipping 560 matching lines @@
 NewTabUI::NewTabHTMLSource::NewTabHTMLSource(Profile* profile)
     : DataSource(chrome::kChromeUINewTabHost, MessageLoop::current()),
       profile_(profile) {
 }
 
 void NewTabUI::NewTabHTMLSource::StartDataRequest(const std::string& path,
                                                   bool is_off_the_record,
                                                   int request_id) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
-  if (!path.empty() && path[0] != '#') {
+  if (path.find(extension_misc::kLaunchWebStorePingURL) != std::string::npos) {

[Aaron Boodman, 2010/11/12 00:29:20]

Can you isolate all of this in AppLauncherHandler instead? The code sort of
makes more sense there than here. That way the constants can also be in
app_launcher_handler.* instead of in extension_constants.*.

You could have like static bool AppLauncherHandler::HandlePing(path). If it
returns true then it was handled, otherwise, this code goes on to do what it
usually does.

The RecordXYZ() methods would become private statics of AppLauncherHandler.

+    RecordWebStoreLaunch(IsPromoActive(path));
+    return;
+
+  } else if (path.find(extension_misc::kLaunchAppPingURL) !=
+             std::string::npos) {
+    RecordAppLaunch(IsPromoActive(path));
+    return;
+
+  } else if (!path.empty() && path[0] != '#') {
     // A path under new-tab was requested; it's likely a bad relative
     // URL from the new tab page, but in any case it's an error.
     NOTREACHED();
     return;
   }
 
   scoped_refptr<RefCountedBytes> html_bytes(
       profile_->GetNTPResourceCache()->GetNewTabHTML(is_off_the_record));
 
   SendResponse(request_id, html_bytes);
 }
 
 std::string NewTabUI::NewTabHTMLSource::GetMimeType(const std::string&) const {
   return "text/html";
 }
+
+void NewTabUI::NewTabHTMLSource::RecordWebStoreLaunch(bool promo_active) {
+  if (!promo_active) return;
+
+  UMA_HISTOGRAM_ENUMERATION(extension_misc::kAppsPromoHistogram,
+                            extension_misc::PROMO_LAUNCH_WEB_STORE,
+                            extension_misc::PROMO_BUCKET_BOUNDARY);
+}
+
+void NewTabUI::NewTabHTMLSource::RecordAppLaunch(bool promo_active) {
+  // TODO(jstritar): record app launches that occur when the promo is not
+  // active using a different histogram.
+
+  if (!promo_active) return;
+
+  UMA_HISTOGRAM_ENUMERATION(extension_misc::kAppsPromoHistogram,
+                            extension_misc::PROMO_LAUNCH_APP,
+                            extension_misc::PROMO_BUCKET_BOUNDARY);
+}
+
+bool NewTabUI::NewTabHTMLSource::IsPromoActive(const std::string& path) {
+  std::vector<std::string> params;
+  base::SplitString(path, '+', &params);
+
+  DCHECK(params.size() == 2);

[Aaron Boodman, 2010/11/12 00:29:20]

You are going to crash below if size < 2, so might as well CHECK here. It's a
programmer error that this would ever not be 2.

+
+  return params.at(1) == "true";
+}