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..c598106d10eb2059381ab346c76b54eb6c0eab50 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,21 @@ void RecordFinalStatus(FinalStatus final_status) { |
| UMA_HISTOGRAM_ENUMERATION("Prerender.FinalStatus", |
| final_status, |
| FINAL_STATUS_MAX); |
| + switch (origin) { |
| + case ORIGIN_LINK_REL_PRERENDER: |
| + UMA_HISTOGRAM_ENUMERATION("Prerender.FinalStatus_LinkRelPrerender", |
| + final_status, |
| + FINAL_STATUS_MAX); |
| + break; |
| + case ORIGIN_OMNIBOX: |
| + UMA_HISTOGRAM_ENUMERATION("Prerender.FinalStatus_Omnibox", |
|
tburkard
2011/06/23 16:24:07
Why not have a different top level name, such as
P
dominich
2011/06/23 16:32:47
After I get this change in, I'm going to look at h
|
| + final_status, |
| + FINAL_STATUS_MAX); |
| + break; |
| + default: |
| + NOTREACHED(); |
| + break; |
| + }; |
| } |
| const char* NameFromFinalStatus(FinalStatus final_status) { |