Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(569)

Side by Side Diff: extensions/browser/api/mime_handler_private/mime_handler_private.cc

Issue 2851173004: Eliminate bind callback that doesn't take a BindSourceInfo parameter. (Closed)
Patch Set: . Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "extensions/browser/api/mime_handler_private/mime_handler_private.h" 5 #include "extensions/browser/api/mime_handler_private/mime_handler_private.h"
6 6
7 #include <unordered_map> 7 #include <unordered_map>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 51
52 MimeHandlerServiceImpl::MimeHandlerServiceImpl( 52 MimeHandlerServiceImpl::MimeHandlerServiceImpl(
53 base::WeakPtr<StreamContainer> stream_container) 53 base::WeakPtr<StreamContainer> stream_container)
54 : stream_(stream_container), weak_factory_(this) {} 54 : stream_(stream_container), weak_factory_(this) {}
55 55
56 MimeHandlerServiceImpl::~MimeHandlerServiceImpl() {} 56 MimeHandlerServiceImpl::~MimeHandlerServiceImpl() {}
57 57
58 // static 58 // static
59 void MimeHandlerServiceImpl::Create( 59 void MimeHandlerServiceImpl::Create(
60 base::WeakPtr<StreamContainer> stream_container, 60 base::WeakPtr<StreamContainer> stream_container,
61 const service_manager::BindSourceInfo& source_info,
61 mime_handler::MimeHandlerServiceRequest request) { 62 mime_handler::MimeHandlerServiceRequest request) {
62 mojo::MakeStrongBinding( 63 mojo::MakeStrongBinding(
63 base::MakeUnique<MimeHandlerServiceImpl>(stream_container), 64 base::MakeUnique<MimeHandlerServiceImpl>(stream_container),
64 std::move(request)); 65 std::move(request));
65 } 66 }
66 67
67 void MimeHandlerServiceImpl::GetStreamInfo( 68 void MimeHandlerServiceImpl::GetStreamInfo(
68 const GetStreamInfoCallback& callback) { 69 const GetStreamInfoCallback& callback) {
69 if (!stream_) { 70 if (!stream_) {
70 callback.Run(mime_handler::StreamInfoPtr()); 71 callback.Run(mime_handler::StreamInfoPtr());
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 result->original_url = info->original_url.spec(); 116 result->original_url = info->original_url.spec();
116 } 117 }
117 118
118 result->stream_url = info->handle->GetURL().spec(); 119 result->stream_url = info->handle->GetURL().spec();
119 result->response_headers = 120 result->response_headers =
120 extensions::CreateResponseHeadersMap(info->response_headers.get()); 121 extensions::CreateResponseHeadersMap(info->response_headers.get());
121 return result; 122 return result;
122 } 123 }
123 124
124 } // namespace mojo 125 } // namespace mojo
OLDNEW
« no previous file with comments | « extensions/browser/api/mime_handler_private/mime_handler_private.h ('k') | extensions/browser/mojo/keep_alive_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698