Replace DISALLOW_COPY_AND_ASSIGN in ash/
This replaces DISALLOW_COPY_AND_ASSIGN with explicit constructor deletes
where a local script is able to detect its insertion place (~Foo() is
public => insert before this line).
This is incomplete as not all classes have a public ~Foo() declared, so
not all DISALLOW_COPY_AND_ASSIGN occurrences are replaced.
IWYU cleanup is left as a separate pass that is easier when these macros
go away.
Bug: 1010217
Change-Id: Iedd7c835cea6c26dd4655c11e875cbf1b06bd1ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3163767
Auto-Submit: Peter Boström <[email protected]>
Commit-Queue: Daniel Cheng <[email protected]>
Reviewed-by: Daniel Cheng <[email protected]>
Owners-Override: Daniel Cheng <[email protected]>
Cr-Commit-Position: refs/heads/main@{#922357}
diff --git a/ash/shell_test_api.cc b/ash/shell_test_api.cc
index 11886ec..19b0554 100644
--- a/ash/shell_test_api.cc
+++ b/ash/shell_test_api.cc
@@ -50,6 +50,9 @@
window_tree_host_->compositor()->AddObserver(this);
}
+ PointerMoveLoopWaiter(const PointerMoveLoopWaiter&) = delete;
+ PointerMoveLoopWaiter& operator=(const PointerMoveLoopWaiter&) = delete;
+
~PointerMoveLoopWaiter() override {
window_tree_host_->compositor()->RemoveObserver(this);
}
@@ -74,8 +77,6 @@
private:
aura::WindowTreeHost* window_tree_host_;
std::unique_ptr<base::RunLoop> run_loop_;
-
- DISALLOW_COPY_AND_ASSIGN(PointerMoveLoopWaiter);
};
class WindowAnimationWaiter : public ui::LayerAnimationObserver {
