Replace DISALLOW_COPY_AND_ASSIGN in components/

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: I917bfbff1357e9e2cfe330d242feff6bb73e1bce
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3167004
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Owners-Override: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/main@{#923103}

components/safe_search_api/fake_url_checker_client.h

@@ -16,6 +16,10 @@ namespace safe_search_api {
 class FakeURLCheckerClient : public URLCheckerClient {
  public:
   FakeURLCheckerClient();
+
+  FakeURLCheckerClient(const FakeURLCheckerClient&) = delete;
+  FakeURLCheckerClient& operator=(const FakeURLCheckerClient&) = delete;
+
   ~FakeURLCheckerClient() override;
 
   // Fake override that simply holds references of |url| and |callback|.
@@ -33,8 +37,6 @@ class FakeURLCheckerClient : public URLCheckerClient {
  private:
   ClientCheckCallback callback_;
   GURL url_;
-
-  DISALLOW_COPY_AND_ASSIGN(FakeURLCheckerClient);
 };
 
 }  // namespace safe_search_api

components/safe_search_api/safe_search/safe_search_url_checker_client.h

@@ -31,6 +31,10 @@ class SafeSearchURLCheckerClient : public URLCheckerClient {
       const net::NetworkTrafficAnnotationTag& traffic_annotation,
       const std::string& api_key = google_apis::GetAPIKey());
 
+  SafeSearchURLCheckerClient(const SafeSearchURLCheckerClient&) = delete;
+  SafeSearchURLCheckerClient& operator=(const SafeSearchURLCheckerClient&) =
+      delete;
+
   ~SafeSearchURLCheckerClient() override;
 
   // Checks whether an |url| is restricted according to SafeSearch.
@@ -52,8 +56,6 @@ class SafeSearchURLCheckerClient : public URLCheckerClient {
   const std::string api_key_;
 
   CheckList checks_in_progress_;
-
-  DISALLOW_COPY_AND_ASSIGN(SafeSearchURLCheckerClient);
 };
 
 }  // namespace safe_search_api

components/safe_search_api/stub_url_checker.h

@@ -29,6 +29,10 @@ class URLChecker;
 class StubURLChecker {
  public:
   StubURLChecker();
+
+  StubURLChecker(const StubURLChecker&) = delete;
+  StubURLChecker& operator=(const StubURLChecker&) = delete;
+
   ~StubURLChecker();
 
   // Returns a URLChecker that will use the stubbed-out responses. Can be called
@@ -51,8 +55,6 @@ class StubURLChecker {
 
   network::TestURLLoaderFactory test_url_loader_factory_;
   scoped_refptr<network::SharedURLLoaderFactory> test_shared_loader_factory_;
-
-  DISALLOW_COPY_AND_ASSIGN(StubURLChecker);
 };
 
 }  // namespace safe_search_api

components/safe_search_api/url_checker.h

@@ -43,6 +43,9 @@ class URLChecker {
   URLChecker(std::unique_ptr<URLCheckerClient> async_checker,
              size_t cache_size);
 
+  URLChecker(const URLChecker&) = delete;
+  URLChecker& operator=(const URLChecker&) = delete;
+
   ~URLChecker();
 
   // Returns whether |callback| was run synchronously.
@@ -73,8 +76,6 @@ class URLChecker {
   base::TimeDelta cache_timeout_;
 
   base::WeakPtrFactory<URLChecker> weak_factory_{this};
-
-  DISALLOW_COPY_AND_ASSIGN(URLChecker);
 };
 
 }  // namespace safe_search_api