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

Unified Diff: chrome/browser/extensions/api/storage/settings_sync_processor.cc

Issue 2965153002: Migrate Extensions code to Task Scheduler API (Closed)
Patch Set: Self review Created 3 years, 5 months 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
Index: chrome/browser/extensions/api/storage/settings_sync_processor.cc
diff --git a/chrome/browser/extensions/api/storage/settings_sync_processor.cc b/chrome/browser/extensions/api/storage/settings_sync_processor.cc
index 02f3f6f926627c385cab86a65020b4ea6fe9e1b6..9e880bd46f46819be9dfaeccd8cb7244ece5d022 100644
--- a/chrome/browser/extensions/api/storage/settings_sync_processor.cc
+++ b/chrome/browser/extensions/api/storage/settings_sync_processor.cc
@@ -7,11 +7,9 @@
#include "components/sync/model/sync_change_processor.h"
#include "components/sync/model/sync_data.h"
#include "components/sync/protocol/extension_setting_specifics.pb.h"
-#include "content/public/browser/browser_thread.h"
+#include "extensions/browser/api/storage/backend_task_runner.h"
#include "extensions/browser/api/storage/settings_namespace.h"
-using content::BrowserThread;
-
namespace extensions {
SettingsSyncProcessor::SettingsSyncProcessor(
@@ -22,17 +20,17 @@ SettingsSyncProcessor::SettingsSyncProcessor(
type_(type),
sync_processor_(sync_processor),
initialized_(false) {
- DCHECK_CURRENTLY_ON(BrowserThread::FILE);
+ DCHECK(IsOnBackendSequence());
CHECK(type == syncer::EXTENSION_SETTINGS || type == syncer::APP_SETTINGS);
CHECK(sync_processor);
}
SettingsSyncProcessor::~SettingsSyncProcessor() {
- DCHECK_CURRENTLY_ON(BrowserThread::FILE);
+ DCHECK(IsOnBackendSequence());
}
void SettingsSyncProcessor::Init(const base::DictionaryValue& initial_state) {
- DCHECK_CURRENTLY_ON(BrowserThread::FILE);
+ DCHECK(IsOnBackendSequence());
CHECK(!initialized_) << "Init called multiple times";
for (base::DictionaryValue::Iterator i(initial_state); !i.IsAtEnd();
@@ -44,7 +42,7 @@ void SettingsSyncProcessor::Init(const base::DictionaryValue& initial_state) {
syncer::SyncError SettingsSyncProcessor::SendChanges(
const ValueStoreChangeList& changes) {
- DCHECK_CURRENTLY_ON(BrowserThread::FILE);
+ DCHECK(IsOnBackendSequence());
CHECK(initialized_) << "Init not called";
syncer::SyncChangeList sync_changes;
@@ -96,7 +94,7 @@ syncer::SyncError SettingsSyncProcessor::SendChanges(
}
void SettingsSyncProcessor::NotifyChanges(const ValueStoreChangeList& changes) {
- DCHECK_CURRENTLY_ON(BrowserThread::FILE);
+ DCHECK(IsOnBackendSequence());
CHECK(initialized_) << "Init not called";
for (ValueStoreChangeList::const_iterator i = changes.begin();

Powered by Google App Engine
This is Rietveld 408576698