It2Me Confirmation Dialog

remoting/host/it2me/it2me_confirmation_dialog.h

+// Copyright 2014 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 REMOTING_HOST_IT2ME_IT2ME_CONFIRMATION_DIALOG_H_
+#define REMOTING_HOST_IT2ME_IT2ME_CONFIRMATION_DIALOG_H_
+
+#include "base/basictypes.h"
+#include "base/callback_forward.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/single_thread_task_runner.h"
+
+namespace remoting {
+
+class It2MeConfirmationDialog {
+ public:
+  enum class Result {

[Sergey Ulanov, 2014/12/19 03:16:19]

maybe just use bool? we are unlikely to add other values here and the code
should be readable without this enum. But feel free to keep it if you disagree.

[dcaiafa, 2014/12/20 00:22:00]

I generally dislike boolean parameters (i.e. ReportResult(false) <
ReportResult(Result::CANCEL)), so I'd like to keep this.

+    OK,
+    CANCEL
+  };
+
+  typedef base::Callback<void(Result)> ResultCallback;
+
+  virtual ~It2MeConfirmationDialog();

[Sergey Ulanov, 2014/12/19 03:16:19]

inline distructor for virtual interfaces.

[dcaiafa, 2014/12/20 00:22:00]

Done.

+
+  virtual void Show(const ResultCallback& callback) = 0;
+};
+
+class It2MeConfirmationDialogFactory {
+ public:
+  It2MeConfirmationDialogFactory();
+  virtual ~It2MeConfirmationDialogFactory();
+
+  virtual scoped_ptr<It2MeConfirmationDialog> Create();
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_IT2ME_IT2ME_CONFIRMATION_DIALOG_H_