| Index: chrome/browser/browsing_data_local_storage_helper.cc
|
| diff --git a/chrome/browser/browsing_data_local_storage_helper.cc b/chrome/browser/browsing_data_local_storage_helper.cc
|
| index a270f211b674b6202d185e8819ec6d9bce159362..8b84fc2c2e646905bb7538c1ee279bca44d69d21 100644
|
| --- a/chrome/browser/browsing_data_local_storage_helper.cc
|
| +++ b/chrome/browser/browsing_data_local_storage_helper.cc
|
| @@ -48,7 +48,6 @@ BrowsingDataLocalStorageHelper::LocalStorageInfo::~LocalStorageInfo() {}
|
| BrowsingDataLocalStorageHelper::BrowsingDataLocalStorageHelper(
|
| Profile* profile)
|
| : profile_(profile),
|
| - completion_callback_(NULL),
|
| is_fetching_(false) {
|
| DCHECK(profile_);
|
| }
|
| @@ -57,12 +56,13 @@ BrowsingDataLocalStorageHelper::~BrowsingDataLocalStorageHelper() {
|
| }
|
|
|
| void BrowsingDataLocalStorageHelper::StartFetching(
|
| - Callback1<const std::list<LocalStorageInfo>& >::Type* callback) {
|
| + const base::Callback<void(const std::list<LocalStorageInfo>&)>& callback) {
|
| DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
|
| DCHECK(!is_fetching_);
|
| - DCHECK(callback);
|
| + DCHECK_EQ(false, callback.is_null());
|
| +
|
| is_fetching_ = true;
|
| - completion_callback_.reset(callback);
|
| + completion_callback_ = callback;
|
| BrowserThread::PostTask(
|
| BrowserThread::WEBKIT, FROM_HERE,
|
| base::Bind(
|
| @@ -72,7 +72,7 @@ void BrowsingDataLocalStorageHelper::StartFetching(
|
|
|
| void BrowsingDataLocalStorageHelper::CancelNotification() {
|
| DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
|
| - completion_callback_.reset(NULL);
|
| + completion_callback_.Reset();
|
| }
|
|
|
| void BrowsingDataLocalStorageHelper::DeleteLocalStorageFile(
|
| @@ -128,9 +128,9 @@ void BrowsingDataLocalStorageHelper::NotifyInUIThread() {
|
| DCHECK(is_fetching_);
|
| // Note: completion_callback_ mutates only in the UI thread, so it's safe to
|
| // test it here.
|
| - if (completion_callback_ != NULL) {
|
| - completion_callback_->Run(local_storage_info_);
|
| - completion_callback_.reset();
|
| + if (!completion_callback_.is_null()) {
|
| + completion_callback_.Run(local_storage_info_);
|
| + completion_callback_.Reset();
|
| }
|
| is_fetching_ = false;
|
| }
|
| @@ -178,12 +178,13 @@ bool CannedBrowsingDataLocalStorageHelper::empty() const {
|
| }
|
|
|
| void CannedBrowsingDataLocalStorageHelper::StartFetching(
|
| - Callback1<const std::list<LocalStorageInfo>& >::Type* callback) {
|
| + const base::Callback<void(const std::list<LocalStorageInfo>&)>& callback) {
|
| DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
|
| DCHECK(!is_fetching_);
|
| - DCHECK(callback);
|
| + DCHECK_EQ(false, callback.is_null());
|
| +
|
| is_fetching_ = true;
|
| - completion_callback_.reset(callback);
|
| + completion_callback_ = callback;
|
| BrowserThread::PostTask(
|
| BrowserThread::WEBKIT, FROM_HERE,
|
| base::Bind(&CannedBrowsingDataLocalStorageHelper::
|
|
|