Using scoped_ptr for URLRequestJobFactoryImpl::SetProtocolHandler

net/url_request/url_request_simple_job_unittest.cc

 // Copyright 2014 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_helpers.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "base/threading/worker_pool.h"
@@ skipping 87 matching lines @@
       : task_runner_(task_runner) {}
   URLRequestJob* MaybeCreateJob(
       URLRequest* request,
       NetworkDelegate* network_delegate) const override {
     if (request->url().spec() == "data:empty")
       return new MockSimpleJob(request, network_delegate, task_runner_, "");
     return new MockSimpleJob(request, network_delegate, task_runner_,
                              kTestData);
   }
 
+  ~SimpleJobProtocolHandler() override {}
+
  private:
   scoped_refptr<base::TaskRunner> task_runner_;
-
-  ~SimpleJobProtocolHandler() override {}
 };
 
 class URLRequestSimpleJobTest : public ::testing::Test {
  public:
   URLRequestSimpleJobTest()
       : worker_pool_(
             new base::SequencedWorkerPool(1, "URLRequestSimpleJobTest")),
         task_runner_(worker_pool_->GetSequencedTaskRunnerWithShutdownBehavior(
             worker_pool_->GetSequenceToken(),
             base::SequencedWorkerPool::SKIP_ON_SHUTDOWN)),
         context_(true) {
-    job_factory_.SetProtocolHandler("data",
-                                    new SimpleJobProtocolHandler(task_runner_));
+    job_factory_.SetProtocolHandler(
+        "data", make_scoped_ptr(new SimpleJobProtocolHandler(task_runner_)));
     context_.set_job_factory(&job_factory_);
     context_.Init();
 
     request_ =
         context_.CreateRequest(GURL("data:test"), DEFAULT_PRIORITY, &delegate_);
   }
 
   ~URLRequestSimpleJobTest() override { worker_pool_->Shutdown(); }
 
   void StartRequest(const HttpRequestHeaders* headers) {
@@ skipping 102 matching lines @@
   run_loop.Run();
 
   EXPECT_EQ(URLRequestStatus::CANCELED, request_->status().status());
   EXPECT_EQ(1, cancel_delegate.response_started_count());
   EXPECT_EQ("", cancel_delegate.data_received());
   // Destroy the request so it doesn't outlive its delegate.
   request_.reset();
 }
 
 }  // namespace net