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

Side by Side Diff: chrome/browser/sync_file_system/drive_backend/remote_change_processor_on_worker.cc

Issue 2189113002: Rename CalledOnValidSequencedThread() to CalledOnValidSequence(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/sync_file_system/drive_backend/remote_change_processor_ on_worker.h" 5 #include "chrome/browser/sync_file_system/drive_backend/remote_change_processor_ on_worker.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "base/memory/weak_ptr.h" 9 #include "base/memory/weak_ptr.h"
10 #include "base/single_thread_task_runner.h" 10 #include "base/single_thread_task_runner.h"
11 #include "chrome/browser/sync_file_system/drive_backend/callback_helper.h" 11 #include "chrome/browser/sync_file_system/drive_backend/callback_helper.h"
12 #include "chrome/browser/sync_file_system/drive_backend/remote_change_processor_ wrapper.h" 12 #include "chrome/browser/sync_file_system/drive_backend/remote_change_processor_ wrapper.h"
13 #include "chrome/browser/sync_file_system/file_change.h" 13 #include "chrome/browser/sync_file_system/file_change.h"
14 #include "chrome/browser/sync_file_system/sync_file_metadata.h" 14 #include "chrome/browser/sync_file_system/sync_file_metadata.h"
15 15
16 namespace sync_file_system { 16 namespace sync_file_system {
17 namespace drive_backend { 17 namespace drive_backend {
18 18
19 RemoteChangeProcessorOnWorker::RemoteChangeProcessorOnWorker( 19 RemoteChangeProcessorOnWorker::RemoteChangeProcessorOnWorker(
20 const base::WeakPtr<RemoteChangeProcessorWrapper>& wrapper, 20 const base::WeakPtr<RemoteChangeProcessorWrapper>& wrapper,
21 base::SingleThreadTaskRunner* ui_task_runner, 21 base::SingleThreadTaskRunner* ui_task_runner,
22 base::SequencedTaskRunner* worker_task_runner) 22 base::SequencedTaskRunner* worker_task_runner)
23 : wrapper_(wrapper), 23 : wrapper_(wrapper),
24 ui_task_runner_(ui_task_runner), 24 ui_task_runner_(ui_task_runner),
25 worker_task_runner_(worker_task_runner) { 25 worker_task_runner_(worker_task_runner) {
26 sequence_checker_.DetachFromSequence(); 26 sequence_checker_.DetachFromSequence();
27 } 27 }
28 28
29 RemoteChangeProcessorOnWorker::~RemoteChangeProcessorOnWorker() { 29 RemoteChangeProcessorOnWorker::~RemoteChangeProcessorOnWorker() {
30 DCHECK(sequence_checker_.CalledOnValidSequencedThread()); 30 DCHECK(sequence_checker_.CalledOnValidSequence());
31 } 31 }
32 32
33 void RemoteChangeProcessorOnWorker::PrepareForProcessRemoteChange( 33 void RemoteChangeProcessorOnWorker::PrepareForProcessRemoteChange(
34 const storage::FileSystemURL& url, 34 const storage::FileSystemURL& url,
35 const PrepareChangeCallback& callback) { 35 const PrepareChangeCallback& callback) {
36 DCHECK(sequence_checker_.CalledOnValidSequencedThread()); 36 DCHECK(sequence_checker_.CalledOnValidSequence());
37 37
38 ui_task_runner_->PostTask( 38 ui_task_runner_->PostTask(
39 FROM_HERE, 39 FROM_HERE,
40 base::Bind(&RemoteChangeProcessorWrapper::PrepareForProcessRemoteChange, 40 base::Bind(&RemoteChangeProcessorWrapper::PrepareForProcessRemoteChange,
41 wrapper_, 41 wrapper_,
42 url, 42 url,
43 RelayCallbackToTaskRunner( 43 RelayCallbackToTaskRunner(
44 worker_task_runner_.get(), FROM_HERE, callback))); 44 worker_task_runner_.get(), FROM_HERE, callback)));
45 } 45 }
46 46
47 void RemoteChangeProcessorOnWorker::ApplyRemoteChange( 47 void RemoteChangeProcessorOnWorker::ApplyRemoteChange(
48 const FileChange& change, 48 const FileChange& change,
49 const base::FilePath& local_path, 49 const base::FilePath& local_path,
50 const storage::FileSystemURL& url, 50 const storage::FileSystemURL& url,
51 const SyncStatusCallback& callback) { 51 const SyncStatusCallback& callback) {
52 DCHECK(sequence_checker_.CalledOnValidSequencedThread()); 52 DCHECK(sequence_checker_.CalledOnValidSequence());
53 53
54 ui_task_runner_->PostTask( 54 ui_task_runner_->PostTask(
55 FROM_HERE, 55 FROM_HERE,
56 base::Bind(&RemoteChangeProcessorWrapper::ApplyRemoteChange, 56 base::Bind(&RemoteChangeProcessorWrapper::ApplyRemoteChange,
57 wrapper_, 57 wrapper_,
58 change, 58 change,
59 local_path, 59 local_path,
60 url, 60 url,
61 RelayCallbackToTaskRunner( 61 RelayCallbackToTaskRunner(
62 worker_task_runner_.get(), FROM_HERE, callback))); 62 worker_task_runner_.get(), FROM_HERE, callback)));
63 } 63 }
64 64
65 void RemoteChangeProcessorOnWorker::FinalizeRemoteSync( 65 void RemoteChangeProcessorOnWorker::FinalizeRemoteSync(
66 const storage::FileSystemURL& url, 66 const storage::FileSystemURL& url,
67 bool clear_local_changes, 67 bool clear_local_changes,
68 const base::Closure& completion_callback) { 68 const base::Closure& completion_callback) {
69 DCHECK(sequence_checker_.CalledOnValidSequencedThread()); 69 DCHECK(sequence_checker_.CalledOnValidSequence());
70 70
71 ui_task_runner_->PostTask( 71 ui_task_runner_->PostTask(
72 FROM_HERE, 72 FROM_HERE,
73 base::Bind( 73 base::Bind(
74 &RemoteChangeProcessorWrapper::FinalizeRemoteSync, 74 &RemoteChangeProcessorWrapper::FinalizeRemoteSync,
75 wrapper_, 75 wrapper_,
76 url, 76 url,
77 clear_local_changes, 77 clear_local_changes,
78 RelayCallbackToTaskRunner( 78 RelayCallbackToTaskRunner(
79 worker_task_runner_.get(), FROM_HERE, completion_callback))); 79 worker_task_runner_.get(), FROM_HERE, completion_callback)));
80 } 80 }
81 81
82 void RemoteChangeProcessorOnWorker::RecordFakeLocalChange( 82 void RemoteChangeProcessorOnWorker::RecordFakeLocalChange(
83 const storage::FileSystemURL& url, 83 const storage::FileSystemURL& url,
84 const FileChange& change, 84 const FileChange& change,
85 const SyncStatusCallback& callback) { 85 const SyncStatusCallback& callback) {
86 DCHECK(sequence_checker_.CalledOnValidSequencedThread()); 86 DCHECK(sequence_checker_.CalledOnValidSequence());
87 87
88 ui_task_runner_->PostTask( 88 ui_task_runner_->PostTask(
89 FROM_HERE, 89 FROM_HERE,
90 base::Bind(&RemoteChangeProcessorWrapper::RecordFakeLocalChange, 90 base::Bind(&RemoteChangeProcessorWrapper::RecordFakeLocalChange,
91 wrapper_, 91 wrapper_,
92 url, 92 url,
93 change, 93 change,
94 RelayCallbackToTaskRunner( 94 RelayCallbackToTaskRunner(
95 worker_task_runner_.get(), FROM_HERE, callback))); 95 worker_task_runner_.get(), FROM_HERE, callback)));
96 } 96 }
97 97
98 void RemoteChangeProcessorOnWorker::DetachFromSequence() { 98 void RemoteChangeProcessorOnWorker::DetachFromSequence() {
99 sequence_checker_.DetachFromSequence(); 99 sequence_checker_.DetachFromSequence();
100 } 100 }
101 101
102 } // namespace drive_backend 102 } // namespace drive_backend
103 } // namespace sync_file_system 103 } // namespace sync_file_system
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698