shill: cleaning glint warnings

The typical warnings addressed include a lot of missing includes
and missing argument names. About 30 files edited in total.

Cpplint was the actual lint being run.

BUG=chromium-os:15873
TEST=Ran all the unit tests.

Change-Id: I64a8b76cd8f94c7729743b76b41a956f1b7370cf
Reviewed-on: https://gerrit.chromium.org/gerrit/12253
Commit-Ready: Hristo Stefanov <hstefanov@chromium.org>
Reviewed-by: Hristo Stefanov <hstefanov@chromium.org>
Tested-by: Hristo Stefanov <hstefanov@chromium.org>
diff --git a/mock_ipconfig.h b/mock_ipconfig.h
index 6c73a7c..a20ece0 100644
--- a/mock_ipconfig.h
+++ b/mock_ipconfig.h
@@ -5,6 +5,8 @@
 #ifndef SHILL_MOCK_IPCONFIG_
 #define SHILL_MOCK_IPCONFIG_
 
+#include <string>
+
 #include <base/basictypes.h>
 #include <gmock/gmock.h>
 
@@ -22,8 +24,10 @@
   MOCK_METHOD0(RenewIP, bool(void));
   MOCK_METHOD0(ReleaseIP, bool(void));
 
-  MOCK_METHOD2(Load, bool(StoreInterface *, const std::string &));
-  MOCK_METHOD2(Save, bool(StoreInterface *, const std::string &));
+  MOCK_METHOD2(Load, bool(StoreInterface *storage,
+                          const std::string &id_suffix));
+  MOCK_METHOD2(Save, bool(StoreInterface *storage,
+                          const std::string &id_suffix));
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockIPConfig);