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

android_webview/browser/net/android_stream_reader_url_request_job_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 "android_webview/browser/input_stream.h"
 #include "android_webview/browser/net/android_stream_reader_url_request_job.h"
 #include "android_webview/browser/net/aw_url_request_job_factory.h"
 #include "android_webview/browser/net/input_stream_reader.h"
 #include "base/format_macros.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
+#include "net/base/request_priority.h"
 #include "net/url_request/url_request_job_factory_impl.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 android_webview::InputStream;
 using android_webview::InputStreamReader;
+using net::DEFAULT_PRIORITY;

[joth, 2013/10/30 21:48:49]

nit: this seems excessive given it's used in one place and doesn't save any
vertical space by being able to omit the net:: where used.

(But hey it's only test code; I'm not strongly opinionated on this)

[akalin, 2013/10/30 23:25:59]

Done.

 using net::TestDelegate;
 using net::TestJobInterceptor;
 using net::TestNetworkDelegate;
 using net::TestURLRequestContext;
 using net::TestURLRequest;
 using testing::DoAll;
 using testing::Ge;
 using testing::Gt;
 using testing::InSequence;
 using testing::Invoke;
@@ skipping 113 matching lines @@
 };
 
 class AndroidStreamReaderURLRequestJobTest : public Test {
  public:
   AndroidStreamReaderURLRequestJobTest() : loop_(base::MessageLoop::TYPE_IO) {}
 
  protected:
   virtual void SetUp() {
     context_.set_job_factory(&factory_);
     context_.set_network_delegate(&network_delegate_);
-    req_.reset(
-        new TestURLRequest(GURL("content://foo"),
-                           &url_request_delegate_,
-                           &context_,
-                           &network_delegate_));
+    req_.reset(new TestURLRequest(GURL("content://foo"),
+                                  DEFAULT_PRIORITY,
+                                  &url_request_delegate_,
+                                  &context_));
     req_->set_method("GET");
   }
 
   void SetRange(net::URLRequest* req, int first_byte, int last_byte) {
     net::HttpRequestHeaders headers;
     headers.SetHeader(net::HttpRequestHeaders::kRange,
                       base::StringPrintf(
                            "bytes=%" PRIuS "-%" PRIuS,
                            first_byte, last_byte));
     req->SetExtraRequestHeaders(headers);
@@ skipping 179 matching lines @@
 
   SetRange(req_.get(), offset, bytes_available);
   req_->Start();
 
   loop.Run();
 
   EXPECT_EQ(0, network_delegate_.completed_requests());
   req_->Cancel();
   EXPECT_EQ(1, network_delegate_.completed_requests());
 }