Most visited iframe now postMessage to signal the iframing page that the link has been displayed

chrome/browser/ui/webui/ntp/ntp_user_data_logger.cc

 // Copyright 2013 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 "chrome/browser/ui/webui/ntp/ntp_user_data_logger.h"
 
 #include "base/metrics/histogram.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/search/most_visited_iframe_source.h"
 #include "chrome/browser/search/search.h"
 #include "chrome/common/search_urls.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/web_contents.h"
 
+// Macro to log UMA statistics related to the 8 tiles shown on the NTP.
+#define UMA_HISTOGRAM_NTP_TILES(name, sample) UMA_HISTOGRAM_CUSTOM_COUNTS( \

[Alexei Svitkine (slow), 2014/01/16 19:50:30]

Nit: I'd wrap it like this:

#define UMA_HISTOGRAM_NTP_TILES(name, sample) \
    UMA_HISTOGRAM_CUSTOM_COUNTS(name, sample, 0, 8, 9)

[beaudoin, 2014/01/17 03:51:46]

I wrapped it exactly like in histogram.h, but I happen to agree your version
looks nicer. :)

Done.

+    name, sample, 0, 8, 9)
+
 namespace {
 
 // Used to track if suggestions were issued by the client or the server.
 enum SuggestionsType {
   CLIENT_SIDE = 0,
   SERVER_SIDE = 1,
   SUGGESTIONS_TYPE_COUNT = 2
 };
 
 // Format string to generate the name for the histogram keeping track of
@@ skipping 19 matching lines @@
   // originate from it. We use the NavigationController's URL since it might
   // differ from the WebContents URL which is usually chrome://newtab/.
   const content::NavigationEntry* entry =
       content->GetController().GetVisibleEntry();
   if (entry)
     logger->ntp_url_ = entry->GetURL();
 
   return logger;
 }
 
-void NTPUserDataLogger::EmitThumbnailErrorRate() {
-  DCHECK_LE(number_of_thumbnail_errors_, number_of_thumbnail_attempts_);
-  if (number_of_thumbnail_attempts_ != 0) {
-    UMA_HISTOGRAM_PERCENTAGE(
-      "NewTabPage.ThumbnailErrorRate",
-      GetPercentError(number_of_thumbnail_errors_,
-                      number_of_thumbnail_attempts_));
-  }
-  DCHECK_LE(number_of_fallback_thumbnails_used_,
-            number_of_fallback_thumbnails_requested_);
-  if (number_of_fallback_thumbnails_requested_ != 0) {
-    UMA_HISTOGRAM_PERCENTAGE(
-        "NewTabPage.ThumbnailFallbackRate",
-        GetPercentError(number_of_fallback_thumbnails_used_,
-                        number_of_fallback_thumbnails_requested_));
-  }
-  number_of_thumbnail_attempts_ = 0;
-  number_of_thumbnail_errors_ = 0;
-  number_of_fallback_thumbnails_requested_ = 0;
-  number_of_fallback_thumbnails_used_ = 0;
-}
-
 void NTPUserDataLogger::EmitNtpStatistics() {
   UMA_HISTOGRAM_COUNTS("NewTabPage.NumberOfMouseOvers", number_of_mouseovers_);
   number_of_mouseovers_ = 0;
-  UMA_HISTOGRAM_COUNTS("NewTabPage.NumberOfExternalTiles",
-                       number_of_external_tiles_);
-  number_of_external_tiles_ = 0;
-  UMA_HISTOGRAM_ENUMERATION(
-      "NewTabPage.SuggestionsType",
-      server_side_suggestions_ ? SERVER_SIDE : CLIENT_SIDE,
-      SUGGESTIONS_TYPE_COUNT);
-  server_side_suggestions_ = false;
+
+  // Only log the following statistics if at least one tile is recorded. This
+  // check is required because the statistics are emitted whenever the user
+  // changes tab away from the NTP. However, if the user comes back to that NTP
+  // later the statistics are not regenerated (i.e. they are all 0). If we log
+  // them again we get a strong bias.
+  if (number_of_tiles_ > 0) {
+    UMA_HISTOGRAM_ENUMERATION(
+        "NewTabPage.SuggestionsType",
+        server_side_suggestions_ ? SERVER_SIDE : CLIENT_SIDE,
+        SUGGESTIONS_TYPE_COUNT);
+    server_side_suggestions_ = false;
+    UMA_HISTOGRAM_NTP_TILES("NewTabPage.NumberOfTiles", number_of_tiles_);
+    number_of_tiles_ = 0;
+    UMA_HISTOGRAM_NTP_TILES("NewTabPage.NumberOfThumbnailTiles",
+                            number_of_thumbnail_tiles_);
+    number_of_thumbnail_tiles_ = 0;
+    UMA_HISTOGRAM_NTP_TILES("NewTabPage.NumberOfGrayTiles",
+                            number_of_gray_tiles_);
+    number_of_gray_tiles_ = 0;
+    UMA_HISTOGRAM_NTP_TILES("NewTabPage.NumberOfExternalTiles",
+                            number_of_external_tiles_);
+    number_of_external_tiles_ = 0;
+    UMA_HISTOGRAM_NTP_TILES("NewTabPage.NumberOfThumbnailErrors",
+                            number_of_thumbnail_errors_);
+    number_of_thumbnail_errors_ = 0;
+    UMA_HISTOGRAM_NTP_TILES("NewTabPage.NumberOfGrayTileFallbacks",
+                            number_of_gray_tile_fallbacks_);
+    number_of_gray_tile_fallbacks_ = 0;
+    UMA_HISTOGRAM_NTP_TILES("NewTabPage.NumberOfExternalTileFallbacks",
+                            number_of_external_tile_fallbacks_);
+    number_of_external_tile_fallbacks_ = 0;
+  }
 }
 
 void NTPUserDataLogger::LogEvent(NTPLoggingEventType event) {
   switch (event) {
-    case NTP_MOUSEOVER:
-      number_of_mouseovers_++;
-      break;
-    case NTP_THUMBNAIL_ATTEMPT:
-      number_of_thumbnail_attempts_++;
-      break;
-    case NTP_THUMBNAIL_ERROR:
-      number_of_thumbnail_errors_++;
-      break;
-    case NTP_FALLBACK_THUMBNAIL_REQUESTED:
-      number_of_fallback_thumbnails_requested_++;
-      break;
-    case NTP_FALLBACK_THUMBNAIL_USED:
-      number_of_fallback_thumbnails_used_++;
-      break;
     case NTP_SERVER_SIDE_SUGGESTION:
       server_side_suggestions_ = true;
       break;
     case NTP_CLIENT_SIDE_SUGGESTION:
       // We should never get a mix of server and client side suggestions,
       // otherwise there could be a race condition depending on the order in
       // which the iframes call this method.
       DCHECK(!server_side_suggestions_);
-    break;
+      break;
+    case NTP_TILE:
+      number_of_tiles_++;
+      break;
+    case NTP_THUMBNAIL_TILE:
+      number_of_thumbnail_tiles_++;
+      break;
+    case NTP_GRAY_TILE:
+      number_of_gray_tiles_++;
+      break;
     case NTP_EXTERNAL_TILE:
       number_of_external_tiles_++;
       break;
+    case NTP_THUMBNAIL_ERROR:
+      number_of_thumbnail_errors_++;
+      break;
+    case NTP_GRAY_TILE_FALLBACK:
+      number_of_gray_tile_fallbacks_++;
+      break;
+    case NTP_EXTERNAL_TILE_FALLBACK:
+      number_of_external_tile_fallbacks_++;
+      break;
+    case NTP_MOUSEOVER:
+      number_of_mouseovers_++;
+      break;
     default:
       NOTREACHED();
   }
 }
 
 void NTPUserDataLogger::LogImpression(int position,
                                       const base::string16& provider) {
   // Cannot rely on UMA histograms macro because the name of the histogram is
   // generated dynamically.
   base::HistogramBase* counter = base::LinearHistogram::FactoryGet(
       base::StringPrintf(kImpressionHistogramWithProvider,
                          base::UTF16ToUTF8(provider).c_str()),
       1, MostVisitedIframeSource::kNumMostVisited,
       MostVisitedIframeSource::kNumMostVisited + 1,
       base::Histogram::kUmaTargetedHistogramFlag);
   counter->Add(position);
 }
 
 // content::WebContentsObserver override
 void NTPUserDataLogger::NavigationEntryCommitted(
     const content::LoadCommittedDetails& load_details) {
   if (!load_details.previous_url.is_valid())
     return;
 
   if (search::MatchesOriginAndPath(ntp_url_, load_details.previous_url)) {
     EmitNtpStatistics();
-    // Only log thumbnail error rates for Instant NTP pages, as we do not have
-    // this data for non-Instant NTPs.
-    if (ntp_url_ != GURL(chrome::kChromeUINewTabURL))
-      EmitThumbnailErrorRate();
   }
 }
 
 NTPUserDataLogger::NTPUserDataLogger(content::WebContents* contents)
     : content::WebContentsObserver(contents),
-      number_of_mouseovers_(0),
-      number_of_thumbnail_attempts_(0),
+      server_side_suggestions_(false),
+      number_of_tiles_(0),
+      number_of_thumbnail_tiles_(0),
+      number_of_gray_tiles_(0),
+      number_of_external_tiles_(0),
       number_of_thumbnail_errors_(0),
-      number_of_fallback_thumbnails_requested_(0),
-      number_of_fallback_thumbnails_used_(0),
-      number_of_external_tiles_(0),
-      server_side_suggestions_(false) {
+      number_of_gray_tile_fallbacks_(0),
+      number_of_external_tile_fallbacks_(0),
+      number_of_mouseovers_(0) {
 }
-
-size_t NTPUserDataLogger::GetPercentError(size_t errors, size_t events) const {
-  return (100 * errors) / events;
-}