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

Side by Side 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, 7 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 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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/safe_browsing/srt_chrome_prompt_impl.h" 5 #include "chrome/browser/safe_browsing/srt_chrome_prompt_impl.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 8
9 namespace safe_browsing { 9 namespace safe_browsing {
10 10
11 using chrome_cleaner::mojom::ChromePrompt; 11 using chrome_cleaner::mojom::ChromePrompt;
12 using chrome_cleaner::mojom::ChromePromptRequest; 12 using chrome_cleaner::mojom::ChromePromptRequest;
13 using chrome_cleaner::mojom::ElevationStatus; 13 using chrome_cleaner::mojom::ElevationStatus;
14 using chrome_cleaner::mojom::PromptAcceptance; 14 using chrome_cleaner::mojom::PromptAcceptance;
15 using chrome_cleaner::mojom::UwSPtr; 15 using chrome_cleaner::mojom::UwSPtr;
16 16
17 ChromePromptImpl::ChromePromptImpl(ChromePromptRequest request) 17 ChromePromptImpl::ChromePromptImpl(ChromePromptRequest request,
18 : binding_(this, std::move(request)) {} 18 base::Closure on_connection_closed)
19 : binding_(this, std::move(request)) {
20 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.
21 }
19 22
20 ChromePromptImpl::~ChromePromptImpl() {} 23 ChromePromptImpl::~ChromePromptImpl() {}
21 24
22 void ChromePromptImpl::PromptUser( 25 void ChromePromptImpl::PromptUser(
23 std::vector<UwSPtr> removable_uws_found, 26 std::vector<UwSPtr> removable_uws_found,
24 ElevationStatus elevation_status, 27 ElevationStatus elevation_status,
25 const ChromePrompt::PromptUserCallback& callback) { 28 const ChromePrompt::PromptUserCallback& callback) {
26 // Placeholder. The actual implementation will show the prompt dialog to the 29 // Placeholder. The actual implementation will show the prompt dialog to the
27 // user and invoke this callback depending on the user's response. 30 // user and invoke this callback depending on the user's response.
28 callback.Run(PromptAcceptance::DENIED); 31 callback.Run(PromptAcceptance::DENIED);
29 } 32 }
30 33
31 } // namespace safe_browsing 34 } // namespace safe_browsing
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698