Remove DISALLOW_* macros from net/
This inlines all remaining DISALLOW_* macros in net/. This is done
manually (vim regex + manually finding insertion position).
IWYU cleanup is left as a separate pass that is easier when these macros
go away.
Bug: 1010217
Change-Id: I7fade65cf1fd0f08e1c3baeff35dfbf3beb3ebce
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3202538
Commit-Queue: Peter Boström <[email protected]>
Reviewed-by: Lei Zhang <[email protected]>
Owners-Override: Lei Zhang <[email protected]>
Cr-Commit-Position: refs/heads/main@{#929037}
diff --git a/net/http/bidirectional_stream.h b/net/http/bidirectional_stream.h
index 6fcf388..0353e222 100644
--- a/net/http/bidirectional_stream.h
+++ b/net/http/bidirectional_stream.h
@@ -51,6 +51,9 @@
public:
Delegate();
+ Delegate(const Delegate&) = delete;
+ Delegate& operator=(const Delegate&) = delete;
+
// Called when the stream is ready for writing and reading. This is called
// at most once for the lifetime of a stream.
// The delegate may call BidirectionalStream::ReadData to start reading,
@@ -95,9 +98,6 @@
protected:
virtual ~Delegate();
-
- private:
- DISALLOW_COPY_AND_ASSIGN(Delegate);
};
// Constructs a BidirectionalStream. |request_info| contains information about
@@ -123,6 +123,9 @@
Delegate* delegate,
std::unique_ptr<base::OneShotTimer> timer);
+ BidirectionalStream(const BidirectionalStream&) = delete;
+ BidirectionalStream& operator=(const BidirectionalStream&) = delete;
+
// Cancels |stream_request_| or |stream_impl_| if applicable.
// |this| should not be destroyed during Delegate::OnHeadersSent or
// Delegate::OnDataSent.
@@ -260,8 +263,6 @@
LoadTimingInfo load_timing_info_;
base::WeakPtrFactory<BidirectionalStream> weak_factory_{this};
-
- DISALLOW_COPY_AND_ASSIGN(BidirectionalStream);
};
} // namespace net
