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

Side by Side Diff: chrome/browser/sync/glue/browser_thread_model_worker.cc

Issue 9036003: Avoid useless SYNC_CYCLE_CONTINUATION sync cycle (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/glue/browser_thread_model_worker.h" 5 #include "chrome/browser/sync/glue/browser_thread_model_worker.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/synchronization/waitable_event.h" 8 #include "base/synchronization/waitable_event.h"
9 #include "content/public/browser/browser_thread.h" 9 #include "content/public/browser/browser_thread.h"
10 10
11 using base::WaitableEvent; 11 using base::WaitableEvent;
12 using content::BrowserThread; 12 using content::BrowserThread;
13 13
14 namespace browser_sync { 14 namespace browser_sync {
15 15
16 BrowserThreadModelWorker::BrowserThreadModelWorker( 16 BrowserThreadModelWorker::BrowserThreadModelWorker(
17 BrowserThread::ID thread, ModelSafeGroup group) 17 BrowserThread::ID thread, ModelSafeGroup group)
18 : thread_(thread), group_(group) {} 18 : thread_(thread), group_(group) {}
19 19
20 BrowserThreadModelWorker::~BrowserThreadModelWorker() {} 20 BrowserThreadModelWorker::~BrowserThreadModelWorker() {}
21 21
22 UnrecoverableErrorInfo BrowserThreadModelWorker::DoWorkAndWaitUntilDone( 22 SyncerError BrowserThreadModelWorker::DoWorkAndWaitUntilDone(
23 const WorkCallback& work) { 23 const WorkCallback& work) {
24 UnrecoverableErrorInfo error_info; 24 SyncerError error = UNINITIALIZED;
25 if (BrowserThread::CurrentlyOn(thread_)) { 25 if (BrowserThread::CurrentlyOn(thread_)) {
26 DLOG(WARNING) << "Already on thread " << thread_; 26 DLOG(WARNING) << "Already on thread " << thread_;
27 return work.Run(); 27 return work.Run();
28 } 28 }
29 WaitableEvent done(false, false); 29 WaitableEvent done(false, false);
30 if (!BrowserThread::PostTask( 30 if (!BrowserThread::PostTask(
31 thread_, 31 thread_,
32 FROM_HERE, 32 FROM_HERE,
33 base::Bind(&BrowserThreadModelWorker::CallDoWorkAndSignalTask, this, 33 base::Bind(&BrowserThreadModelWorker::CallDoWorkAndSignalTask, this,
34 work, &done, &error_info))) { 34 work, &done, &error))) {
35 NOTREACHED() << "Failed to post task to thread " << thread_; 35 NOTREACHED() << "Failed to post task to thread " << thread_;
36 return error_info; 36 return error;
37 } 37 }
38 done.Wait(); 38 done.Wait();
39 return error_info; 39 return error;
40 } 40 }
41 41
42 void BrowserThreadModelWorker::CallDoWorkAndSignalTask( 42 void BrowserThreadModelWorker::CallDoWorkAndSignalTask(
43 const WorkCallback& work, 43 const WorkCallback& work,
44 WaitableEvent* done, 44 WaitableEvent* done,
45 UnrecoverableErrorInfo* error_info) { 45 SyncerError* error) {
46 DCHECK(BrowserThread::CurrentlyOn(thread_)); 46 DCHECK(BrowserThread::CurrentlyOn(thread_));
47 *error_info = work.Run(); 47 *error = work.Run();
48 done->Signal(); 48 done->Signal();
49 } 49 }
50 50
51 ModelSafeGroup BrowserThreadModelWorker::GetModelSafeGroup() { 51 ModelSafeGroup BrowserThreadModelWorker::GetModelSafeGroup() {
52 return group_; 52 return group_;
53 } 53 }
54 54
55 DatabaseModelWorker::DatabaseModelWorker() 55 DatabaseModelWorker::DatabaseModelWorker()
56 : BrowserThreadModelWorker(BrowserThread::DB, GROUP_DB) {} 56 : BrowserThreadModelWorker(BrowserThread::DB, GROUP_DB) {}
57 57
58 DatabaseModelWorker::~DatabaseModelWorker() {} 58 DatabaseModelWorker::~DatabaseModelWorker() {}
59 59
60 void DatabaseModelWorker::CallDoWorkAndSignalTask( 60 void DatabaseModelWorker::CallDoWorkAndSignalTask(
61 const WorkCallback& work, 61 const WorkCallback& work,
62 WaitableEvent* done, 62 WaitableEvent* done,
63 UnrecoverableErrorInfo* error_info) { 63 SyncerError* error) {
64 BrowserThreadModelWorker::CallDoWorkAndSignalTask(work, done, error_info); 64 BrowserThreadModelWorker::CallDoWorkAndSignalTask(work, done, error);
65 } 65 }
66 66
67 FileModelWorker::FileModelWorker() 67 FileModelWorker::FileModelWorker()
68 : BrowserThreadModelWorker(BrowserThread::FILE, GROUP_FILE) {} 68 : BrowserThreadModelWorker(BrowserThread::FILE, GROUP_FILE) {}
69 69
70 FileModelWorker::~FileModelWorker() {} 70 FileModelWorker::~FileModelWorker() {}
71 71
72 void FileModelWorker::CallDoWorkAndSignalTask( 72 void FileModelWorker::CallDoWorkAndSignalTask(
73 const WorkCallback& work, 73 const WorkCallback& work,
74 WaitableEvent* done, 74 WaitableEvent* done,
75 UnrecoverableErrorInfo* error_info) { 75 SyncerError* error) {
76 BrowserThreadModelWorker::CallDoWorkAndSignalTask(work, done, error_info); 76 BrowserThreadModelWorker::CallDoWorkAndSignalTask(work, done, error);
77 } 77 }
78 78
79 } // namespace browser_sync 79 } // namespace browser_sync
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698