Remove weave::Privet interface

Methods moved into weave::Device

BUG:24267885
Change-Id: Id400faff1db457ad1a4911c8c8c5d4118f03deb4
Reviewed-on: https://weave-review.googlesource.com/1216
Reviewed-by: Alex Vakulenko <avakulenko@google.com>
diff --git a/libweave/include/weave/device.h b/libweave/include/weave/device.h
index edc2075..5517d10 100644
--- a/libweave/include/weave/device.h
+++ b/libweave/include/weave/device.h
@@ -12,7 +12,6 @@
 #include <weave/cloud.h>
 #include <weave/commands.h>
 #include <weave/export.h>
-#include <weave/privet.h>
 #include <weave/provider/bluetooth.h>
 #include <weave/provider/config_store.h>
 #include <weave/provider/dns_service_discovery.h>
@@ -27,10 +26,6 @@
 
 class Device {
  public:
-  // Callback type for AddSettingsChangedCallback.
-  using SettingsChangedCallback =
-      base::Callback<void(const Settings& settings)>;
-
   virtual ~Device() = default;
 
   virtual void Start(provider::ConfigStore* config_store,
@@ -45,6 +40,9 @@
   // Returns reference the current settings.
   virtual const Settings& GetSettings() = 0;
 
+  // Callback type for AddSettingsChangedCallback.
+  using SettingsChangedCallback =
+      base::Callback<void(const Settings& settings)>;
   // Subscribes to notification settings changes.
   virtual void AddSettingsChangedCallback(
       const SettingsChangedCallback& callback) = 0;
@@ -52,7 +50,19 @@
   virtual Commands* GetCommands() = 0;
   virtual State* GetState() = 0;
   virtual Cloud* GetCloud() = 0;
-  virtual Privet* GetPrivet() = 0;
+
+  // Handler should display pin code to the user.
+  using PairingBeginCallback =
+      base::Callback<void(const std::string& session_id,
+                          PairingType pairing_type,
+                          const std::vector<uint8_t>& code)>;
+  // Handler should stop displaying pin code.
+  using PairingEndCallback =
+      base::Callback<void(const std::string& session_id)>;
+  // Subscribes to notification about client pairing events.
+  virtual void AddPairingChangedCallbacks(
+      const PairingBeginCallback& begin_callback,
+      const PairingEndCallback& end_callback) = 0;
 
   LIBWEAVE_EXPORT static std::unique_ptr<Device> Create();
 };
diff --git a/libweave/include/weave/privet.h b/libweave/include/weave/privet.h
deleted file mode 100644
index 25f15a4..0000000
--- a/libweave/include/weave/privet.h
+++ /dev/null
@@ -1,35 +0,0 @@
-// Copyright 2015 The Chromium OS 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 LIBWEAVE_INCLUDE_WEAVE_PRIVET_H_
-#define LIBWEAVE_INCLUDE_WEAVE_PRIVET_H_
-
-#include <string>
-#include <vector>
-
-#include <base/callback.h>
-#include <weave/settings.h>
-
-namespace weave {
-
-class Privet {
- public:
-  using OnPairingStartedCallback =
-      base::Callback<void(const std::string& session_id,
-                          PairingType pairing_type,
-                          const std::vector<uint8_t>& code)>;
-  using OnPairingEndedCallback =
-      base::Callback<void(const std::string& session_id)>;
-
-  virtual void AddOnPairingChangedCallbacks(
-      const OnPairingStartedCallback& on_start,
-      const OnPairingEndedCallback& on_end) = 0;
-
- protected:
-  virtual ~Privet() = default;
-};
-
-}  // namespace weave
-
-#endif  // LIBWEAVE_INCLUDE_WEAVE_PRIVET_H_
diff --git a/libweave/src/device_manager.cc b/libweave/src/device_manager.cc
index ff19082..c2712e8 100644
--- a/libweave/src/device_manager.cc
+++ b/libweave/src/device_manager.cc
@@ -88,10 +88,6 @@
   return device_info_.get();
 }
 
-Privet* DeviceManager::GetPrivet() {
-  return privet_.get();
-}
-
 void DeviceManager::StartPrivet(provider::TaskRunner* task_runner,
                                 provider::Network* network,
                                 provider::DnsServiceDiscovery* dns_sd,
@@ -104,6 +100,12 @@
                  state_manager_.get());
 }
 
+void DeviceManager::AddPairingChangedCallbacks(
+      const PairingBeginCallback& begin_callback,
+      const PairingEndCallback& end_callback) {
+  privet_->AddOnPairingChangedCallbacks(begin_callback, end_callback);
+}
+
 std::unique_ptr<Device> Device::Create() {
   return std::unique_ptr<Device>{new DeviceManager};
 }
diff --git a/libweave/src/device_manager.h b/libweave/src/device_manager.h
index de294e7..6f68054 100644
--- a/libweave/src/device_manager.h
+++ b/libweave/src/device_manager.h
@@ -41,7 +41,9 @@
   Commands* GetCommands() override;
   State* GetState() override;
   Cloud* GetCloud() override;
-  Privet* GetPrivet() override;
+  void AddPairingChangedCallbacks(
+      const PairingBeginCallback& begin_callback,
+      const PairingEndCallback& end_callback) override;
 
   Config* GetConfig();
 
diff --git a/libweave/src/privet/privet_manager.h b/libweave/src/privet/privet_manager.h
index 3881971..35fb6f9 100644
--- a/libweave/src/privet/privet_manager.h
+++ b/libweave/src/privet/privet_manager.h
@@ -42,7 +42,7 @@
 class Publisher;
 class SecurityManager;
 
-class Manager : public Privet, public CloudDelegate::Observer {
+class Manager : public CloudDelegate::Observer {
  public:
   Manager();
   ~Manager() override;
@@ -59,8 +59,8 @@
   std::string GetCurrentlyConnectedSsid() const;
 
   void AddOnPairingChangedCallbacks(
-      const OnPairingStartedCallback& on_start,
-      const OnPairingEndedCallback& on_end) override;
+      const Device::PairingBeginCallback& begin_callback,
+      const Device::PairingEndCallback& end_callback);
 
  private:
   // CloudDelegate::Observer
diff --git a/libweave/src/privet/privet_types.cc b/libweave/src/privet/privet_types.cc
index 3b801f4..8d380f0 100644
--- a/libweave/src/privet/privet_types.cc
+++ b/libweave/src/privet/privet_types.cc
@@ -8,7 +8,6 @@
 
 #include <weave/enum_to_string.h>
 #include <weave/export.h>
-#include <weave/privet.h>
 #include <weave/provider/network.h>
 
 namespace weave {
diff --git a/libweave/src/privet/security_delegate.h b/libweave/src/privet/security_delegate.h
index 46eb425..e56dee7 100644
--- a/libweave/src/privet/security_delegate.h
+++ b/libweave/src/privet/security_delegate.h
@@ -10,7 +10,6 @@
 #include <string>
 
 #include <base/time/time.h>
-#include <weave/privet.h>
 
 #include "src/privet/privet_types.h"
 
diff --git a/libweave/src/privet/wifi_bootstrap_manager.h b/libweave/src/privet/wifi_bootstrap_manager.h
index d93884f..6423b9d 100644
--- a/libweave/src/privet/wifi_bootstrap_manager.h
+++ b/libweave/src/privet/wifi_bootstrap_manager.h
@@ -14,7 +14,6 @@
 #include <base/memory/weak_ptr.h>
 #include <base/scoped_observer.h>
 #include <base/time/time.h>
-#include <weave/privet.h>
 
 #include "src/privet/privet_types.h"
 #include "src/privet/wifi_delegate.h"