diff options
| author | bbudge@chromium.org <bbudge@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-28 15:30:04 +0000 |
|---|---|---|
| committer | bbudge@chromium.org <bbudge@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-28 15:30:04 +0000 |
| commit | 9ea0ecd0243f27205f9e6cf0ad2e554cc180e4cf (patch) | |
| tree | 37acb4b88e0631fd7c70f43675f72854f5429bfe /ppapi/host | |
| parent | 85372b02a7dfb87c35896c463296be5cdab305cb (diff) | |
| download | chromium_src-9ea0ecd0243f27205f9e6cf0ad2e554cc180e4cf.zip chromium_src-9ea0ecd0243f27205f9e6cf0ad2e554cc180e4cf.tar.gz chromium_src-9ea0ecd0243f27205f9e6cf0ad2e554cc180e4cf.tar.bz2 | |
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
Diffstat (limited to 'ppapi/host')
| -rw-r--r-- | ppapi/host/resource_message_filter_unittest.cc | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/ppapi/host/resource_message_filter_unittest.cc b/ppapi/host/resource_message_filter_unittest.cc index 78fc23c..38fd9d8f 100644 --- a/ppapi/host/resource_message_filter_unittest.cc +++ b/ppapi/host/resource_message_filter_unittest.cc @@ -64,8 +64,7 @@ class MyResourceHost : public ResourceHost { HostMessageContext* context) OVERRIDE { last_handled_msg_ = msg; if (msg.type() == msg_type_) { - context->reply_msg = IPC::Message(0, reply_msg_type_, - IPC::Message::PRIORITY_NORMAL); + context->reply_msg = IPC::Message(0, reply_msg_type_); return PP_OK; } return PP_ERROR_FAILED; @@ -123,8 +122,7 @@ class MyResourceFilter : public ResourceMessageFilter { last_handled_msg_ = msg; last_message_loop_ = base::MessageLoop::current(); if (msg.type() == msg_type_) { - context->reply_msg = IPC::Message(0, reply_msg_type_, - IPC::Message::PRIORITY_NORMAL); + context->reply_msg = IPC::Message(0, reply_msg_type_); return PP_OK; } return PP_ERROR_FAILED; @@ -166,9 +164,9 @@ TEST_F(ResourceMessageFilterTest, TestHandleMessage) { proxy::ResourceMessageCallParams params(resource, 1); params.set_has_callback(); HostMessageContext context(params); - IPC::Message message1(0, MSG1_TYPE, IPC::Message::PRIORITY_NORMAL); - IPC::Message message2(0, MSG2_TYPE, IPC::Message::PRIORITY_NORMAL); - IPC::Message message3(0, MSG3_TYPE, IPC::Message::PRIORITY_NORMAL); + IPC::Message message1(0, MSG1_TYPE); + IPC::Message message2(0, MSG2_TYPE); + IPC::Message message3(0, MSG3_TYPE); // Message 1 handled by the first filter. host.HandleMessage(message1, &context); |