Make a separate histogram for MatchComplete Final Status'es and

chrome/browser/prerender/prerender_browsertest.cc

Index: chrome/browser/prerender/prerender_browsertest.cc
===================================================================
--- chrome/browser/prerender/prerender_browsertest.cc	(revision 118385)
+++ chrome/browser/prerender/prerender_browsertest.cc	(working copy)
@@ -144,10 +144,14 @@
   }
 
   virtual ~TestPrerenderContents() {
-    EXPECT_EQ(expected_final_status_, final_status()) <<
-              " when testing URL " << prerender_url().path() <<
-              " (Expected: " << NameFromFinalStatus(expected_final_status_) <<
-              ", Actual: " << NameFromFinalStatus(final_status()) << ")";
+    if (expected_final_status_ == FINAL_STATUS_MATCH_COMPLETE_DUMMY) {
+      EXPECT_EQ(mc_status(), MC_REPLACEMENT);

[dominich, 2012/01/20 22:23:37]

Can you add some output like the below case that has the URL in it? It really
helps when figuring out why a test fails.

[tburkard, 2012/01/20 23:23:00]

I agree that in the other case, it's helpful, because you see what was expected
and what it is.  Here, the fact that it's not marked as MC_REPLACEMENT is cause
for error.  It's a binary check, is it marked or not.  So re-printing it
wouldn't help much, I think.

On 2012/01/20 22:23:37, dominich wrote:

+    } else {
+      EXPECT_EQ(expected_final_status_, final_status()) <<
+          " when testing URL " << prerender_url().path() <<
+          " (Expected: " << NameFromFinalStatus(expected_final_status_) <<
+          ", Actual: " << NameFromFinalStatus(final_status()) << ")";
+    }
     // Prerendering RenderViewHosts should be hidden before the first
     // navigation, so this should be happen for every PrerenderContents for
     // which a RenderViewHost is created, regardless of whether or not it's