Index: chrome/browser/prerender/prerender_final_status.cc |
diff --git a/chrome/browser/prerender/prerender_final_status.cc b/chrome/browser/prerender/prerender_final_status.cc |
index 8d71e41a6768a047dce88cae8ebdfbec53eda4fd..96f408ebf2a748d7aad16f743ee4539b95bcd439 100644 |
--- a/chrome/browser/prerender/prerender_final_status.cc |
+++ b/chrome/browser/prerender/prerender_final_status.cc |
@@ -51,7 +51,7 @@ COMPILE_ASSERT(arraysize(kFinalStatusNames) == FINAL_STATUS_MAX + 1, |
} |
-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 |
@@ -62,6 +62,23 @@ void RecordFinalStatus(FinalStatus final_status) { |
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_Li
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) { |