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

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

Issue 1731483003: chrome: Add out-of-line copy ctors for complex classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
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/sync_task_manager.h" 5 #include "chrome/browser/sync_file_system/drive_backend/sync_task_manager.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 27 matching lines...) Expand all
38 }; 38 };
39 39
40 } // namespace 40 } // namespace
41 41
42 SyncTaskManager::PendingTask::PendingTask() {} 42 SyncTaskManager::PendingTask::PendingTask() {}
43 43
44 SyncTaskManager::PendingTask::PendingTask( 44 SyncTaskManager::PendingTask::PendingTask(
45 const base::Closure& task, Priority pri, int seq) 45 const base::Closure& task, Priority pri, int seq)
46 : task(task), priority(pri), seq(seq) {} 46 : task(task), priority(pri), seq(seq) {}
47 47
48 SyncTaskManager::PendingTask::PendingTask(const PendingTask& other) = default;
49
48 SyncTaskManager::PendingTask::~PendingTask() {} 50 SyncTaskManager::PendingTask::~PendingTask() {}
49 51
50 bool SyncTaskManager::PendingTaskComparator::operator()( 52 bool SyncTaskManager::PendingTaskComparator::operator()(
51 const PendingTask& left, 53 const PendingTask& left,
52 const PendingTask& right) const { 54 const PendingTask& right) const {
53 if (left.priority != right.priority) 55 if (left.priority != right.priority)
54 return left.priority < right.priority; 56 return left.priority < right.priority;
55 return left.seq > right.seq; 57 return left.seq > right.seq;
56 } 58 }
57 59
(...skipping 346 matching lines...) Expand 10 before | Expand all | Expand 10 after
404 closure.Run(); 406 closure.Run();
405 return; 407 return;
406 } 408 }
407 409
408 if (client_) 410 if (client_)
409 client_->MaybeScheduleNextTask(); 411 client_->MaybeScheduleNextTask();
410 } 412 }
411 413
412 } // namespace drive_backend 414 } // namespace drive_backend
413 } // namespace sync_file_system 415 } // namespace sync_file_system
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698