Commit fb2a7605 authored by dtapuska's avatar dtapuska Committed by Commit bot

Add DBus Bindings for leaderd

leaderd is a ChromeOS system daemon that exposes group leader
capability. Expose the ability to create groups, join and listen
to events on the groups.

BUG=453872

Review URL: https://codereview.chromium.org/887083002

Cr-Commit-Position: refs/heads/master@{#316288}
parent f7550482
...@@ -154,6 +154,8 @@ ...@@ -154,6 +154,8 @@
'dbus/fake_image_burner_client.h', 'dbus/fake_image_burner_client.h',
'dbus/fake_introspectable_client.cc', 'dbus/fake_introspectable_client.cc',
'dbus/fake_introspectable_client.h', 'dbus/fake_introspectable_client.h',
'dbus/fake_leadership_daemon_manager_client.cc',
'dbus/fake_leadership_daemon_manager_client.h',
'dbus/fake_lorgnette_manager_client.cc', 'dbus/fake_lorgnette_manager_client.cc',
'dbus/fake_lorgnette_manager_client.h', 'dbus/fake_lorgnette_manager_client.h',
'dbus/fake_modem_messaging_client.cc', 'dbus/fake_modem_messaging_client.cc',
...@@ -196,6 +198,8 @@ ...@@ -196,6 +198,8 @@
'dbus/image_burner_client.h', 'dbus/image_burner_client.h',
'dbus/introspectable_client.cc', 'dbus/introspectable_client.cc',
'dbus/introspectable_client.h', 'dbus/introspectable_client.h',
'dbus/leadership_daemon_manager_client.cc',
'dbus/leadership_daemon_manager_client.h',
'dbus/lorgnette_manager_client.cc', 'dbus/lorgnette_manager_client.cc',
'dbus/lorgnette_manager_client.h', 'dbus/lorgnette_manager_client.h',
'dbus/modem_messaging_client.cc', 'dbus/modem_messaging_client.cc',
......
...@@ -44,6 +44,7 @@ ...@@ -44,6 +44,7 @@
#include "chromeos/dbus/fake_gsm_sms_client.h" #include "chromeos/dbus/fake_gsm_sms_client.h"
#include "chromeos/dbus/fake_image_burner_client.h" #include "chromeos/dbus/fake_image_burner_client.h"
#include "chromeos/dbus/fake_introspectable_client.h" #include "chromeos/dbus/fake_introspectable_client.h"
#include "chromeos/dbus/fake_leadership_daemon_manager_client.h"
#include "chromeos/dbus/fake_lorgnette_manager_client.h" #include "chromeos/dbus/fake_lorgnette_manager_client.h"
#include "chromeos/dbus/fake_modem_messaging_client.h" #include "chromeos/dbus/fake_modem_messaging_client.h"
#include "chromeos/dbus/fake_nfc_adapter_client.h" #include "chromeos/dbus/fake_nfc_adapter_client.h"
...@@ -65,6 +66,7 @@ ...@@ -65,6 +66,7 @@
#include "chromeos/dbus/gsm_sms_client.h" #include "chromeos/dbus/gsm_sms_client.h"
#include "chromeos/dbus/image_burner_client.h" #include "chromeos/dbus/image_burner_client.h"
#include "chromeos/dbus/introspectable_client.h" #include "chromeos/dbus/introspectable_client.h"
#include "chromeos/dbus/leadership_daemon_manager_client.h"
#include "chromeos/dbus/lorgnette_manager_client.h" #include "chromeos/dbus/lorgnette_manager_client.h"
#include "chromeos/dbus/modem_messaging_client.h" #include "chromeos/dbus/modem_messaging_client.h"
#include "chromeos/dbus/nfc_adapter_client.h" #include "chromeos/dbus/nfc_adapter_client.h"
...@@ -103,6 +105,7 @@ const struct { ...@@ -103,6 +105,7 @@ const struct {
{ "cryptohome", DBusClientBundle::CRYPTOHOME }, { "cryptohome", DBusClientBundle::CRYPTOHOME },
{ "debug_daemon", DBusClientBundle::DEBUG_DAEMON }, { "debug_daemon", DBusClientBundle::DEBUG_DAEMON },
{ "easy_unlock", DBusClientBundle::EASY_UNLOCK }, { "easy_unlock", DBusClientBundle::EASY_UNLOCK },
{ "leadership_daemon", DBusClientBundle::LEADERSHIP_DAEMON },
{ "lorgnette_manager", DBusClientBundle::LORGNETTE_MANAGER }, { "lorgnette_manager", DBusClientBundle::LORGNETTE_MANAGER },
{ "shill", DBusClientBundle::SHILL }, { "shill", DBusClientBundle::SHILL },
{ "gsm_sms", DBusClientBundle::GSM_SMS }, { "gsm_sms", DBusClientBundle::GSM_SMS },
...@@ -277,6 +280,14 @@ DBusClientBundle::DBusClientBundle(DBusClientTypeMask unstub_client_mask) ...@@ -277,6 +280,14 @@ DBusClientBundle::DBusClientBundle(DBusClientTypeMask unstub_client_mask)
else else
privet_daemon_client_.reset(new FakePrivetDaemonClient); privet_daemon_client_.reset(new FakePrivetDaemonClient);
if (!IsUsingStub(LEADERSHIP_DAEMON)) {
leadership_daemon_manager_client_.reset(
LeadershipDaemonManagerClient::Create());
} else {
leadership_daemon_manager_client_.reset(
new FakeLeadershipDaemonManagerClient);
}
power_manager_client_.reset(PowerManagerClient::Create( power_manager_client_.reset(PowerManagerClient::Create(
IsUsingStub(POWER_MANAGER) ? STUB_DBUS_CLIENT_IMPLEMENTATION IsUsingStub(POWER_MANAGER) ? STUB_DBUS_CLIENT_IMPLEMENTATION
: REAL_DBUS_CLIENT_IMPLEMENTATION)); : REAL_DBUS_CLIENT_IMPLEMENTATION));
......
...@@ -28,6 +28,7 @@ class CrosDisksClient; ...@@ -28,6 +28,7 @@ class CrosDisksClient;
class CryptohomeClient; class CryptohomeClient;
class DebugDaemonClient; class DebugDaemonClient;
class EasyUnlockClient; class EasyUnlockClient;
class LeadershipDaemonManagerClient;
class LorgnetteManagerClient; class LorgnetteManagerClient;
class ShillDeviceClient; class ShillDeviceClient;
class ShillIPConfigClient; class ShillIPConfigClient;
...@@ -85,6 +86,7 @@ class CHROMEOS_EXPORT DBusClientBundle { ...@@ -85,6 +86,7 @@ class CHROMEOS_EXPORT DBusClientBundle {
SYSTEM_CLOCK = 1 << 18, SYSTEM_CLOCK = 1 << 18,
UPDATE_ENGINE = 1 << 19, UPDATE_ENGINE = 1 << 19,
PEER_DAEMON = 1 << 20, PEER_DAEMON = 1 << 20,
LEADERSHIP_DAEMON = 1 << 21,
}; };
explicit DBusClientBundle(DBusClientTypeMask unstub_client_mask); explicit DBusClientBundle(DBusClientTypeMask unstub_client_mask);
...@@ -167,6 +169,10 @@ class CHROMEOS_EXPORT DBusClientBundle { ...@@ -167,6 +169,10 @@ class CHROMEOS_EXPORT DBusClientBundle {
return easy_unlock_client_.get(); return easy_unlock_client_.get();
} }
LeadershipDaemonManagerClient* leadership_daemon_manager_client() {
return leadership_daemon_manager_client_.get();
}
LorgnetteManagerClient* lorgnette_manager_client() { LorgnetteManagerClient* lorgnette_manager_client() {
return lorgnette_manager_client_.get(); return lorgnette_manager_client_.get();
} }
...@@ -287,6 +293,7 @@ class CHROMEOS_EXPORT DBusClientBundle { ...@@ -287,6 +293,7 @@ class CHROMEOS_EXPORT DBusClientBundle {
scoped_ptr<CryptohomeClient> cryptohome_client_; scoped_ptr<CryptohomeClient> cryptohome_client_;
scoped_ptr<DebugDaemonClient> debug_daemon_client_; scoped_ptr<DebugDaemonClient> debug_daemon_client_;
scoped_ptr<EasyUnlockClient> easy_unlock_client_; scoped_ptr<EasyUnlockClient> easy_unlock_client_;
scoped_ptr<LeadershipDaemonManagerClient> leadership_daemon_manager_client_;
scoped_ptr<LorgnetteManagerClient> lorgnette_manager_client_; scoped_ptr<LorgnetteManagerClient> lorgnette_manager_client_;
scoped_ptr<PeerDaemonManagerClient> peer_daemon_manager_client_; scoped_ptr<PeerDaemonManagerClient> peer_daemon_manager_client_;
scoped_ptr<PrivetDaemonClient> privet_daemon_client_; scoped_ptr<PrivetDaemonClient> privet_daemon_client_;
......
...@@ -29,6 +29,7 @@ ...@@ -29,6 +29,7 @@
#include "chromeos/dbus/gsm_sms_client.h" #include "chromeos/dbus/gsm_sms_client.h"
#include "chromeos/dbus/image_burner_client.h" #include "chromeos/dbus/image_burner_client.h"
#include "chromeos/dbus/introspectable_client.h" #include "chromeos/dbus/introspectable_client.h"
#include "chromeos/dbus/leadership_daemon_manager_client.h"
#include "chromeos/dbus/lorgnette_manager_client.h" #include "chromeos/dbus/lorgnette_manager_client.h"
#include "chromeos/dbus/modem_messaging_client.h" #include "chromeos/dbus/modem_messaging_client.h"
#include "chromeos/dbus/nfc_adapter_client.h" #include "chromeos/dbus/nfc_adapter_client.h"
...@@ -185,6 +186,12 @@ DebugDaemonClient* DBusThreadManager::GetDebugDaemonClient() { ...@@ -185,6 +186,12 @@ DebugDaemonClient* DBusThreadManager::GetDebugDaemonClient() {
EasyUnlockClient* DBusThreadManager::GetEasyUnlockClient() { EasyUnlockClient* DBusThreadManager::GetEasyUnlockClient() {
return client_bundle_->easy_unlock_client(); return client_bundle_->easy_unlock_client();
} }
LeadershipDaemonManagerClient*
DBusThreadManager::GetLeadershipDaemonManagerClient() {
return client_bundle_->leadership_daemon_manager_client();
}
LorgnetteManagerClient* LorgnetteManagerClient*
DBusThreadManager::GetLorgnetteManagerClient() { DBusThreadManager::GetLorgnetteManagerClient() {
return client_bundle_->lorgnette_manager_client(); return client_bundle_->lorgnette_manager_client();
...@@ -308,6 +315,7 @@ void DBusThreadManager::InitializeClients() { ...@@ -308,6 +315,7 @@ void DBusThreadManager::InitializeClients() {
GetGsmSMSClient()->Init(GetSystemBus()); GetGsmSMSClient()->Init(GetSystemBus());
GetImageBurnerClient()->Init(GetSystemBus()); GetImageBurnerClient()->Init(GetSystemBus());
GetIntrospectableClient()->Init(GetSystemBus()); GetIntrospectableClient()->Init(GetSystemBus());
GetLeadershipDaemonManagerClient()->Init(GetSystemBus());
GetLorgnetteManagerClient()->Init(GetSystemBus()); GetLorgnetteManagerClient()->Init(GetSystemBus());
GetModemMessagingClient()->Init(GetSystemBus()); GetModemMessagingClient()->Init(GetSystemBus());
GetPermissionBrokerClient()->Init(GetSystemBus()); GetPermissionBrokerClient()->Init(GetSystemBus());
...@@ -536,6 +544,12 @@ void DBusThreadManagerSetter::SetEasyUnlockClient( ...@@ -536,6 +544,12 @@ void DBusThreadManagerSetter::SetEasyUnlockClient(
DBusThreadManager::Get()->client_bundle_->easy_unlock_client_ = client.Pass(); DBusThreadManager::Get()->client_bundle_->easy_unlock_client_ = client.Pass();
} }
void DBusThreadManagerSetter::SetLeadershipDaemonManagerClient(
scoped_ptr<LeadershipDaemonManagerClient> client) {
DBusThreadManager::Get()->client_bundle_->leadership_daemon_manager_client_ =
client.Pass();
}
void DBusThreadManagerSetter::SetLorgnetteManagerClient( void DBusThreadManagerSetter::SetLorgnetteManagerClient(
scoped_ptr<LorgnetteManagerClient> client) { scoped_ptr<LorgnetteManagerClient> client) {
DBusThreadManager::Get()->client_bundle_->lorgnette_manager_client_ = DBusThreadManager::Get()->client_bundle_->lorgnette_manager_client_ =
......
...@@ -45,6 +45,7 @@ class EasyUnlockClient; ...@@ -45,6 +45,7 @@ class EasyUnlockClient;
class GsmSMSClient; class GsmSMSClient;
class ImageBurnerClient; class ImageBurnerClient;
class IntrospectableClient; class IntrospectableClient;
class LeadershipDaemonManagerClient;
class LorgnetteManagerClient; class LorgnetteManagerClient;
class ModemMessagingClient; class ModemMessagingClient;
class NfcAdapterClient; class NfcAdapterClient;
...@@ -136,6 +137,7 @@ class CHROMEOS_EXPORT DBusThreadManager { ...@@ -136,6 +137,7 @@ class CHROMEOS_EXPORT DBusThreadManager {
GsmSMSClient* GetGsmSMSClient(); GsmSMSClient* GetGsmSMSClient();
ImageBurnerClient* GetImageBurnerClient(); ImageBurnerClient* GetImageBurnerClient();
IntrospectableClient* GetIntrospectableClient(); IntrospectableClient* GetIntrospectableClient();
LeadershipDaemonManagerClient* GetLeadershipDaemonManagerClient();
LorgnetteManagerClient* GetLorgnetteManagerClient(); LorgnetteManagerClient* GetLorgnetteManagerClient();
ModemMessagingClient* GetModemMessagingClient(); ModemMessagingClient* GetModemMessagingClient();
NfcAdapterClient* GetNfcAdapterClient(); NfcAdapterClient* GetNfcAdapterClient();
...@@ -223,6 +225,8 @@ class CHROMEOS_EXPORT DBusThreadManagerSetter { ...@@ -223,6 +225,8 @@ class CHROMEOS_EXPORT DBusThreadManagerSetter {
void SetCryptohomeClient(scoped_ptr<CryptohomeClient> client); void SetCryptohomeClient(scoped_ptr<CryptohomeClient> client);
void SetDebugDaemonClient(scoped_ptr<DebugDaemonClient> client); void SetDebugDaemonClient(scoped_ptr<DebugDaemonClient> client);
void SetEasyUnlockClient(scoped_ptr<EasyUnlockClient> client); void SetEasyUnlockClient(scoped_ptr<EasyUnlockClient> client);
void SetLeadershipDaemonManagerClient(
scoped_ptr<LeadershipDaemonManagerClient> client);
void SetLorgnetteManagerClient(scoped_ptr<LorgnetteManagerClient> client); void SetLorgnetteManagerClient(scoped_ptr<LorgnetteManagerClient> client);
void SetShillDeviceClient(scoped_ptr<ShillDeviceClient> client); void SetShillDeviceClient(scoped_ptr<ShillDeviceClient> client);
void SetShillIPConfigClient(scoped_ptr<ShillIPConfigClient> client); void SetShillIPConfigClient(scoped_ptr<ShillIPConfigClient> client);
......
// Copyright 2015 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.
#include "chromeos/dbus/fake_leadership_daemon_manager_client.h"
#include "base/message_loop/message_loop.h"
namespace chromeos {
namespace {
void StringDBBusMethodCallbackThunk(const StringDBusMethodCallback& callback) {
callback.Run(DBUS_METHOD_CALL_SUCCESS, std::string());
}
void VoidDBBusMethodCallbackThunk(const VoidDBusMethodCallback& callback) {
callback.Run(DBUS_METHOD_CALL_SUCCESS);
}
}
FakeLeadershipDaemonManagerClient::FakeLeadershipDaemonManagerClient() {
}
FakeLeadershipDaemonManagerClient::~FakeLeadershipDaemonManagerClient() {
}
void FakeLeadershipDaemonManagerClient::Init(dbus::Bus* bus) {
}
void FakeLeadershipDaemonManagerClient::AddObserver(Observer* observer) {
}
void FakeLeadershipDaemonManagerClient::RemoveObserver(Observer* observer) {
}
void FakeLeadershipDaemonManagerClient::JoinGroup(
const std::string& group,
const base::DictionaryValue& options,
const StringDBusMethodCallback& callback) {
base::MessageLoop::current()->PostTask(
FROM_HERE, base::Bind(&StringDBBusMethodCallbackThunk, callback));
}
void FakeLeadershipDaemonManagerClient::LeaveGroup(
const std::string& object_path,
const VoidDBusMethodCallback& callback) {
base::MessageLoop::current()->PostTask(
FROM_HERE, base::Bind(&VoidDBBusMethodCallbackThunk, callback));
}
void FakeLeadershipDaemonManagerClient::SetScore(
const std::string& object_path,
int score,
const VoidDBusMethodCallback& callback) {
base::MessageLoop::current()->PostTask(
FROM_HERE, base::Bind(&VoidDBBusMethodCallbackThunk, callback));
}
void FakeLeadershipDaemonManagerClient::PokeLeader(
const std::string& object_path,
const VoidDBusMethodCallback& callback) {
base::MessageLoop::current()->PostTask(
FROM_HERE, base::Bind(&VoidDBBusMethodCallbackThunk, callback));
}
void FakeLeadershipDaemonManagerClient::Ping(
const StringDBusMethodCallback& callback) {
base::MessageLoop::current()->PostTask(
FROM_HERE, base::Bind(&StringDBBusMethodCallbackThunk, callback));
}
const LeadershipDaemonManagerClient::GroupProperties*
FakeLeadershipDaemonManagerClient::GetGroupProperties(
const dbus::ObjectPath& object_path) {
return nullptr;
}
} // namespace chromeos
// Copyright 2015 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 CHROMEOS_DBUS_FAKE_LEADERSHIP_DAEMON_MANAGER_CLIENT_H_
#define CHROMEOS_DBUS_FAKE_LEADERSHIP_DAEMON_MANAGER_CLIENT_H_
#include <map>
#include <string>
#include <vector>
#include "chromeos/dbus/leadership_daemon_manager_client.h"
namespace chromeos {
// A fake implementation of LeadershipDaemonManagerClient. Invokes callbacks
// immediately.
class FakeLeadershipDaemonManagerClient : public LeadershipDaemonManagerClient {
public:
FakeLeadershipDaemonManagerClient();
~FakeLeadershipDaemonManagerClient() override;
// DBusClient overrides:
void Init(dbus::Bus* bus) override;
// LeadershipDaemonManagerClient overrides:
void AddObserver(Observer* observer) override;
void RemoveObserver(Observer* observer) override;
void JoinGroup(const std::string& group,
const base::DictionaryValue& options,
const StringDBusMethodCallback& callback) override;
void LeaveGroup(const std::string& object_path,
const VoidDBusMethodCallback& callback) override;
void SetScore(const std::string& object_path,
int score,
const VoidDBusMethodCallback& callback) override;
void PokeLeader(const std::string& object_path,
const VoidDBusMethodCallback& callback) override;
void Ping(const StringDBusMethodCallback& callback) override;
const GroupProperties* GetGroupProperties(
const dbus::ObjectPath& object_path) override;
private:
DISALLOW_COPY_AND_ASSIGN(FakeLeadershipDaemonManagerClient);
};
} // namespace chromeos
#endif // CHROMEOS_DBUS_FAKE_LEADERSHIP_DAEMON_MANAGER_CLIENT_H_
This diff is collapsed.
// Copyright 2015 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 CHROMEOS_DBUS_LEADERSHIP_DAEMON_MANAGER_CLIENT_H_
#define CHROMEOS_DBUS_LEADERSHIP_DAEMON_MANAGER_CLIENT_H_
#include <map>
#include <string>
#include <vector>
#include "base/callback.h"
#include "base/macros.h"
#include "base/values.h"
#include "chromeos/chromeos_export.h"
#include "chromeos/dbus/dbus_client.h"
#include "chromeos/dbus/dbus_method_call_status.h"
#include "dbus/object_path.h"
#include "dbus/property.h"
namespace chromeos {
// LeadershipDaemonManagerClient is used to communicate with the
// leaderd's Manager service. All methods should be called from
// the origin thread which initializes the DBusThreadManager instance.
class CHROMEOS_EXPORT LeadershipDaemonManagerClient : public DBusClient {
public:
// Structure of properties associated with advertised groups.
class GroupProperties : public dbus::PropertySet {
public:
GroupProperties(dbus::ObjectProxy* object_proxy,
const std::string& interface_name,
const PropertyChangedCallback& callback);
~GroupProperties() override;
const std::string& leader_uuid() const { return leader_uuid_.value(); }
const std::vector<std::string>& group_members() const {
return group_members_.value();
}
private:
dbus::Property<std::string> leader_uuid_;
dbus::Property<std::vector<std::string>> group_members_;
};
// Interface for observing changes from a leadership daemon.
class Observer {
public:
virtual ~Observer();
// Called when the manager has been added.
virtual void ManagerAdded();
// Called when the manager has been removed.
virtual void ManagerRemoved();
// Called when the group with object path |object_path| is added to the
// system.
virtual void GroupAdded(const dbus::ObjectPath& object_path);
// Called when the group with object path |object_path| is removed from
// the system.
virtual void GroupRemoved(const dbus::ObjectPath& object_path);
// Called when the adapter with object path |object_path| has a
// change in value of the property named |property_name|.
virtual void GroupPropertyChanged(const dbus::ObjectPath& object_path,
const std::string& property_name);
};
~LeadershipDaemonManagerClient() override;
// Factory function, creates a new instance which is owned by the caller.
// For normal usage, access the singleton via DBusThreadManager::Get().
static LeadershipDaemonManagerClient* Create();
// Adds and removes observers for events on all leadership group
// events. Check the |object_path| parameter of observer methods to
// determine which group is issuing the event.
virtual void AddObserver(Observer* observer) = 0;
virtual void RemoveObserver(Observer* observer) = 0;
// Calls JoinGroup method.
// |callback| is called with its |call_status| argument set to
// DBUS_METHOD_CALL_SUCCESS if the method call succeeds. Otherwise,
// |callback| is called with |call_status| set to DBUS_METHOD_CALL_FAILURE.
virtual void JoinGroup(const std::string& group,
const base::DictionaryValue& options,
const StringDBusMethodCallback& callback) = 0;
// Calls LeaveGroup method.
// |callback| is called with its |call_status| argument set to
// DBUS_METHOD_CALL_SUCCESS if the method call succeeds. Otherwise,
// |callback| is called with |call_status| set to DBUS_METHOD_CALL_FAILURE.
virtual void LeaveGroup(const std::string& object_path,
const VoidDBusMethodCallback& callback) = 0;
// Calls SetScore method.
// |callback| is called with its |call_status| argument set to
// DBUS_METHOD_CALL_SUCCESS if the method call succeeds. Otherwise,
// |callback| is called with |call_status| set to DBUS_METHOD_CALL_FAILURE.
virtual void SetScore(const std::string& object_path,
int score,
const VoidDBusMethodCallback& callback) = 0;
// Calls PokeLeader method.
// |callback| is called with its |call_status| argument set to
// DBUS_METHOD_CALL_SUCCESS if the method call succeeds. Otherwise,
// |callback| is called with |call_status| set to DBUS_METHOD_CALL_FAILURE.
virtual void PokeLeader(const std::string& object_path,
const VoidDBusMethodCallback& callback) = 0;
// Calls Ping method.
// |callback| is called with its |call_status| argument set to
// DBUS_METHOD_CALL_SUCCESS if the method call succeeds. Otherwise,
// |callback| is called with |call_status| set to DBUS_METHOD_CALL_FAILURE.
virtual void Ping(const StringDBusMethodCallback& callback) = 0;
// Obtains the properties for the group with object path |object_path|,
// any values should be copied if needed.
virtual const GroupProperties* GetGroupProperties(
const dbus::ObjectPath& object_path) = 0;
protected:
// Create() should be used instead.
LeadershipDaemonManagerClient();
private:
DISALLOW_COPY_AND_ASSIGN(LeadershipDaemonManagerClient);
};
} // namespace chromeos
#endif // CHROMEOS_DBUS_LEADERSHIP_DAEMON_MANAGER_CLIENT_H_
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment