Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(143)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/history/history_unittest.cc

Issue 722723005: Move constants used by history component to history namespace (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Initial patchset Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/history/history_service.cc ('k') | chrome/browser/history/top_sites_impl.cc » ('j') | chrome/browser/history/url_utils.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/history_unittest.cc
diff --git a/chrome/browser/history/history_unittest.cc b/chrome/browser/history/history_unittest.cc
index 31c831ce7c76bdde9f4d9819e680801ab930aab4..aade885adc3d6ab3e3ffa82122ef6d75f191de9e 100644
--- a/chrome/browser/history/history_unittest.cc
+++ b/chrome/browser/history/history_unittest.cc
@@ -50,9 +50,9 @@
#include "chrome/browser/history/history_service.h"
#include "chrome/browser/history/history_unittest_base.h"
#include "chrome/browser/history/in_memory_history_backend.h"
-#include "chrome/common/chrome_constants.h"
#include "chrome/common/chrome_paths.h"
#include "chrome/tools/profiles/thumbnail-inl.h"
+#include "components/history/core/browser/history_constants.h"
#include "components/history/core/browser/in_memory_database.h"
#include "components/history/core/browser/page_usage_data.h"
#include "components/history/core/common/thumbnail_score.h"
@@ -140,8 +140,7 @@ class HistoryBackendDBTest : public HistoryUnitTestBase {
data_path =
data_path.AppendASCII(base::StringPrintf("history.%d.sql", version));
ASSERT_NO_FATAL_FAILURE(
- ExecuteSQLScript(data_path, history_dir_.Append(
- chrome::kHistoryFilename)));
+ ExecuteSQLScript(data_path, history_dir_.Append(kHistoryFilename)));
}
void CreateArchivedDB() {
@@ -149,9 +148,8 @@ class HistoryBackendDBTest : public HistoryUnitTestBase {
ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &data_path));
data_path = data_path.AppendASCII("History");
data_path = data_path.AppendASCII("archived_history.4.sql");
- ASSERT_NO_FATAL_FAILURE(
- ExecuteSQLScript(data_path, history_dir_.Append(
- chrome::kArchivedHistoryFilename)));
+ ASSERT_NO_FATAL_FAILURE(ExecuteSQLScript(
+ data_path, history_dir_.Append(kArchivedHistoryFilename)));
}
// testing::Test
@@ -288,7 +286,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadsState) {
{
// Open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
// Manually insert corrupted rows; there's infrastructure in place now to
// make this impossible, at least according to the test above.
@@ -318,7 +316,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadsState) {
{
// Re-open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
{
// The version should have been updated.
int cur_version = HistoryDatabase::GetCurrentVersion();
@@ -356,7 +354,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadsReasonPathsAndDangerType) {
{
// Re-open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
// Manually insert some rows.
sql::Statement s(db.GetUniqueStatement(
@@ -399,7 +397,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadsReasonPathsAndDangerType) {
{
// Re-open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
{
// The version should have been updated.
int cur_version = HistoryDatabase::GetCurrentVersion();
@@ -466,7 +464,7 @@ TEST_F(HistoryBackendDBTest, MigrateReferrer) {
ASSERT_NO_FATAL_FAILURE(CreateDBVersion(22));
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
sql::Statement s(db.GetUniqueStatement(
"INSERT INTO downloads (id, full_path, url, start_time, "
"received_bytes, total_bytes, state, end_time, opened) VALUES "
@@ -490,7 +488,7 @@ TEST_F(HistoryBackendDBTest, MigrateReferrer) {
{
// Re-open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
// The version should have been updated.
int cur_version = HistoryDatabase::GetCurrentVersion();
ASSERT_LE(26, cur_version);
@@ -514,7 +512,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadedByExtension) {
ASSERT_NO_FATAL_FAILURE(CreateDBVersion(26));
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
{
sql::Statement s(db.GetUniqueStatement(
"INSERT INTO downloads (id, current_path, target_path, start_time, "
@@ -552,7 +550,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadedByExtension) {
{
// Re-open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
// The version should have been updated.
int cur_version = HistoryDatabase::GetCurrentVersion();
ASSERT_LE(27, cur_version);
@@ -577,7 +575,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadValidators) {
ASSERT_NO_FATAL_FAILURE(CreateDBVersion(27));
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
{
sql::Statement s(db.GetUniqueStatement(
"INSERT INTO downloads (id, current_path, target_path, start_time, "
@@ -617,7 +615,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadValidators) {
{
// Re-open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
// The version should have been updated.
int cur_version = HistoryDatabase::GetCurrentVersion();
ASSERT_LE(28, cur_version);
@@ -641,16 +639,14 @@ TEST_F(HistoryBackendDBTest, PurgeArchivedDatabase) {
ASSERT_NO_FATAL_FAILURE(CreateDBVersion(27));
ASSERT_NO_FATAL_FAILURE(CreateArchivedDB());
- ASSERT_TRUE(base::PathExists(
- history_dir_.Append(chrome::kArchivedHistoryFilename)));
+ ASSERT_TRUE(base::PathExists(history_dir_.Append(kArchivedHistoryFilename)));
CreateBackendAndDatabase();
DeleteBackend();
// We do not retain expired history entries in an archived database as of M37.
// Verify that any legacy archived database is deleted on start-up.
- ASSERT_FALSE(base::PathExists(
- history_dir_.Append(chrome::kArchivedHistoryFilename)));
+ ASSERT_FALSE(base::PathExists(history_dir_.Append(kArchivedHistoryFilename)));
}
TEST_F(HistoryBackendDBTest, MigrateDownloadMimeType) {
@@ -658,7 +654,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadMimeType) {
ASSERT_NO_FATAL_FAILURE(CreateDBVersion(28));
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
{
sql::Statement s(db.GetUniqueStatement(
"INSERT INTO downloads (id, current_path, target_path, start_time, "
@@ -701,7 +697,7 @@ TEST_F(HistoryBackendDBTest, MigrateDownloadMimeType) {
{
// Re-open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
// The version should have been updated.
int cur_version = HistoryDatabase::GetCurrentVersion();
ASSERT_LE(29, cur_version);
@@ -737,7 +733,7 @@ TEST_F(HistoryBackendDBTest, ConfirmDownloadRowCreateAndDelete) {
DeleteBackend();
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
sql::Statement statement(db.GetUniqueStatement(
"Select Count(*) from downloads"));
EXPECT_TRUE(statement.Step());
@@ -756,7 +752,7 @@ TEST_F(HistoryBackendDBTest, ConfirmDownloadRowCreateAndDelete) {
DeleteBackend();
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
sql::Statement statement(db.GetUniqueStatement(
"Select Count(*) from downloads"));
EXPECT_TRUE(statement.Step());
@@ -803,7 +799,7 @@ TEST_F(HistoryBackendDBTest, DownloadNukeRecordsMissingURLs) {
DeleteBackend();
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
sql::Statement statement(db.GetUniqueStatement(
"DELETE FROM downloads_url_chains WHERE id=1"));
ASSERT_TRUE(statement.Run());
@@ -817,7 +813,7 @@ TEST_F(HistoryBackendDBTest, DownloadNukeRecordsMissingURLs) {
DeleteBackend();
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
{
sql::Statement statement(db.GetUniqueStatement(
"SELECT count(*) from downloads"));
@@ -840,7 +836,7 @@ TEST_F(HistoryBackendDBTest, ConfirmDownloadInProgressCleanup) {
DeleteBackend();
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
sql::Statement statement(db.GetUniqueStatement(
"Select Count(*) from downloads"));
EXPECT_TRUE(statement.Step());
@@ -870,7 +866,7 @@ TEST_F(HistoryBackendDBTest, ConfirmDownloadInProgressCleanup) {
DeleteBackend();
{
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
sql::Statement statement(db.GetUniqueStatement(
"Select Count(*) from downloads"));
EXPECT_TRUE(statement.Step());
@@ -1818,7 +1814,7 @@ TEST_F(HistoryBackendDBTest, MigratePresentations) {
{
// Re-open the db for manual manipulation.
sql::Connection db;
- ASSERT_TRUE(db.Open(history_dir_.Append(chrome::kHistoryFilename)));
+ ASSERT_TRUE(db.Open(history_dir_.Append(kHistoryFilename)));
// Add an entry to urls.
{
« no previous file with comments | « chrome/browser/history/history_service.cc ('k') | chrome/browser/history/top_sites_impl.cc » ('j') | chrome/browser/history/url_utils.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698