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

Side by Side Diff: content/browser/loader/resource_loader_unittest.cc

Issue 2741433002: Fix ResourceDispatcherHostDelegate::OnResponseStarted not getting called for navigation requests (Closed)
Patch Set: fix mojo unittests Created 3 years, 9 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 unified diff | Download patch
« no previous file with comments | « content/browser/loader/resource_loader_delegate.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/loader/resource_loader.h" 5 #include "content/browser/loader/resource_loader.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <deque> 10 #include <deque>
(...skipping 474 matching lines...) Expand 10 before | Expand all | Expand 10 after
485 } 485 }
486 void DidReceiveRedirect(ResourceLoader* loader, 486 void DidReceiveRedirect(ResourceLoader* loader,
487 const GURL& new_url, 487 const GURL& new_url,
488 ResourceResponse* response) override { 488 ResourceResponse* response) override {
489 EXPECT_EQ(loader, loader_.get()); 489 EXPECT_EQ(loader, loader_.get());
490 EXPECT_EQ(0, did_finish_loading_); 490 EXPECT_EQ(0, did_finish_loading_);
491 EXPECT_EQ(0, did_receive_response_); 491 EXPECT_EQ(0, did_receive_response_);
492 EXPECT_EQ(1, did_start_request_); 492 EXPECT_EQ(1, did_start_request_);
493 ++did_received_redirect_; 493 ++did_received_redirect_;
494 } 494 }
495 void DidReceiveResponse(ResourceLoader* loader) override { 495 void DidReceiveResponse(ResourceLoader* loader,
496 ResourceResponse* response) override {
496 EXPECT_EQ(loader, loader_.get()); 497 EXPECT_EQ(loader, loader_.get());
497 EXPECT_EQ(0, did_finish_loading_); 498 EXPECT_EQ(0, did_finish_loading_);
498 EXPECT_EQ(0, did_receive_response_); 499 EXPECT_EQ(0, did_receive_response_);
499 EXPECT_EQ(1, did_start_request_); 500 EXPECT_EQ(1, did_start_request_);
500 ++did_receive_response_; 501 ++did_receive_response_;
501 } 502 }
502 void DidFinishLoading(ResourceLoader* loader) override { 503 void DidFinishLoading(ResourceLoader* loader) override {
503 EXPECT_EQ(loader, loader_.get()); 504 EXPECT_EQ(loader, loader_.get());
504 EXPECT_EQ(0, did_finish_loading_); 505 EXPECT_EQ(0, did_finish_loading_);
505 506
(...skipping 977 matching lines...) Expand 10 before | Expand all | Expand 10 after
1483 1484
1484 // Tests that the effective connection type is not set on non-main frame 1485 // Tests that the effective connection type is not set on non-main frame
1485 // requests. 1486 // requests.
1486 TEST_F(EffectiveConnectionTypeResourceLoaderTest, DoesNotBelongToMainFrame) { 1487 TEST_F(EffectiveConnectionTypeResourceLoaderTest, DoesNotBelongToMainFrame) {
1487 VerifyEffectiveConnectionType(RESOURCE_TYPE_OBJECT, false, 1488 VerifyEffectiveConnectionType(RESOURCE_TYPE_OBJECT, false,
1488 net::EFFECTIVE_CONNECTION_TYPE_3G, 1489 net::EFFECTIVE_CONNECTION_TYPE_3G,
1489 net::EFFECTIVE_CONNECTION_TYPE_UNKNOWN); 1490 net::EFFECTIVE_CONNECTION_TYPE_UNKNOWN);
1490 } 1491 }
1491 1492
1492 } // namespace content 1493 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/loader/resource_loader_delegate.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698