Remove DISALLOW_COPY_AND_ASSIGN from base/*
Also fix cpp lint warnings. No functional changes.
Bug: 1010217
Change-Id: I60c13a4f4667522c7661b3ab91b948e71a06e657
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432204
Commit-Queue: David Bienvenu <[email protected]>
Reviewed-by: Lei Zhang <[email protected]>
Cr-Commit-Position: refs/heads/master@{#811059}
diff --git a/base/observer_list_threadsafe_unittest.cc b/base/observer_list_threadsafe_unittest.cc
index efcb423..446c516 100644
--- a/base/observer_list_threadsafe_unittest.cc
+++ b/base/observer_list_threadsafe_unittest.cc
@@ -304,6 +304,9 @@
explicit SequenceVerificationObserver(
scoped_refptr<SequencedTaskRunner> task_runner)
: task_runner_(std::move(task_runner)) {}
+ SequenceVerificationObserver(const SequenceVerificationObserver&) = delete;
+ SequenceVerificationObserver& operator=(const SequenceVerificationObserver&) =
+ delete;
~SequenceVerificationObserver() override = default;
void Observe(int x) override {
@@ -315,8 +318,6 @@
private:
const scoped_refptr<SequencedTaskRunner> task_runner_;
bool called_on_valid_sequence_ = false;
-
- DISALLOW_COPY_AND_ASSIGN(SequenceVerificationObserver);
};
} // namespace
@@ -378,6 +379,10 @@
WaitableEvent::InitialState::NOT_SIGNALED),
barrier_(WaitableEvent::ResetPolicy::AUTOMATIC,
WaitableEvent::InitialState::NOT_SIGNALED) {}
+ RemoveWhileNotificationIsRunningObserver(
+ const RemoveWhileNotificationIsRunningObserver&) = delete;
+ RemoveWhileNotificationIsRunningObserver& operator=(
+ const RemoveWhileNotificationIsRunningObserver&) = delete;
~RemoveWhileNotificationIsRunningObserver() override = default;
void Observe(int x) override {
@@ -392,8 +397,6 @@
private:
WaitableEvent notification_running_;
WaitableEvent barrier_;
-
- DISALLOW_COPY_AND_ASSIGN(RemoveWhileNotificationIsRunningObserver);
};
} // namespace
