diff options
| author | svaldez <svaldez@chromium.org> | 2015-08-24 14:36:20 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-08-24 21:39:35 +0000 |
| commit | 5d58c9eb2baa203be1b84ac88cde82c59d72f143 (patch) | |
| tree | 3446bf04741d0b6e307fdc0b51593323834e680d /mojo/services | |
| parent | cc19cb9a644b1c385b36b108bd9bf5438d8d1f2b (diff) | |
| download | chromium_src-5d58c9eb2baa203be1b84ac88cde82c59d72f143.zip chromium_src-5d58c9eb2baa203be1b84ac88cde82c59d72f143.tar.gz chromium_src-5d58c9eb2baa203be1b84ac88cde82c59d72f143.tar.bz2 | |
Using scoped_ptr for URLRequestJobFactoryImpl::SetProtocolHandler
Adding scoped_ptr to call in order to take ownership of handler.
BUG=517161
Review URL: https://codereview.chromium.org/1295523006
Cr-Commit-Position: refs/heads/master@{#345185}
Diffstat (limited to 'mojo/services')
| -rw-r--r-- | mojo/services/network/url_loader_impl_apptest.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mojo/services/network/url_loader_impl_apptest.cc b/mojo/services/network/url_loader_impl_apptest.cc index 777889a..2fb2210 100644 --- a/mojo/services/network/url_loader_impl_apptest.cc +++ b/mojo/services/network/url_loader_impl_apptest.cc @@ -93,7 +93,6 @@ class TestProtocolHandler : public net::URLRequestJobFactory::ProtocolHandler { return new TestURLRequestJob(request, network_delegate); } - private: ~TestProtocolHandler() override {} }; @@ -112,7 +111,7 @@ class UrlLoaderImplTest : public test::ApplicationTestBase { scoped_ptr<net::TestURLRequestContext> url_request_context( new net::TestURLRequestContext(true)); ASSERT_TRUE(url_request_job_factory_.SetProtocolHandler( - "http", new TestProtocolHandler())); + "http", make_scoped_ptr(new TestProtocolHandler()))); url_request_context->set_job_factory(&url_request_job_factory_); url_request_context->Init(); network_context_.reset(new NetworkContext(url_request_context.Pass())); |