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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: dbus/message_unittest.cc

Issue 176693003: chromeos: Make dbus::MessageReader memory ownership explicit (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update a few more callers Created 6 years, 10 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
« chromeos/dbus/debug_daemon_client.cc ('K') | « dbus/message.cc ('k') | device/media_transfer_protocol/media_transfer_protocol_daemon_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dbus/message_unittest.cc
diff --git a/dbus/message_unittest.cc b/dbus/message_unittest.cc
index 16348df5826ae0b10347033e0c3d5b7f4c112f24..0c944d9d6171fd7a04fc8a7e18b22b9b3e9eaa26 100644
--- a/dbus/message_unittest.cc
+++ b/dbus/message_unittest.cc
@@ -208,7 +208,7 @@ TEST(MessageTest, ArrayOfBytes) {
writer.AppendArrayOfBytes(bytes.data(), bytes.size());
MessageReader reader(message.get());
- uint8* output_bytes = NULL;
+ const uint8* output_bytes = NULL;
size_t length = 0;
ASSERT_TRUE(reader.PopArrayOfBytes(&output_bytes, &length));
ASSERT_FALSE(reader.HasMoreData());
@@ -225,7 +225,7 @@ TEST(MessageTest, ArrayOfBytes_Empty) {
writer.AppendArrayOfBytes(bytes.data(), bytes.size());
MessageReader reader(message.get());
- uint8* output_bytes = NULL;
+ const uint8* output_bytes = NULL;
size_t length = 0;
ASSERT_TRUE(reader.PopArrayOfBytes(&output_bytes, &length));
ASSERT_FALSE(reader.HasMoreData());
« chromeos/dbus/debug_daemon_client.cc ('K') | « dbus/message.cc ('k') | device/media_transfer_protocol/media_transfer_protocol_daemon_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698