diff options
author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-06 19:24:05 +0000 |
---|---|---|
committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-06 19:24:05 +0000 |
commit | f465319aff0eac96b84b4f4754738b97a1c2dfed (patch) | |
tree | d7d4b376f8b808974001fa1aacf3ba6f28afd705 /content/browser/loader/resource_dispatcher_host_unittest.cc | |
parent | b926d6a87c7f9438d9c2b6a6eca638b52427737f (diff) | |
download | chromium_src-f465319aff0eac96b84b4f4754738b97a1c2dfed.zip chromium_src-f465319aff0eac96b84b4f4754738b97a1c2dfed.tar.gz chromium_src-f465319aff0eac96b84b4f4754738b97a1c2dfed.tar.bz2 |
Convert a bunch of resource IPCs to be control messages. They didn't need to be routed.
BUG=286074
R=ajwong@chromium.org
Review URL: https://codereview.chromium.org/23583039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221751 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/loader/resource_dispatcher_host_unittest.cc')
-rw-r--r-- | content/browser/loader/resource_dispatcher_host_unittest.cc | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/content/browser/loader/resource_dispatcher_host_unittest.cc b/content/browser/loader/resource_dispatcher_host_unittest.cc index be46339..3818bed 100644 --- a/content/browser/loader/resource_dispatcher_host_unittest.cc +++ b/content/browser/loader/resource_dispatcher_host_unittest.cc @@ -708,7 +708,7 @@ class ResourceDispatcherHostTest : public testing::Test, bool result = PickleIterator(msg).ReadInt(&request_id); DCHECK(result); scoped_ptr<IPC::Message> ack( - new ResourceHostMsg_DataReceived_ACK(msg.routing_id(), request_id)); + new ResourceHostMsg_DataReceived_ACK(request_id)); base::MessageLoop::current()->PostTask( FROM_HERE, @@ -1609,7 +1609,7 @@ TEST_F(ResourceDispatcherHostTest, IgnoreCancelForDownloads) { EXPECT_TRUE(net::URLRequestTestJob::ProcessOnePendingMessage()); // And now simulate a cancellation coming from the renderer. - ResourceHostMsg_CancelRequest msg(filter_->child_id(), request_id); + ResourceHostMsg_CancelRequest msg(request_id); bool msg_was_ok; host_.OnMessageReceived(msg, filter_.get(), &msg_was_ok); @@ -1644,7 +1644,7 @@ TEST_F(ResourceDispatcherHostTest, CancelRequestsForContext) { EXPECT_TRUE(net::URLRequestTestJob::ProcessOnePendingMessage()); // And now simulate a cancellation coming from the renderer. - ResourceHostMsg_CancelRequest msg(filter_->child_id(), request_id); + ResourceHostMsg_CancelRequest msg(request_id); bool msg_was_ok; host_.OnMessageReceived(msg, filter_.get(), &msg_was_ok); @@ -1685,7 +1685,7 @@ TEST_F(ResourceDispatcherHostTest, CancelRequestsForContextTransferred) { GURL("http://example.com/blah")); // And now simulate a cancellation coming from the renderer. - ResourceHostMsg_CancelRequest msg(filter_->child_id(), request_id); + ResourceHostMsg_CancelRequest msg(request_id); bool msg_was_ok; host_.OnMessageReceived(msg, filter_.get(), &msg_was_ok); @@ -1832,7 +1832,7 @@ TEST_F(ResourceDispatcherHostTest, TransferNavigationAndThenRedirect) { // Now, simulate the renderer choosing to follow the redirect. ResourceHostMsg_FollowRedirect redirect_msg( - new_render_view_id, new_request_id, false, GURL()); + new_request_id, false, GURL()); host_.OnMessageReceived(redirect_msg, second_filter.get(), &msg_was_ok); base::MessageLoop::current()->RunUntilIdle(); @@ -1938,7 +1938,7 @@ TEST_F(ResourceDispatcherHostTest, DelayedDataReceivedACKs) { EXPECT_EQ(ResourceMsg_DataReceived::ID, msgs[0][i].type()); - ResourceHostMsg_DataReceived_ACK msg(0, 1); + ResourceHostMsg_DataReceived_ACK msg(1); bool msg_was_ok; host_.OnMessageReceived(msg, filter_.get(), &msg_was_ok); } @@ -1973,7 +1973,7 @@ TEST_F(ResourceDispatcherHostTest, DataReceivedUnexpectedACKs) { // Send some unexpected ACKs. for (size_t i = 0; i < 128; ++i) { - ResourceHostMsg_DataReceived_ACK msg(0, 1); + ResourceHostMsg_DataReceived_ACK msg(1); bool msg_was_ok; host_.OnMessageReceived(msg, filter_.get(), &msg_was_ok); } @@ -1992,7 +1992,7 @@ TEST_F(ResourceDispatcherHostTest, DataReceivedUnexpectedACKs) { EXPECT_EQ(ResourceMsg_DataReceived::ID, msgs[0][i].type()); - ResourceHostMsg_DataReceived_ACK msg(0, 1); + ResourceHostMsg_DataReceived_ACK msg(1); bool msg_was_ok; host_.OnMessageReceived(msg, filter_.get(), &msg_was_ok); } |