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

Side by Side Diff: chrome/browser/chromeos/drive/sync_client.cc

Issue 150113002: drive: Also lock SearchOperation when locking ChangeListLoader (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/drive/sync_client.h" 5 #include "chrome/browser/chromeos/drive/sync_client.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/message_loop/message_loop_proxy.h" 10 #include "base/message_loop/message_loop_proxy.h"
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 } // namespace 129 } // namespace
130 130
131 SyncClient::SyncTask::SyncTask() : state(PENDING), should_run_again(false) {} 131 SyncClient::SyncTask::SyncTask() : state(PENDING), should_run_again(false) {}
132 SyncClient::SyncTask::~SyncTask() {} 132 SyncClient::SyncTask::~SyncTask() {}
133 133
134 SyncClient::SyncClient(base::SequencedTaskRunner* blocking_task_runner, 134 SyncClient::SyncClient(base::SequencedTaskRunner* blocking_task_runner,
135 file_system::OperationObserver* observer, 135 file_system::OperationObserver* observer,
136 JobScheduler* scheduler, 136 JobScheduler* scheduler,
137 ResourceMetadata* metadata, 137 ResourceMetadata* metadata,
138 FileCache* cache, 138 FileCache* cache,
139 ChangeListLoader* change_list_loader, 139 LoaderController* loader_controller,
140 const base::FilePath& temporary_file_directory) 140 const base::FilePath& temporary_file_directory)
141 : blocking_task_runner_(blocking_task_runner), 141 : blocking_task_runner_(blocking_task_runner),
142 operation_observer_(observer), 142 operation_observer_(observer),
143 metadata_(metadata), 143 metadata_(metadata),
144 cache_(cache), 144 cache_(cache),
145 download_operation_(new file_system::DownloadOperation( 145 download_operation_(new file_system::DownloadOperation(
146 blocking_task_runner, 146 blocking_task_runner,
147 observer, 147 observer,
148 scheduler, 148 scheduler,
149 metadata, 149 metadata,
150 cache, 150 cache,
151 temporary_file_directory)), 151 temporary_file_directory)),
152 entry_update_performer_(new EntryUpdatePerformer(blocking_task_runner, 152 entry_update_performer_(new EntryUpdatePerformer(blocking_task_runner,
153 observer, 153 observer,
154 scheduler, 154 scheduler,
155 metadata, 155 metadata,
156 cache, 156 cache,
157 change_list_loader)), 157 loader_controller)),
158 delay_(base::TimeDelta::FromSeconds(kDelaySeconds)), 158 delay_(base::TimeDelta::FromSeconds(kDelaySeconds)),
159 long_delay_(base::TimeDelta::FromSeconds(kLongDelaySeconds)), 159 long_delay_(base::TimeDelta::FromSeconds(kLongDelaySeconds)),
160 weak_ptr_factory_(this) { 160 weak_ptr_factory_(this) {
161 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 161 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
162 } 162 }
163 163
164 SyncClient::~SyncClient() { 164 SyncClient::~SyncClient() {
165 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 165 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
166 } 166 }
167 167
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
416 file_system::DRIVE_SYNC_ERROR_MISC, 416 file_system::DRIVE_SYNC_ERROR_MISC,
417 local_id); 417 local_id);
418 LOG(WARNING) << "Failed to update " << local_id << ": " 418 LOG(WARNING) << "Failed to update " << local_id << ": "
419 << FileErrorToString(error); 419 << FileErrorToString(error);
420 } 420 }
421 } 421 }
422 } 422 }
423 423
424 } // namespace internal 424 } // namespace internal
425 } // namespace drive 425 } // namespace drive
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/drive/sync_client.h ('k') | chrome/browser/chromeos/drive/sync_client_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698