summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--mojo/system/message_in_transit.cc9
-rw-r--r--mojo/system/message_in_transit.h5
2 files changed, 7 insertions, 7 deletions
diff --git a/mojo/system/message_in_transit.cc b/mojo/system/message_in_transit.cc
index 5b6236b..a444a8e 100644
--- a/mojo/system/message_in_transit.cc
+++ b/mojo/system/message_in_transit.cc
@@ -48,12 +48,13 @@ STATIC_CONST_MEMBER_DEFINITION const size_t MessageInTransit::kMessageAlignment;
MessageInTransit::View::View(size_t message_size, const void* buffer)
- : message_size_(message_size),
- buffer_(buffer) {
+ : buffer_(buffer) {
size_t next_message_size = 0;
- DCHECK(MessageInTransit::GetNextMessageSize(buffer_, message_size_,
+ DCHECK(MessageInTransit::GetNextMessageSize(buffer_, message_size,
&next_message_size));
- DCHECK_EQ(message_size_, next_message_size);
+ DCHECK_EQ(message_size, next_message_size);
+ // This should be equivalent.
+ DCHECK_EQ(message_size, total_size());
}
MessageInTransit::MessageInTransit(Type type,
diff --git a/mojo/system/message_in_transit.h b/mojo/system/message_in_transit.h
index d8eb07d..66f374d 100644
--- a/mojo/system/message_in_transit.h
+++ b/mojo/system/message_in_transit.h
@@ -83,11 +83,11 @@ class MOJO_SYSTEM_IMPL_EXPORT MessageInTransit {
return RoundUpMessageAlignment(sizeof(Header) + header()->num_bytes);
}
const void* secondary_buffer() const {
- return (message_size_ > main_buffer_size()) ?
+ return (total_size() > main_buffer_size()) ?
static_cast<const char*>(buffer_) + main_buffer_size() : NULL;
}
size_t secondary_buffer_size() const {
- return message_size_ - main_buffer_size();
+ return total_size() - main_buffer_size();
}
size_t total_size() const { return header()->total_size; }
uint32_t num_bytes() const { return header()->num_bytes; }
@@ -103,7 +103,6 @@ class MOJO_SYSTEM_IMPL_EXPORT MessageInTransit {
private:
const Header* header() const { return static_cast<const Header*>(buffer_); }
- const size_t message_size_;
const void* const buffer_;
// Though this struct is trivial, disallow copy and assign, since it doesn't