[Net] Add a priority parameter to URLRequest's constructor

components/navigation_interception/intercept_navigation_resource_throttle_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/run_loop.h"
 #include "base/synchronization/waitable_event.h"
 #include "components/navigation_interception/intercept_navigation_resource_throttle.h"
 #include "components/navigation_interception/navigation_params.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/resource_context.h"
 #include "content/public/browser/resource_controller.h"
 #include "content/public/browser/resource_dispatcher_host.h"
 #include "content/public/browser/resource_dispatcher_host_delegate.h"
 #include "content/public/browser/resource_request_info.h"
 #include "content/public/browser/resource_throttle.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_delegate.h"
 #include "content/public/common/page_transition_types.h"
 #include "content/public/test/mock_resource_context.h"
 #include "content/public/test/test_renderer_host.h"
+#include "net/base/request_priority.h"
 #include "net/http/http_response_headers.h"
 #include "net/http/http_response_info.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_test_util.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using testing::_;
 using testing::Eq;
 using testing::Ne;
@@ skipping 76 matching lines @@
 
 class TestIOThreadState {
  public:
   TestIOThreadState(const GURL& url,
                     int render_process_id,
                     int render_view_id,
                     const std::string& request_method,
                     RedirectMode redirect_mode,
                     MockInterceptCallbackReceiver* callback_receiver)
       : resource_context_(&test_url_request_context_),
-        request_(url, NULL, resource_context_.GetRequestContext()) {
+        request_(url,
+                 net::DEFAULT_PRIORITY,
+                 NULL,
+                 resource_context_.GetRequestContext(),
+                 NULL) {

[mmenke, 2013/10/30 15:27:11]

This depends on the resource_context_ having a NULL NetworkDelegate.  Maybe
switch to using resource_context_.GetRequestContext()->CreateRequest, and
sotring the request in a scoped_ptr?

[akalin, 2013/10/30 21:44:37]

removed arg.

     DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
     if (render_process_id != MSG_ROUTING_NONE &&
         render_view_id != MSG_ROUTING_NONE) {
       content::ResourceRequestInfo::AllocateForTesting(
           &request_,
           ResourceType::MAIN_FRAME,
           &resource_context_,
           render_process_id,
           render_view_id,
           false);
@@ skipping 331 matching lines @@
           REDIRECT_MODE_302,
           web_contents()->GetRenderViewHost()->GetProcess()->GetID(),
           web_contents()->GetRenderViewHost()->GetRoutingID(),
           base::Unretained(&defer)));
 
   // Wait for the request to finish processing.
   base::RunLoop().RunUntilIdle();
 }
 
 }  // namespace navigation_interception