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

Side by Side Diff: content/browser/loader/downloaded_temp_file_impl.cc

Issue 2658583002: Revert of Use associated interface on DownloadedTempFile (Closed)
Patch Set: Rebased. Created 3 years, 11 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "content/browser/loader/downloaded_temp_file_impl.h" 5 #include "content/browser/loader/downloaded_temp_file_impl.h"
6 6
7 #include "base/memory/ptr_util.h" 7 #include "base/memory/ptr_util.h"
8 #include "content/browser/loader/resource_dispatcher_host_impl.h" 8 #include "content/browser/loader/resource_dispatcher_host_impl.h"
9 #include "mojo/public/cpp/bindings/associated_group.h"
10 #include "mojo/public/cpp/bindings/strong_associated_binding.h"
11 #include "mojo/public/cpp/bindings/strong_binding.h" 9 #include "mojo/public/cpp/bindings/strong_binding.h"
12 10
13 namespace content { 11 namespace content {
14 12
15 // static 13 // static
16 mojom::DownloadedTempFileAssociatedPtrInfo DownloadedTempFileImpl::Create( 14 mojo::InterfacePtr<mojom::DownloadedTempFile> DownloadedTempFileImpl::Create(
17 mojo::AssociatedGroup* associated_group,
18 int child_id,
19 int request_id) {
20 mojo::AssociatedInterfacePtrInfo<mojom::DownloadedTempFile> ptr_info;
21 mojo::AssociatedInterfaceRequest<mojom::DownloadedTempFile> request;
22 associated_group->CreateAssociatedInterface(
23 mojo::AssociatedGroup::WILL_PASS_PTR, &ptr_info, &request);
24 mojo::MakeStrongAssociatedBinding(
25 base::MakeUnique<DownloadedTempFileImpl>(child_id, request_id),
26 std::move(request));
27 return ptr_info;
28 }
29
30 // static
31 mojom::DownloadedTempFilePtr DownloadedTempFileImpl::CreateForTesting(
32 int child_id, 15 int child_id,
33 int request_id) { 16 int request_id) {
34 mojo::InterfacePtr<mojom::DownloadedTempFile> ptr; 17 mojo::InterfacePtr<mojom::DownloadedTempFile> ptr;
35 mojo::MakeStrongBinding( 18 auto binding = mojo::MakeStrongBinding(
36 base::MakeUnique<DownloadedTempFileImpl>(child_id, request_id), 19 base::MakeUnique<DownloadedTempFileImpl>(child_id, request_id),
37 mojo::MakeRequest(&ptr)); 20 mojo::MakeRequest(&ptr));
38 return ptr; 21 return ptr;
39 } 22 }
40 23
41 DownloadedTempFileImpl::~DownloadedTempFileImpl() { 24 DownloadedTempFileImpl::~DownloadedTempFileImpl() {
42 ResourceDispatcherHostImpl::Get()->UnregisterDownloadedTempFile(child_id_, 25 ResourceDispatcherHostImpl::Get()->UnregisterDownloadedTempFile(child_id_,
43 request_id_); 26 request_id_);
44 } 27 }
45 DownloadedTempFileImpl::DownloadedTempFileImpl(int child_id, int request_id) 28 DownloadedTempFileImpl::DownloadedTempFileImpl(int child_id, int request_id)
46 : child_id_(child_id), request_id_(request_id) {} 29 : child_id_(child_id), request_id_(request_id) {}
47 30
48 } // namespace content 31 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/loader/downloaded_temp_file_impl.h ('k') | content/browser/loader/mojo_async_resource_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698