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

Side by Side Diff: chrome/browser/browsing_data/browsing_data_media_license_helper.cc

Issue 2826783003: Rewrite base::Bind to base::BindOnce with base_bind_rewriters in //chrome/browser/browsing_data (Closed)
Patch Set: Created 3 years, 8 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 "chrome/browser/browsing_data/browsing_data_media_license_helper.h" 5 #include "chrome/browser/browsing_data/browsing_data_media_license_helper.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <set> 8 #include <set>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 DCHECK(filesystem_context_.get()); 64 DCHECK(filesystem_context_.get());
65 } 65 }
66 66
67 BrowsingDataMediaLicenseHelperImpl::~BrowsingDataMediaLicenseHelperImpl() {} 67 BrowsingDataMediaLicenseHelperImpl::~BrowsingDataMediaLicenseHelperImpl() {}
68 68
69 void BrowsingDataMediaLicenseHelperImpl::StartFetching( 69 void BrowsingDataMediaLicenseHelperImpl::StartFetching(
70 const FetchCallback& callback) { 70 const FetchCallback& callback) {
71 DCHECK_CURRENTLY_ON(BrowserThread::UI); 71 DCHECK_CURRENTLY_ON(BrowserThread::UI);
72 DCHECK(!callback.is_null()); 72 DCHECK(!callback.is_null());
73 file_task_runner()->PostTask( 73 file_task_runner()->PostTask(
74 FROM_HERE, base::Bind(&BrowsingDataMediaLicenseHelperImpl:: 74 FROM_HERE, base::BindOnce(&BrowsingDataMediaLicenseHelperImpl::
75 FetchMediaLicenseInfoOnFileTaskRunner, 75 FetchMediaLicenseInfoOnFileTaskRunner,
76 this, callback)); 76 this, callback));
77 } 77 }
78 78
79 void BrowsingDataMediaLicenseHelperImpl::DeleteMediaLicenseOrigin( 79 void BrowsingDataMediaLicenseHelperImpl::DeleteMediaLicenseOrigin(
80 const GURL& origin) { 80 const GURL& origin) {
81 DCHECK_CURRENTLY_ON(BrowserThread::UI); 81 DCHECK_CURRENTLY_ON(BrowserThread::UI);
82 file_task_runner()->PostTask( 82 file_task_runner()->PostTask(
83 FROM_HERE, base::Bind(&BrowsingDataMediaLicenseHelperImpl:: 83 FROM_HERE, base::BindOnce(&BrowsingDataMediaLicenseHelperImpl::
84 DeleteMediaLicenseOriginOnFileTaskRunner, 84 DeleteMediaLicenseOriginOnFileTaskRunner,
85 this, origin)); 85 this, origin));
86 } 86 }
87 87
88 void BrowsingDataMediaLicenseHelperImpl::FetchMediaLicenseInfoOnFileTaskRunner( 88 void BrowsingDataMediaLicenseHelperImpl::FetchMediaLicenseInfoOnFileTaskRunner(
89 const FetchCallback& callback) { 89 const FetchCallback& callback) {
90 DCHECK(file_task_runner()->RunsTasksOnCurrentThread()); 90 DCHECK(file_task_runner()->RunsTasksOnCurrentThread());
91 DCHECK(!callback.is_null()); 91 DCHECK(!callback.is_null());
92 92
93 const storage::FileSystemType kType = storage::kFileSystemTypePluginPrivate; 93 const storage::FileSystemType kType = storage::kFileSystemTypePluginPrivate;
94 94
95 storage::PluginPrivateFileSystemBackend* backend = 95 storage::PluginPrivateFileSystemBackend* backend =
96 static_cast<storage::PluginPrivateFileSystemBackend*>( 96 static_cast<storage::PluginPrivateFileSystemBackend*>(
97 filesystem_context_->GetFileSystemBackend(kType)); 97 filesystem_context_->GetFileSystemBackend(kType));
98 98
99 // Determine the set of origins used. 99 // Determine the set of origins used.
100 std::set<GURL> origins; 100 std::set<GURL> origins;
101 std::list<MediaLicenseInfo> result; 101 std::list<MediaLicenseInfo> result;
102 backend->GetOriginsForTypeOnFileTaskRunner(kType, &origins); 102 backend->GetOriginsForTypeOnFileTaskRunner(kType, &origins);
103 for (const GURL& origin : origins) { 103 for (const GURL& origin : origins) {
104 if (!BrowsingDataHelper::HasWebScheme(origin)) 104 if (!BrowsingDataHelper::HasWebScheme(origin))
105 continue; // Non-websafe state is not considered browsing data. 105 continue; // Non-websafe state is not considered browsing data.
106 106
107 int64_t size; 107 int64_t size;
108 base::Time last_modified_time; 108 base::Time last_modified_time;
109 backend->GetOriginDetailsOnFileTaskRunner(filesystem_context_.get(), origin, 109 backend->GetOriginDetailsOnFileTaskRunner(filesystem_context_.get(), origin,
110 &size, &last_modified_time); 110 &size, &last_modified_time);
111 result.push_back(MediaLicenseInfo(origin, size, last_modified_time)); 111 result.push_back(MediaLicenseInfo(origin, size, last_modified_time));
112 } 112 }
113 113
114 BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, 114 BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
115 base::Bind(callback, result)); 115 base::BindOnce(callback, result));
116 } 116 }
117 117
118 void BrowsingDataMediaLicenseHelperImpl:: 118 void BrowsingDataMediaLicenseHelperImpl::
119 DeleteMediaLicenseOriginOnFileTaskRunner(const GURL& origin) { 119 DeleteMediaLicenseOriginOnFileTaskRunner(const GURL& origin) {
120 DCHECK(file_task_runner()->RunsTasksOnCurrentThread()); 120 DCHECK(file_task_runner()->RunsTasksOnCurrentThread());
121 121
122 const storage::FileSystemType kType = storage::kFileSystemTypePluginPrivate; 122 const storage::FileSystemType kType = storage::kFileSystemTypePluginPrivate;
123 storage::FileSystemBackend* backend = 123 storage::FileSystemBackend* backend =
124 filesystem_context_->GetFileSystemBackend(kType); 124 filesystem_context_->GetFileSystemBackend(kType);
125 storage::FileSystemQuotaUtil* quota_util = backend->GetQuotaUtil(); 125 storage::FileSystemQuotaUtil* quota_util = backend->GetQuotaUtil();
(...skipping 13 matching lines...) Expand all
139 BrowsingDataMediaLicenseHelper::MediaLicenseInfo::MediaLicenseInfo( 139 BrowsingDataMediaLicenseHelper::MediaLicenseInfo::MediaLicenseInfo(
140 const MediaLicenseInfo& other) = default; 140 const MediaLicenseInfo& other) = default;
141 141
142 BrowsingDataMediaLicenseHelper::MediaLicenseInfo::~MediaLicenseInfo() {} 142 BrowsingDataMediaLicenseHelper::MediaLicenseInfo::~MediaLicenseInfo() {}
143 143
144 // static 144 // static
145 BrowsingDataMediaLicenseHelper* BrowsingDataMediaLicenseHelper::Create( 145 BrowsingDataMediaLicenseHelper* BrowsingDataMediaLicenseHelper::Create(
146 storage::FileSystemContext* filesystem_context) { 146 storage::FileSystemContext* filesystem_context) {
147 return new BrowsingDataMediaLicenseHelperImpl(filesystem_context); 147 return new BrowsingDataMediaLicenseHelperImpl(filesystem_context);
148 } 148 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698