Add firewall methods to chromeos::PermissionBrokerClient.

chromeos/dbus/permission_broker_client.cc

Index: chromeos/dbus/permission_broker_client.cc
diff --git a/chromeos/dbus/permission_broker_client.cc b/chromeos/dbus/permission_broker_client.cc
index 4a9e432d017efa829bd98b6a368fd82638aaec78..2945c6b3f84e894565fafd876370d9da0ddcc581 100644
--- a/chromeos/dbus/permission_broker_client.cc
+++ b/chromeos/dbus/permission_broker_client.cc
@@ -15,6 +15,10 @@ using permission_broker::kPermissionBrokerInterface;
 using permission_broker::kPermissionBrokerServiceName;
 using permission_broker::kPermissionBrokerServicePath;
 using permission_broker::kRequestPathAccess;
+using permission_broker::kRequestTcpPortAccess;
+using permission_broker::kRequestUdpPortAccess;
+using permission_broker::kReleaseTcpPort;
+using permission_broker::kReleaseUdpPort;

[hashimoto, 2015/02/26 03:22:22]

nit: How about sorting these lines lexicographically?

 
 namespace chromeos {
 
@@ -36,6 +40,60 @@ class PermissionBrokerClientImpl : public PermissionBrokerClient {
                                   weak_ptr_factory_.GetWeakPtr(), callback));
   }
 
+  void RequestTcpPortAccess(uint16 port,
+                            const std::string& interface,
+                            const dbus::FileDescriptor& lifeline_fd,
+                            const ResultCallback& callback) override {
+    dbus::MethodCall method_call(kPermissionBrokerInterface,
+                                 kRequestTcpPortAccess);
+    dbus::MessageWriter writer(&method_call);
+    writer.AppendUint16(port);
+    writer.AppendString(interface);
+    writer.AppendFileDescriptor(lifeline_fd);
+    proxy_->CallMethod(&method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+                       base::Bind(&PermissionBrokerClientImpl::OnResponse,
+                                  weak_ptr_factory_.GetWeakPtr(), callback));
+  }
+
+  void RequestUdpPortAccess(uint16 port,
+                            const std::string& interface,
+                            const dbus::FileDescriptor& lifeline_fd,
+                            const ResultCallback& callback) override {
+    dbus::MethodCall method_call(kPermissionBrokerInterface,
+                                 kRequestUdpPortAccess);
+    dbus::MessageWriter writer(&method_call);
+    writer.AppendUint16(port);
+    writer.AppendString(interface);
+    writer.AppendFileDescriptor(lifeline_fd);
+    proxy_->CallMethod(&method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+                       base::Bind(&PermissionBrokerClientImpl::OnResponse,
+                                  weak_ptr_factory_.GetWeakPtr(), callback));
+  }
+
+  void ReleaseTcpPort(uint16 port,
+                      const std::string& interface,
+                      const ResultCallback& callback) override {
+    dbus::MethodCall method_call(kPermissionBrokerInterface, kReleaseTcpPort);
+    dbus::MessageWriter writer(&method_call);
+    writer.AppendUint16(port);
+    writer.AppendString(interface);
+    proxy_->CallMethod(&method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+                       base::Bind(&PermissionBrokerClientImpl::OnResponse,
+                                  weak_ptr_factory_.GetWeakPtr(), callback));
+  }
+
+  void ReleaseUdpPort(uint16 port,
+                      const std::string& interface,
+                      const ResultCallback& callback) override {
+    dbus::MethodCall method_call(kPermissionBrokerInterface, kReleaseUdpPort);
+    dbus::MessageWriter writer(&method_call);
+    writer.AppendUint16(port);
+    writer.AppendString(interface);
+    proxy_->CallMethod(&method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+                       base::Bind(&PermissionBrokerClientImpl::OnResponse,
+                                  weak_ptr_factory_.GetWeakPtr(), callback));
+  }
+
  protected:
   void Init(dbus::Bus* bus) override {
     proxy_ =