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

Unified Diff: components/proximity_auth/wire_message.h

Issue 562763004: [EasyUnlock] Port PermitMessage class to native code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Left-shift by zero bits for parallelism Created 6 years, 3 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
« no previous file with comments | « components/proximity_auth/connection_unittest.cc ('k') | components/proximity_auth/wire_message.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/proximity_auth/wire_message.h
diff --git a/components/proximity_auth/wire_message.h b/components/proximity_auth/wire_message.h
index db5af37ec2b6261f86cd54fc05d3d620a0d94589..36107620f22bb585250c5196b0433d700b41f8ee 100644
--- a/components/proximity_auth/wire_message.h
+++ b/components/proximity_auth/wire_message.h
@@ -16,21 +16,29 @@ class WireMessage {
public:
virtual ~WireMessage();
- // Returns |true| iff the size of |message_bytes| is at least equal to the
- // message length encoded in the message header. Returns false if the message
- // header is not available.
- static bool IsCompleteMessage(const std::string& message_bytes);
-
// Returns the deserialized message from |serialized_message|, or NULL if the
- // message is malformed.
+ // message is malformed. Sets |is_incomplete_message| to true if the message
+ // does not have enough data to parse the header, or if the message length
+ // encoded in the message header exceeds the size of the |serialized_message|.
static scoped_ptr<WireMessage> Deserialize(
- const std::string& serialized_message);
+ const std::string& serialized_message,
+ bool* is_incomplete_message);
+
+ const std::string& permit_id() const { return permit_id_; }
+ const std::string& payload() const { return payload_; }
protected:
// Visible for tests.
- WireMessage();
+ WireMessage(const std::string& permit_id, const std::string& payload);
private:
+ // Identifier of the permit being used.
+ // TODO(isherman): Describe in a bit more detail.
+ const std::string permit_id_;
+
+ // The message payload.
+ const std::string payload_;
+
DISALLOW_COPY_AND_ASSIGN(WireMessage);
};
« no previous file with comments | « components/proximity_auth/connection_unittest.cc ('k') | components/proximity_auth/wire_message.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698