Added prerendering to omnibox.

chrome/browser/prerender/prerender_final_status.cc

 // Copyright (c) 2011 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/prerender/prerender_final_status.h"
 
 #include "base/metrics/histogram.h"
 #include "chrome/browser/prerender/prerender_manager.h"
 
 namespace prerender {
@@ skipping 33 matching lines @@
   "Fragment Mismatch",
   "SSL Client Certificate Requested",
   "Cache or History Cleared",
   "Max",
 };
 COMPILE_ASSERT(arraysize(kFinalStatusNames) == FINAL_STATUS_MAX + 1,
                PrerenderFinalStatus_name_count_mismatch);
 
 }
 
-void RecordFinalStatus(FinalStatus final_status) {
+void RecordFinalStatus(Origin origin, FinalStatus final_status) {
   DCHECK(final_status != FINAL_STATUS_MAX);
   // FINAL_STATUS_CONTROL_GROUP indicates that the PrerenderContents
   // was created only to measure "would-have-been-prerendered" for
   // control group measurements. Don't pollute data with it.
   if (PrerenderManager::IsControlGroup() ||
       final_status == FINAL_STATUS_CONTROL_GROUP)
     return;
   UMA_HISTOGRAM_ENUMERATION("Prerender.FinalStatus",
                             final_status,
                             FINAL_STATUS_MAX);
+  switch (origin) {
+    case ORIGIN_LINK_REL_PRERENDER:
+    case ORIGIN_PENDING_LINK_REL_PRERENDER:
+      UMA_HISTOGRAM_ENUMERATION("Prerender.FinalStatusLinkRelPrerender",

[cbentzel, 2011/06/22 17:27:32]

Should these be done as 

Prerender.FinalStatus_LinkRelPrerender and Prerender.FinalStatus_Omnibox? That's
more consistent with typical naming schemes.

[dominich, 2011/06/22 19:00:51]

Done.

+                                final_status,
+                                FINAL_STATUS_MAX);
+      break;
+    case ORIGIN_OMNIBOX:
+      UMA_HISTOGRAM_ENUMERATION("Prerender.FinalStatusOmnibox",
+                                final_status,
+                                FINAL_STATUS_MAX);
+      break;
+    default:
+      LOG(WARNING) << "Not logging final status for origin " <<

[cbentzel, 2011/06/22 17:27:32]

This should be a NOTREACHED(); 

[dominich, 2011/06/22 19:00:51]

Done.

+                   NameFromOrigin(origin);
+      break;
+  };
 }
 
 const char* NameFromFinalStatus(FinalStatus final_status) {
   DCHECK(static_cast<int>(final_status) >= 0 &&
          static_cast<int>(final_status) <= FINAL_STATUS_MAX);
   return kFinalStatusNames[final_status];
 }
 
 }  // namespace prerender