shill: Naming and lint cleanup pass

Reduce some double-speak in IPAddress constants.  Do a cursory
glint pass and clean up a few no-brainer issues.

BUG=None
TEST=Re-run unit tests

Change-Id: I063cd72111bd1f6321f81c6c6591b9f51d87434d
Reviewed-on: http://gerrit.chromium.org/gerrit/7181
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>
diff --git a/ip_address.h b/ip_address.h
index 8ae07be..d6f0139 100644
--- a/ip_address.h
+++ b/ip_address.h
@@ -14,9 +14,9 @@
 class IPAddress {
  public:
   typedef unsigned char Family;
-  static const Family kAddressFamilyUnknown;
-  static const Family kAddressFamilyIPv4;
-  static const Family kAddressFamilyIPv6;
+  static const Family kFamilyUnknown;
+  static const Family kFamilyIPv4;
+  static const Family kFamilyIPv6;
 
   explicit IPAddress(Family family);
   IPAddress(Family family, const ByteString &address);
@@ -48,7 +48,7 @@
   int GetLength() const { return address_.GetLength(); }
   bool IsDefault() const { return address_.IsZero(); }
   bool IsValid() const {
-    return family_ != kAddressFamilyUnknown &&
+    return family_ != kFamilyUnknown &&
         GetLength() == GetAddressLength(family_);
   }