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

Unified Diff: chrome/browser/safe_browsing/srt_chrome_prompt_impl.cc

Issue 2834613003: Adds error handling support for the SwReporter launcher. (Closed)
Patch Set: Code review 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/srt_chrome_prompt_impl.cc
diff --git a/chrome/browser/safe_browsing/srt_chrome_prompt_impl.cc b/chrome/browser/safe_browsing/srt_chrome_prompt_impl.cc
index 38bb6015efc851f563ca2bece0c69110a7fda174..0bf38d1ba50cb25777f7291dba3481aade289b08 100644
--- a/chrome/browser/safe_browsing/srt_chrome_prompt_impl.cc
+++ b/chrome/browser/safe_browsing/srt_chrome_prompt_impl.cc
@@ -14,8 +14,11 @@ using chrome_cleaner::mojom::ElevationStatus;
using chrome_cleaner::mojom::PromptAcceptance;
using chrome_cleaner::mojom::UwSPtr;
-ChromePromptImpl::ChromePromptImpl(ChromePromptRequest request)
- : binding_(this, std::move(request)) {}
+ChromePromptImpl::ChromePromptImpl(ChromePromptRequest request,
+ base::Closure on_connection_closed)
+ : binding_(this, std::move(request)) {
+ binding_.set_connection_error_handler(on_connection_closed);
grt (UTC plus 2) 2017/04/25 07:06:39 may as well std::move the closure, no?
ftirelo 2017/04/25 22:24:05 Done.
+}
ChromePromptImpl::~ChromePromptImpl() {}

Powered by Google App Engine
This is Rietveld 408576698