Remove recovery diagnostics for the archived history database.

chrome/browser/diagnostics/sqlite_diagnostics.cc

Index: chrome/browser/diagnostics/sqlite_diagnostics.cc
diff --git a/chrome/browser/diagnostics/sqlite_diagnostics.cc b/chrome/browser/diagnostics/sqlite_diagnostics.cc
index 6a92c9d0abaaa530289f6882fc0b676a83974a80..2982d2b0dc8bc3c5ae56ee6d6c16c905d84dd552 100644
--- a/chrome/browser/diagnostics/sqlite_diagnostics.cc
+++ b/chrome/browser/diagnostics/sqlite_diagnostics.cc
@@ -200,13 +200,6 @@ class SqliteIntegrityTest : public DiagnosticsTest {
 
 }  // namespace
 
-DiagnosticsTest* MakeSqliteArchivedHistoryDbTest() {
-  return new SqliteIntegrityTest(
-      SqliteIntegrityTest::NO_FLAGS_SET,
-      DIAGNOSTICS_SQLITE_INTEGRITY_ARCHIVED_HISTORY_TEST,
-      base::FilePath(chrome::kArchivedHistoryFilename));
-}
-

[cpu_(ooo_6.6-7.5), 2014/06/20 13:31:38]

How about you rename this factory to include the word obsolete and to return
either null or a no-op object and leave the other things as they are?

[engedy, 2014/06/20 13:37:45]

I initially did not want to go down that path, as it would entail sending back
meaningless UMA histograms. However, I am happy to do it that way if you think
this is an acceptable trade-off.

 DiagnosticsTest* MakeSqliteCookiesDbTest() {
   return new SqliteIntegrityTest(SqliteIntegrityTest::CRITICAL,
                                  DIAGNOSTICS_SQLITE_INTEGRITY_COOKIE_TEST,