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

Unified Diff: chrome/browser/sync_file_system/sync_process_runner.cc

Issue 377563002: [SyncFS] Eliminate SyncFileSystemService dependency from SyncProcessRunner (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
« no previous file with comments | « chrome/browser/sync_file_system/sync_process_runner.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync_file_system/sync_process_runner.cc
diff --git a/chrome/browser/sync_file_system/sync_process_runner.cc b/chrome/browser/sync_file_system/sync_process_runner.cc
index 2d880b0f936bb3d6da2c738f2ff398836cad2939..d7c71cecdd5649859cb6c03cd7f7d3234d44b204 100644
--- a/chrome/browser/sync_file_system/sync_process_runner.cc
+++ b/chrome/browser/sync_file_system/sync_process_runner.cc
@@ -6,7 +6,6 @@
#include "base/format_macros.h"
#include "chrome/browser/sync_file_system/logger.h"
-#include "chrome/browser/sync_file_system/sync_file_system_service.h"
namespace sync_file_system {
@@ -66,11 +65,11 @@ bool WasSuccessfulSync(SyncStatusCode status) {
SyncProcessRunner::SyncProcessRunner(
const std::string& name,
- SyncFileSystemService* sync_service,
+ Client* client,
scoped_ptr<TimerHelper> timer_helper,
int max_parallel_task)
: name_(name),
- sync_service_(sync_service),
+ client_(client),
max_parallel_task_(max_parallel_task),
running_tasks_(0),
timer_helper_(timer_helper.Pass()),
@@ -130,7 +129,7 @@ void SyncProcessRunner::OnChangesUpdated(
pending_changes_ = pending_changes;
if (old_pending_changes != pending_changes) {
if (pending_changes == 0)
- sync_service()->OnSyncIdle();
+ client_->OnSyncIdle();
util::Log(logging::LOG_VERBOSE, FROM_HERE,
"[%s] pending_changes updated: %" PRId64,
name_.c_str(), pending_changes);
@@ -138,8 +137,12 @@ void SyncProcessRunner::OnChangesUpdated(
Schedule();
}
+SyncFileSystemService* SyncProcessRunner::GetSyncService() {
+ return client_->GetSyncService();
+}
+
SyncServiceState SyncProcessRunner::GetServiceState() {
- return sync_service()->GetSyncServiceState();
+ return client_->GetSyncServiceState();
}
void SyncProcessRunner::Finished(const base::TimeTicks& start_time,
« no previous file with comments | « chrome/browser/sync_file_system/sync_process_runner.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698