Add new UMA stats to get a handle on Downloads UI Usage

chrome/browser/web_resource/promo_resource_service.cc

Index: chrome/browser/web_resource/promo_resource_service.cc
diff --git a/chrome/browser/web_resource/promo_resource_service.cc b/chrome/browser/web_resource/promo_resource_service.cc
index 7b62757bbe66abedf4c8fceb3165a2f622d7826e..5ee1cff4d476e6442570d53d06e694b1f167be54 100644
--- a/chrome/browser/web_resource/promo_resource_service.cc
+++ b/chrome/browser/web_resource/promo_resource_service.cc
@@ -296,7 +296,8 @@ void PromoResourceService::SetNotificationParams(DictionaryValue* a_dic) {
       max_group < 0 ||
       max_group >= kNTPPromoGroupSize) {
     // If values are not valid, do not show promo.
-    NOTREACHED() << "Invalid server data, question=" << question <<
+    LOG(ERROR) << "Invalid server data, question=" << question <<
+        ", err=" << err <<

[Randy Smith (Not in Mondays), 2011/09/30 15:57:21]

What's the context for this change?

[benjhayden, 2011/10/03 20:54:39]

I reached the NOTREACHED, but this file changed dramatically in today's merge
and I'm not reaching the NOTREACHED anymore.

         ", build=" << promo_build <<
         ", time_slice=" << time_slice <<
         ", max_group=" << max_group;