Chromium Code Reviews| 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) { |