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

Unified Diff: chrome/browser/chromeos/file_manager/zip_file_creator.cc

Issue 2730583002: Convert utility process zip creator IPC to mojo: part 2 (Closed)
Patch Set: Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/file_manager/zip_file_creator.cc
diff --git a/chrome/browser/chromeos/file_manager/zip_file_creator.cc b/chrome/browser/chromeos/file_manager/zip_file_creator.cc
index 19887d92d8d342c368e5c876248bf742d61003cd..0cbfc5994c7bcd82eb5bf8c21da7e19335d6fb1a 100644
--- a/chrome/browser/chromeos/file_manager/zip_file_creator.cc
+++ b/chrome/browser/chromeos/file_manager/zip_file_creator.cc
@@ -75,12 +75,8 @@ void ZipFileCreator::CreateZipFile(base::File file) {
void ZipFileCreator::ReportDone(bool success) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
- // The current user of this class holds no reference to |this| so resetting
- // the |utility_process_mojo_client_| here could release the last reference
- // and delete |this|. So save |callback_| before resetting the client.
- auto callback = base::ResetAndReturn(&callback_);
utility_process_mojo_client_.reset();
- callback.Run(success);
+ base::ResetAndReturn(&callback_).Run(success);
}
} // namespace file_manager
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698