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

Unified Diff: chrome/browser/prerender/prerender_tracker_unittest.cc

Issue 51953002: [Net] Add a priority parameter to URLRequest's constructor (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 7 years, 2 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
« no previous file with comments | « chrome/browser/predictors/resource_prefetcher.cc ('k') | chrome/browser/search/iframe_source_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_tracker_unittest.cc
diff --git a/chrome/browser/prerender/prerender_tracker_unittest.cc b/chrome/browser/prerender/prerender_tracker_unittest.cc
index 2ee44da3fcd2525ab7a3a3cdf3f96f56279585f9..b96159ab9bd90feb97c756ad3683bde1fb18d853 100644
--- a/chrome/browser/prerender/prerender_tracker_unittest.cc
+++ b/chrome/browser/prerender/prerender_tracker_unittest.cc
@@ -20,6 +20,7 @@
#include "content/public/browser/resource_request_info.h"
#include "content/public/test/test_browser_thread.h"
#include "content/test/net/url_request_mock_http_job.h"
+#include "net/base/request_priority.h"
#include "net/url_request/url_request.h"
#include "net/url_request/url_request_test_util.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -527,9 +528,10 @@ TEST_F(PrerenderTrackerTest, PrerenderThrottledRedirectResume) {
net::TestURLRequestContext url_request_context;
DeferredRedirectDelegate delegate;
net::URLRequest request(
- content::URLRequestMockHTTPJob::GetMockUrl(
- base::FilePath(kRedirectPath)),
- &delegate, &url_request_context);
+ content::URLRequestMockHTTPJob::GetMockUrl(base::FilePath(kRedirectPath)),
+ net::DEFAULT_PRIORITY,
+ &delegate,
+ &url_request_context);
content::ResourceRequestInfo::AllocateForTesting(
&request, ResourceType::IMAGE, NULL,
kDefaultChildId, kDefaultRouteId, true);
@@ -566,9 +568,10 @@ TEST_F(PrerenderTrackerTest, PrerenderThrottledRedirectCancel) {
net::TestURLRequestContext url_request_context;
DeferredRedirectDelegate delegate;
net::URLRequest request(
- content::URLRequestMockHTTPJob::GetMockUrl(
- base::FilePath(kRedirectPath)),
- &delegate, &url_request_context);
+ content::URLRequestMockHTTPJob::GetMockUrl(base::FilePath(kRedirectPath)),
+ net::DEFAULT_PRIORITY,
+ &delegate,
+ &url_request_context);
content::ResourceRequestInfo::AllocateForTesting(
&request, ResourceType::IMAGE, NULL,
kDefaultChildId, kDefaultRouteId, true);
@@ -605,9 +608,10 @@ TEST_F(PrerenderTrackerTest, PrerenderThrottledRedirectMainFrame) {
net::TestURLRequestContext url_request_context;
DeferredRedirectDelegate delegate;
net::URLRequest request(
- content::URLRequestMockHTTPJob::GetMockUrl(
- base::FilePath(kRedirectPath)),
- &delegate, &url_request_context);
+ content::URLRequestMockHTTPJob::GetMockUrl(base::FilePath(kRedirectPath)),
+ net::DEFAULT_PRIORITY,
+ &delegate,
+ &url_request_context);
content::ResourceRequestInfo::AllocateForTesting(
&request, ResourceType::MAIN_FRAME, NULL,
kDefaultChildId, kDefaultRouteId, true);
@@ -643,9 +647,10 @@ TEST_F(PrerenderTrackerTest, PrerenderThrottledRedirectSyncXHR) {
net::TestURLRequestContext url_request_context;
DeferredRedirectDelegate delegate;
net::URLRequest request(
- content::URLRequestMockHTTPJob::GetMockUrl(
- base::FilePath(kRedirectPath)),
- &delegate, &url_request_context);
+ content::URLRequestMockHTTPJob::GetMockUrl(base::FilePath(kRedirectPath)),
+ net::DEFAULT_PRIORITY,
+ &delegate,
+ &url_request_context);
content::ResourceRequestInfo::AllocateForTesting(
&request, ResourceType::XHR, NULL,
kDefaultChildId, kDefaultRouteId, false);
« no previous file with comments | « chrome/browser/predictors/resource_prefetcher.cc ('k') | chrome/browser/search/iframe_source_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698