Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(165)

Unified Diff: content/public/common/presentation_connection_message.h

Issue 2947403004: [MediaRouter] Replace RouteMessage with PresentationConnectionMessage (Closed)
Patch Set: Addressed Takumi's comments Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/common/presentation_connection_message.h
diff --git a/content/public/common/presentation_connection_message.h b/content/public/common/presentation_connection_message.h
index d9130c13860f1f0ed32a54a23a2eda036669f004..91c15d2bdffd003315e753f8bbf3d7fb3f77d639 100644
--- a/content/public/common/presentation_connection_message.h
+++ b/content/public/common/presentation_connection_message.h
@@ -26,20 +26,25 @@ struct CONTENT_EXPORT PresentationConnectionMessage {
// Constructs a new, untyped message (for Mojo). These messages are not valid
// and exactly one of |message| or |data| must be set.
PresentationConnectionMessage();
- // PCM is a move-only type.
- PresentationConnectionMessage(PresentationConnectionMessage&& other);
+ PresentationConnectionMessage(const PresentationConnectionMessage& other);
+ PresentationConnectionMessage(PresentationConnectionMessage&& other) noexcept;
// Constructs a text message from |message|.
explicit PresentationConnectionMessage(std::string message);
// Constructs a binary message from |data|.
explicit PresentationConnectionMessage(std::vector<uint8_t> data);
+
~PresentationConnectionMessage();
bool is_binary() const;
bool operator==(const PresentationConnectionMessage& other) const;
+ PresentationConnectionMessage& operator=(
+ const PresentationConnectionMessage& other);
PresentationConnectionMessage& operator=(
PresentationConnectionMessage&& other);
+ std::string ToHumanReadableString() const;
+
base::Optional<std::string> message;
base::Optional<std::vector<uint8_t>> data;
};

Powered by Google App Engine
This is Rietveld 408576698