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_store.h b/mock_store.h
index 581d983..e42063e 100644
--- a/mock_store.h
+++ b/mock_store.h
@@ -5,6 +5,10 @@
 #ifndef SHILL_MOCK_STORE_
 #define SHILL_MOCK_STORE_
 
+#include <set>
+#include <string>
+#include <vector>
+
 #include <base/basictypes.h>
 #include <gmock/gmock.h>
 
@@ -22,8 +26,8 @@
   MOCK_METHOD1(GetGroupsWithKey,
                std::set<std::string>(const std::string &key));
   MOCK_METHOD1(ContainsGroup, bool(const std::string &group));
-  MOCK_METHOD2(DeleteKey,
-               bool(const std::string &group, const std::string &key));
+  MOCK_METHOD2(DeleteKey, bool(const std::string &group,
+                               const std::string &key));
   MOCK_METHOD1(DeleteGroup, bool(const std::string &group));
   MOCK_METHOD1(SetHeader, bool(const std::string &header));
   MOCK_METHOD3(GetString, bool(const std::string &group,