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/me2me_desktop_environment.h b/remoting/host/me2me_desktop_environment.h
index 535820b..54989376 100644
--- a/remoting/host/me2me_desktop_environment.h
+++ b/remoting/host/me2me_desktop_environment.h
@@ -19,6 +19,9 @@
// notifications on Linux.
class Me2MeDesktopEnvironment : public BasicDesktopEnvironment {
public:
+ Me2MeDesktopEnvironment(const Me2MeDesktopEnvironment&) = delete;
+ Me2MeDesktopEnvironment& operator=(const Me2MeDesktopEnvironment&) = delete;
+
~Me2MeDesktopEnvironment() override;
// DesktopEnvironment interface.
@@ -51,8 +54,6 @@
// Notifies the client session about the local mouse movements.
std::unique_ptr<LocalInputMonitor> local_input_monitor_;
-
- DISALLOW_COPY_AND_ASSIGN(Me2MeDesktopEnvironment);
};
// Used to create |Me2MeDesktopEnvironment| instances.
@@ -63,15 +64,18 @@
scoped_refptr<base::SingleThreadTaskRunner> video_capture_task_runner,
scoped_refptr<base::SingleThreadTaskRunner> input_task_runner,
scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner);
+
+ Me2MeDesktopEnvironmentFactory(const Me2MeDesktopEnvironmentFactory&) =
+ delete;
+ Me2MeDesktopEnvironmentFactory& operator=(
+ const Me2MeDesktopEnvironmentFactory&) = delete;
+
~Me2MeDesktopEnvironmentFactory() override;
// DesktopEnvironmentFactory interface.
std::unique_ptr<DesktopEnvironment> Create(
base::WeakPtr<ClientSessionControl> client_session_control,
const DesktopEnvironmentOptions& options) override;
-
- private:
- DISALLOW_COPY_AND_ASSIGN(Me2MeDesktopEnvironmentFactory);
};
} // namespace remoting
