Protocol / client plugin changes to support interactively asking for a PIN

remoting/client/plugin/pepper_pin_fetcher.h

+// Copyright 2013 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_CLIENT_PLUGIN_PEPPER_PIN_FETCHER_H_
+#define REMOTING_CLIENT_PLUGIN_PEPPER_PIN_FETCHER_H_
+
+#include "base/callback.h"
+#include "base/memory/weak_ptr.h"
+#include "remoting/protocol/pin_fetcher.h"
+
+namespace remoting {
+
+class ChromotingInstance;
+
+class PepperPinFetcher : public protocol::PinFetcher {
+ public:
+  explicit PepperPinFetcher(base::WeakPtr<ChromotingInstance> parent);
+  virtual ~PepperPinFetcher() OVERRIDE;
+
+  // protocol::PinClientAuthenticator::PinFetcher interface.
+  virtual void FetchPin(
+      const PinFetchedCallback& pin_fetched_callback) OVERRIDE;
+  virtual void OnPinFetched(const std::string& pin);
+ private:

[Sergey Ulanov, 2013/03/17 21:29:21]

nit: add empty line before private:

[rmsousa, 2013/03/18 21:07:26]

Done.

+  base::WeakPtr<ChromotingInstance> parent_;

[Sergey Ulanov, 2013/03/17 21:29:21]

parent_ is ambiguous. Call it plugin_ or plugin_instance

[rmsousa, 2013/03/18 21:07:26]

Done.

+  base::WeakPtrFactory<PepperPinFetcher> weak_factory_;
+  PinFetchedCallback pin_fetched_callback_;
+  DISALLOW_COPY_AND_ASSIGN(PepperPinFetcher);

[Sergey Ulanov, 2013/03/17 21:29:21]

nit: empty line 

[rmsousa, 2013/03/18 21:07:26]

Done.

+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_CLIENT_PLUGIN_PEPPER_PIN_FETCHER_H_