summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--content/browser/browser_context.cc2
-rw-r--r--webkit/database/database_quota_client_unittest.cc2
-rw-r--r--webkit/database/database_tracker.cc3
-rw-r--r--webkit/database/database_tracker.h1
-rw-r--r--webkit/database/database_tracker_unittest.cc28
-rw-r--r--webkit/support/simple_database_system.cc2
6 files changed, 17 insertions, 21 deletions
diff --git a/content/browser/browser_context.cc b/content/browser/browser_context.cc
index 5a74bed..c6c48a7 100644
--- a/content/browser/browser_context.cc
+++ b/content/browser/browser_context.cc
@@ -65,7 +65,7 @@ void CreateQuotaManagerAndClients(BrowserContext* context) {
new UserDataAdapter<FileSystemContext>(filesystem_context));
scoped_refptr<DatabaseTracker> db_tracker = new DatabaseTracker(
- context->GetPath(), context->IsOffTheRecord(), false,
+ context->GetPath(), context->IsOffTheRecord(),
context->GetSpecialStoragePolicy(), quota_manager->proxy(),
BrowserThread::GetMessageLoopProxyForThread(BrowserThread::FILE));
context->SetUserData(kDatabaseTrackerKeyName,
diff --git a/webkit/database/database_quota_client_unittest.cc b/webkit/database/database_quota_client_unittest.cc
index b070046..ee62722 100644
--- a/webkit/database/database_quota_client_unittest.cc
+++ b/webkit/database/database_quota_client_unittest.cc
@@ -27,7 +27,7 @@ static const quota::StorageType kPerm = quota::kStorageTypePersistent;
class MockDatabaseTracker : public DatabaseTracker {
public:
MockDatabaseTracker()
- : DatabaseTracker(FilePath(), false, false, NULL, NULL, NULL),
+ : DatabaseTracker(FilePath(), false, NULL, NULL, NULL),
delete_called_count_(0),
async_delete_(false) {}
diff --git a/webkit/database/database_tracker.cc b/webkit/database/database_tracker.cc
index 2264997..8ffa7b1 100644
--- a/webkit/database/database_tracker.cc
+++ b/webkit/database/database_tracker.cc
@@ -93,13 +93,12 @@ OriginInfo::OriginInfo(const string16& origin, int64 total_size)
DatabaseTracker::DatabaseTracker(
const FilePath& profile_path,
bool is_incognito,
- bool clear_local_state_on_exit,
quota::SpecialStoragePolicy* special_storage_policy,
quota::QuotaManagerProxy* quota_manager_proxy,
base::MessageLoopProxy* db_tracker_thread)
: is_initialized_(false),
is_incognito_(is_incognito),
- clear_local_state_on_exit_(clear_local_state_on_exit),
+ clear_local_state_on_exit_(false),
save_session_state_(false),
shutting_down_(false),
profile_path_(profile_path),
diff --git a/webkit/database/database_tracker.h b/webkit/database/database_tracker.h
index 340cdcc..709e662 100644
--- a/webkit/database/database_tracker.h
+++ b/webkit/database/database_tracker.h
@@ -91,7 +91,6 @@ class DatabaseTracker
DatabaseTracker(const FilePath& profile_path,
bool is_incognito,
- bool clear_local_state_on_exit,
quota::SpecialStoragePolicy* special_storage_policy,
quota::QuotaManagerProxy* quota_manager_proxy,
base::MessageLoopProxy* db_tracker_thread);
diff --git a/webkit/database/database_tracker_unittest.cc b/webkit/database/database_tracker_unittest.cc
index 88babea..14d72a8 100644
--- a/webkit/database/database_tracker_unittest.cc
+++ b/webkit/database/database_tracker_unittest.cc
@@ -191,7 +191,7 @@ class DatabaseTracker_TestHelper_Test {
new quota::MockSpecialStoragePolicy;
special_storage_policy->AddProtected(GURL(kOrigin2Url));
scoped_refptr<DatabaseTracker> tracker(
- new DatabaseTracker(temp_dir.path(), incognito_mode, false,
+ new DatabaseTracker(temp_dir.path(), incognito_mode,
special_storage_policy, NULL, NULL));
// Create and open three databases.
@@ -296,7 +296,7 @@ class DatabaseTracker_TestHelper_Test {
new quota::MockSpecialStoragePolicy;
special_storage_policy->AddProtected(GURL(kOrigin2Url));
scoped_refptr<DatabaseTracker> tracker(
- new DatabaseTracker(temp_dir.path(), incognito_mode, false,
+ new DatabaseTracker(temp_dir.path(), incognito_mode,
special_storage_policy, NULL, NULL));
// Add two observers.
@@ -446,7 +446,7 @@ class DatabaseTracker_TestHelper_Test {
scoped_refptr<TestQuotaManagerProxy> test_quota_proxy(
new TestQuotaManagerProxy);
scoped_refptr<DatabaseTracker> tracker(
- new DatabaseTracker(temp_dir.path(), false /* incognito */, false,
+ new DatabaseTracker(temp_dir.path(), false /* incognito */,
NULL, test_quota_proxy, NULL));
EXPECT_TRUE(test_quota_proxy->registered_client_);
@@ -548,9 +548,10 @@ class DatabaseTracker_TestHelper_Test {
special_storage_policy->AddProtected(GURL(kOrigin2Url));
scoped_refptr<DatabaseTracker> tracker(
new DatabaseTracker(
- temp_dir.path(), false, true,
+ temp_dir.path(), false,
special_storage_policy, NULL,
base::MessageLoopProxy::current()));
+ tracker->SetClearLocalStateOnExit(true);
// Open three new databases.
tracker->DatabaseOpened(kOrigin1, kDB1, kDescription, 0,
@@ -610,7 +611,7 @@ class DatabaseTracker_TestHelper_Test {
new quota::MockSpecialStoragePolicy;
special_storage_policy->AddProtected(GURL(kOrigin2Url));
scoped_refptr<DatabaseTracker> tracker(
- new DatabaseTracker(temp_dir.path(), false, false,
+ new DatabaseTracker(temp_dir.path(), false,
special_storage_policy, NULL, NULL));
// Get all data for all origins.
@@ -648,8 +649,7 @@ class DatabaseTracker_TestHelper_Test {
special_storage_policy->AddSessionOnly(GURL(kOrigin2Url));
scoped_refptr<DatabaseTracker> tracker(
new DatabaseTracker(
- temp_dir.path(), false, false /*clear_local_state_on_exit*/,
- special_storage_policy, NULL,
+ temp_dir.path(), false, special_storage_policy, NULL,
base::MessageLoopProxy::current()));
// Open two new databases.
@@ -686,8 +686,7 @@ class DatabaseTracker_TestHelper_Test {
// At this point, the database tracker should be gone. Create a new one.
scoped_refptr<DatabaseTracker> tracker(
- new DatabaseTracker(temp_dir.path(), false, false,
- NULL, NULL, NULL));
+ new DatabaseTracker(temp_dir.path(), false, NULL, NULL, NULL));
// Get all data for all origins.
std::vector<OriginInfo> origins_info;
@@ -727,9 +726,9 @@ class DatabaseTracker_TestHelper_Test {
special_storage_policy->AddSessionOnly(GURL(kOrigin2Url));
scoped_refptr<DatabaseTracker> tracker(
new DatabaseTracker(
- temp_dir.path(), false, true /*clear_local_state_on_exit*/,
- special_storage_policy, NULL,
+ temp_dir.path(), false, special_storage_policy, NULL,
base::MessageLoopProxy::current()));
+ tracker->SetClearLocalStateOnExit(true);
tracker->SaveSessionState();
// Open two new databases.
@@ -766,8 +765,7 @@ class DatabaseTracker_TestHelper_Test {
// At this point, the database tracker should be gone. Create a new one.
scoped_refptr<DatabaseTracker> tracker(
- new DatabaseTracker(temp_dir.path(), false, false,
- NULL, NULL, NULL));
+ new DatabaseTracker(temp_dir.path(), false, NULL, NULL, NULL));
// Get all data for all origins.
std::vector<OriginInfo> origins_info;
@@ -796,7 +794,7 @@ class DatabaseTracker_TestHelper_Test {
ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
scoped_refptr<DatabaseTracker> tracker(
new DatabaseTracker(temp_dir.path(), kUseInMemoryTrackerDatabase,
- false, NULL, NULL, NULL));
+ NULL, NULL, NULL));
// Starts off with no databases.
std::vector<OriginInfo> infos;
@@ -842,7 +840,7 @@ class DatabaseTracker_TestHelper_Test {
ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
scoped_refptr<DatabaseTracker> tracker(
new DatabaseTracker(temp_dir.path(), kUseInMemoryTrackerDatabase,
- false, NULL, NULL, NULL));
+ NULL, NULL, NULL));
// Setup to observe OnScheduledForDelete notifications.
TestObserver observer(false, true);
diff --git a/webkit/support/simple_database_system.cc b/webkit/support/simple_database_system.cc
index 77fec92..845f9c4 100644
--- a/webkit/support/simple_database_system.cc
+++ b/webkit/support/simple_database_system.cc
@@ -39,7 +39,7 @@ SimpleDatabaseSystem::SimpleDatabaseSystem()
instance_ = this;
CHECK(temp_dir_.CreateUniqueTempDir());
db_tracker_ =
- new DatabaseTracker(temp_dir_.path(), false, false, NULL, NULL, NULL);
+ new DatabaseTracker(temp_dir_.path(), false, NULL, NULL, NULL);
db_tracker_->AddObserver(this);
db_thread_.Start();
db_thread_proxy_ = db_thread_.message_loop_proxy();