Chromium Code Reviews| Index: chrome/browser/safe_browsing/chrome_cleaner/srt_chrome_prompt_impl.h |
| diff --git a/chrome/browser/safe_browsing/chrome_cleaner/srt_chrome_prompt_impl.h b/chrome/browser/safe_browsing/chrome_cleaner/srt_chrome_prompt_impl.h |
| index fe92d7c7c781f246f8c72201e7e1cc080d6540b4..67215d9fbf69e3a4f17bd5c6070024acae820d86 100644 |
| --- a/chrome/browser/safe_browsing/chrome_cleaner/srt_chrome_prompt_impl.h |
| +++ b/chrome/browser/safe_browsing/chrome_cleaner/srt_chrome_prompt_impl.h |
| @@ -5,7 +5,12 @@ |
| #ifndef CHROME_BROWSER_SAFE_BROWSING_CHROME_CLEANER_SRT_CHROME_PROMPT_IMPL_H_ |
| #define CHROME_BROWSER_SAFE_BROWSING_CHROME_CLEANER_SRT_CHROME_PROMPT_IMPL_H_ |
| +#include <memory> |
| +#include <set> |
| +#include <vector> |
| + |
| #include "base/callback.h" |
| +#include "base/files/file_path.h" |
| #include "base/macros.h" |
| #include "components/chrome_cleaner/public/interfaces/chrome_prompt.mojom.h" |
| #include "mojo/public/cpp/bindings/binding.h" |
| @@ -15,8 +20,13 @@ namespace safe_browsing { |
| // Implementation of the ChromePrompt Mojo interface. |
| class ChromePromptImpl : public chrome_cleaner::mojom::ChromePrompt { |
| public: |
| + using OnPromptUser = base::OnceCallback<void( |
| + std::unique_ptr<std::set<base::FilePath>>, |
|
ftirelo
2017/05/24 01:45:20
Would it make sense to use std::set<base::FilePath
alito
2017/05/24 03:20:23
My reading of the style guide is that this isn't a
|
| + chrome_cleaner::mojom::ChromePrompt::PromptUserCallback)>; |
| + |
| ChromePromptImpl(chrome_cleaner::mojom::ChromePromptRequest request, |
| - base::Closure on_connection_closed); |
| + base::Closure on_connection_closed, |
| + OnPromptUser on_prompt_user); |
| ~ChromePromptImpl() override; |
| void PromptUser( |
| @@ -27,6 +37,7 @@ class ChromePromptImpl : public chrome_cleaner::mojom::ChromePrompt { |
| private: |
| mojo::Binding<chrome_cleaner::mojom::ChromePrompt> binding_; |
| + OnPromptUser on_prompt_user_; |
| DISALLOW_COPY_AND_ASSIGN(ChromePromptImpl); |
| }; |