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

chrome/browser/net/chrome_fraudulent_certificate_reporter_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 "chrome/browser/net/chrome_fraudulent_certificate_reporter.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/synchronization/waitable_event.h"
 #include "base/threading/thread.h"
 #include "chrome/browser/net/chrome_url_request_context.h"
 #include "content/public/test/test_browser_thread.h"
+#include "net/base/request_priority.h"
 #include "net/base/test_data_directory.h"
 #include "net/cert/x509_certificate.h"
 #include "net/http/transport_security_state.h"
 #include "net/ssl/ssl_info.h"
 #include "net/test/cert_test_util.h"
 #include "net/url_request/fraudulent_certificate_reporter.h"
 #include "net/url_request/url_request.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using content::BrowserThread;
@@ skipping 89 matching lines @@
 
 // For the first version of the feature, sending reports is "fire and forget".
 // Therefore, we test only that the Reporter tried to send a request at all.
 // In the future, when we have more sophisticated (i.e., any) error handling
 // and re-tries, we will need more sopisticated tests as well.
 //
 // This class doesn't do anything now, but in near future versions it will.
 class MockURLRequest : public net::URLRequest {
  public:
   explicit MockURLRequest(net::URLRequestContext* context)
-      : net::URLRequest(GURL(std::string()), NULL, context) {}
+      : net::URLRequest(GURL(std::string()),
+                        net::DEFAULT_PRIORITY,
+                        NULL,
+                        context) {}
 
  private:
 };
 
 // A ChromeFraudulentCertificateReporter that uses a MockURLRequest, but is
 // otherwise normal: reports are constructed and sent in the usual way.
 class MockReporter : public ChromeFraudulentCertificateReporter {
  public:
   explicit MockReporter(net::URLRequestContext* request_context)
     : ChromeFraudulentCertificateReporter(request_context) {}
 
-  virtual net::URLRequest* CreateURLRequest(
+  virtual scoped_ptr<net::URLRequest> CreateURLRequest(
       net::URLRequestContext* context) OVERRIDE {
-    return new MockURLRequest(context);
+    return scoped_ptr<net::URLRequest>(new MockURLRequest(context));
   }
 
   virtual void SendReport(
       const std::string& hostname,
       const net::SSLInfo& ssl_info,
       bool sni_available) OVERRIDE {
     DCHECK(!hostname.empty());
     DCHECK(ssl_info.is_valid());
     ChromeFraudulentCertificateReporter::SendReport(hostname, ssl_info,
                                                     sni_available);
@@ skipping 47 matching lines @@
 }
 
 TEST(ChromeFraudulentCertificateReporterTest, ReportIsNotSent) {
   base::MessageLoop loop(base::MessageLoop::TYPE_IO);
   content::TestBrowserThread io_thread(BrowserThread::IO, &loop);
   loop.PostTask(FROM_HERE, base::Bind(&DoReportIsNotSent));
   loop.RunUntilIdle();
 }
 
 }  // namespace chrome_browser_net