Replace DISALLOW_COPY_AND_ASSIGN in remoting/
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: Ibbae02ba444d7308489dfb03c31be23dcfb3c2cb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3174302
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@{#923948}
diff --git a/remoting/host/host_window.h b/remoting/host/host_window.h
index 66a12b6..ba1d456 100644
--- a/remoting/host/host_window.h
+++ b/remoting/host/host_window.h
@@ -19,6 +19,9 @@
class HostWindow {
public:
+ HostWindow(const HostWindow&) = delete;
+ HostWindow& operator=(const HostWindow&) = delete;
+
virtual ~HostWindow();
// Creates a platform-specific instance of the continue window.
@@ -46,9 +49,6 @@
HostWindow() {}
SEQUENCE_CHECKER(sequence_checker_);
-
- private:
- DISALLOW_COPY_AND_ASSIGN(HostWindow);
};
} // namespace remoting
