Adds error handling support for the SwReporter launcher.

chrome/browser/safe_browsing/srt_chrome_prompt_impl.h

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SAFE_BROWSING_SRT_CHROME_PROMPT_IMPL_H_
 #define CHROME_BROWSER_SAFE_BROWSING_SRT_CHROME_PROMPT_IMPL_H_
 
+#include "base/macros.h"
 #include "components/chrome_cleaner/public/interfaces/chrome_prompt.mojom.h"
 #include "mojo/public/cpp/bindings/binding.h"
 
 namespace safe_browsing {
 
 // Implementation of the ChromePrompt Mojo interface.
 class ChromePromptImpl : public chrome_cleaner::mojom::ChromePrompt {
  public:
-  explicit ChromePromptImpl(chrome_cleaner::mojom::ChromePromptRequest request);
+  explicit ChromePromptImpl(chrome_cleaner::mojom::ChromePromptRequest request,

[grt (UTC plus 2), 2017/04/25 07:06:40]

remove "explicit" since there are two args now

[ftirelo, 2017/04/25 22:24:05]

Done.

+                            base::Closure on_connection_closed);

[grt (UTC plus 2), 2017/04/25 07:06:39]

#include "base/callback.h"

[ftirelo, 2017/04/25 22:24:05]

Included "base/callback_forward.h" instead that defined base::Closure.

[grt (UTC plus 2), 2017/04/26 08:46:46]

That would be sufficient if the Closure were being passed by reference, but
since you're taking by value here the caller really needs the full definition of
the type.

[ftirelo, 2017/04/26 14:14:15]

Done.

   ~ChromePromptImpl() override;
 
   void PromptUser(
       std::vector<chrome_cleaner::mojom::UwSPtr> removable_uws_found,
       chrome_cleaner::mojom::ElevationStatus elevation_status,
       const chrome_cleaner::mojom::ChromePrompt::PromptUserCallback& callback)
       override;
 
  private:
   mojo::Binding<chrome_cleaner::mojom::ChromePrompt> binding_;
+
+  DISALLOW_COPY_AND_ASSIGN(ChromePromptImpl);
 };
 
 }  // namespace safe_browsing
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_SRT_CHROME_PROMPT_IMPL_H_