Cleanup network type matching.

chromeos/network/managed_state.h

Index: chromeos/network/managed_state.h
diff --git a/chromeos/network/managed_state.h b/chromeos/network/managed_state.h
index a1dd3ec3864924949a404e93eccb449d564fa540..1db1be1995f276bb4d6ad01c71705317d452652b 100644
--- a/chromeos/network/managed_state.h
+++ b/chromeos/network/managed_state.h
@@ -24,7 +24,7 @@ class NetworkState;
 
 // Base class for states managed by NetworkStateManger which are associated
 // with a Shill path (e.g. service path or device path).
-class ManagedState {
+class CHROMEOS_EXPORT ManagedState {
  public:
   enum ManagedType {
     MANAGED_TYPE_NETWORK,
@@ -74,6 +74,9 @@ class ManagedState {
     update_requested_ = update_requested;
   }
 
+  // Returns true if the network's |type()| is ethernet (with or without EAP).
+  bool IsTypeEthernet() const;

[stevenjb, 2013/08/28 15:51:30]

This should really be IsTypeWired() and we should add Wireless and Mobile also,
or just use shill_property_util helper methods directly.

Now that I think about it, maybe TypeEthernet is better than TypeWired, but we
should be consistent one way or the other.

[pneubeck (no reviews), 2013/09/02 20:41:21]

Done.

+
  protected:
   ManagedState(ManagedType type, const std::string& path);