From 9ea0ecd0243f27205f9e6cf0ad2e554cc180e4cf Mon Sep 17 00:00:00 2001 From: "bbudge@chromium.org" Date: Mon, 28 Oct 2013 15:30:04 +0000 Subject: Remove unused IPC::Message priority. Removes the PriorityValue enum and field from IPC::Message. This doesn't appear to be used anywhere. Changes the data message ctor to take a size_t data_len parameter. This works around an ambiguity problem with the main ctor, which has a similar signature and would require lots of futzing with our test code to fix. To make this work, the matching Pickle constructor is also changed to take a size_t data_len parameter. BUG=194304 Review URL: https://codereview.chromium.org/35643005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@231330 0039d316-1c4b-4281-b951-d872f2087c98 --- ipc/ipc_message_utils_unittest.cc | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'ipc/ipc_message_utils_unittest.cc') diff --git a/ipc/ipc_message_utils_unittest.cc b/ipc/ipc_message_utils_unittest.cc index 2156eeb..9df1bef 100644 --- a/ipc/ipc_message_utils_unittest.cc +++ b/ipc/ipc_message_utils_unittest.cc @@ -16,16 +16,14 @@ TEST(IPCMessageUtilsTest, NestedMessages) { int32 nested_routing = 12; uint32 nested_type = 78; int nested_content = 456789; - Message::PriorityValue nested_priority = Message::PRIORITY_HIGH; - Message nested_msg(nested_routing, nested_type, nested_priority); + Message nested_msg(nested_routing, nested_type); nested_msg.set_sync(); ParamTraits::Write(&nested_msg, nested_content); // Outer message contains the nested one as its parameter. int32 outer_routing = 91; uint32 outer_type = 88; - Message::PriorityValue outer_priority = Message::PRIORITY_NORMAL; - Message outer_msg(outer_routing, outer_type, outer_priority); + Message outer_msg(outer_routing, outer_type); ParamTraits::Write(&outer_msg, nested_msg); // Read back the nested message. @@ -36,7 +34,6 @@ TEST(IPCMessageUtilsTest, NestedMessages) { // Verify nested message headers. EXPECT_EQ(nested_msg.routing_id(), result_msg.routing_id()); EXPECT_EQ(nested_msg.type(), result_msg.type()); - EXPECT_EQ(nested_msg.priority(), result_msg.priority()); EXPECT_EQ(nested_msg.flags(), result_msg.flags()); // Verify nested message content -- cgit v1.1