shill: Almost complete support for terminating dhcpcd.

BUG=chromium-os:16365,chromium-os:16013
TEST=unit tests

Change-Id: I00a46e8364fc4de3cc48ef72c1b0a9a88e95e6a0
Reviewed-on: http://gerrit.chromium.org/gerrit/2435
Tested-by: Darin Petkov <petkov@chromium.org>
Reviewed-by: Chris Masone <cmasone@chromium.org>
diff --git a/dhcp_provider.h b/dhcp_provider.h
index 28e07e4..ca7014b 100644
--- a/dhcp_provider.h
+++ b/dhcp_provider.h
@@ -47,18 +47,18 @@
 
   // Binds a |pid| to a DHCP |config|. When a DHCP config spawns a new DHCP
   // client, it binds itself to that client's |pid|.
-  void BindPID(unsigned int pid, DHCPConfigRefPtr config);
+  void BindPID(int pid, DHCPConfigRefPtr config);
 
   // Unbinds a |pid|. This method is used by a DHCP config to signal the
   // provider that the DHCP client has been terminated. This may result in
   // destruction of the DHCP config instance if its reference count goes to 0.
-  void UnbindPID(unsigned int pid);
+  void UnbindPID(int pid);
 
  private:
   friend struct DefaultSingletonTraits<DHCPProvider>;
-  FRIEND_TEST(DHCPConfigTest, Start);
+  FRIEND_TEST(DHCPConfigTest, StartSuccess);
 
-  typedef std::map<unsigned int, DHCPConfigRefPtr> PIDConfigMap;
+  typedef std::map<int, DHCPConfigRefPtr> PIDConfigMap;
 
   // Private to ensure that this behaves as a singleton.
   DHCPProvider();