Clean up shill fake implementations

chromeos/network/network_state_handler.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROMEOS_NETWORK_NETWORK_STATE_HANDLER_H_
 #define CHROMEOS_NETWORK_NETWORK_STATE_HANDLER_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 224 matching lines @@
   const std::string& GetCheckPortalListForTest() const {
     return check_portal_list_;
   }
 
   // Returns the FavoriteState of the EthernetEAP service, which contains the
   // EAP parameters used by the ethernet with |service_path|. If |service_path|
   // doesn't refer to an ethernet service or if the ethernet service is not
   // connected using EAP, returns NULL.
   const FavoriteState* GetEAPForEthernet(const std::string& service_path) const;
 
+  const std::string& default_network_path() const {

[pneubeck (no reviews), 2014/06/06 20:04:24]

equivalent to DefaultNetwork()->path() ?
might be worth a comment.

[stevenjb, 2014/06/06 20:06:12]

It's not quite, because DefaultNetwork() might be NULL, whereas this is always
safe to call. Also this doesn't require knowledge of NetworkState.

+    return default_network_path_;
+  }
+
   // Construct and initialize an instance for testing.
   static NetworkStateHandler* InitializeForTest();
 
   // Default set of comma separated interfaces on which to enable
   // portal checking.
   static const char kDefaultCheckPortalList[];
 
  protected:
   friend class NetworkHandler;
   NetworkStateHandler();
@@ skipping 142 matching lines @@
   // Map of network specifiers to guids. Contains an entry for each
   // FavoriteState that is not saved in a profile.
   SpecifierGuidMap specifier_guid_map_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkStateHandler);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_NETWORK_NETWORK_STATE_HANDLER_H_