commit 1b2531423a15ccc5ac5acaf09edac9a687902dfa
author Paul Stewart <pstew@chromium.org> Thu Jan 26 14:05:52 2012 -0800
committer Gerrit <chrome-bot@google.com> Thu Jan 26 18:42:09 2012 -0800
tree aa666a84c54f73bef05413302f5c45bea753ac36
parent 7c1fece3da9a33649b02c9b8de2564417d0dee74

shill: Implement more of Profile DBus interface

Return a DBus path from Manager.GetActiveProfile().
Implement the "Profiles" property on the manager Manager.
Fix the "Entries" property on the Profile to only report group
identifiers that correspond to technologies (not ipconfig,
devices, etc).
Fix the "Services" Profile property, to only appear as a property
of the active profile.

BUG=chromium-os:25538, chromium-os:23702
TEST=Manual: Running "list-profiles" from the flimflam test suite now
works correctly.

Change-Id: I3120fe54f02662822186ac033fab0b3566449705
Reviewed-on: https://gerrit.chromium.org/gerrit/14904
Commit-Ready: Paul Stewart <pstew@chromium.org>
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>

manager.h

diff --git a/manager.h b/manager.h
index 085abf3..f61d4ce 100644
--- a/manager.h
+++ b/manager.h
@@ -56,7 +56,8 @@
   void Start();
   void Stop();
 
-  const ProfileRefPtr &ActiveProfile();
+  const ProfileRefPtr &ActiveProfile() const;
+  bool IsActiveProfile(const ProfileRefPtr &profile) const;
   bool MoveServiceToProfile(const ServiceRefPtr &to_move,
                             const ProfileRefPtr &destination);
 
@@ -119,13 +120,14 @@
  private:
   friend class ManagerAdaptorInterface;
   friend class ManagerTest;
-  FRIEND_TEST(ManagerTest, DeviceRegistrationAndStart);
-  FRIEND_TEST(ManagerTest, PushPopProfile);
-  FRIEND_TEST(ManagerTest, SortServices);
-  FRIEND_TEST(ManagerTest, SortServicesWithConnection);
   FRIEND_TEST(ManagerTest, AvailableTechnologies);
   FRIEND_TEST(ManagerTest, ConnectedTechnologies);
   FRIEND_TEST(ManagerTest, DefaultTechnology);
+  FRIEND_TEST(ManagerTest, DeviceRegistrationAndStart);
+  FRIEND_TEST(ManagerTest, EnumerateProfiles);
+  FRIEND_TEST(ManagerTest, PushPopProfile);
+  FRIEND_TEST(ManagerTest, SortServices);
+  FRIEND_TEST(ManagerTest, SortServicesWithConnection);
 
   static const char kManagerErrorNoDevice[];
 
@@ -137,9 +139,10 @@
   std::string DefaultTechnology(Error *error);
   std::vector<std::string> EnabledTechnologies(Error *error);
   std::vector<std::string> EnumerateDevices(Error *error);
+  std::vector<std::string> EnumerateProfiles(Error *error);
   // TODO(cmasone): This should be implemented by filtering |services_|.
   std::vector<std::string> EnumerateWatchedServices(Error *error);
-  std::string GetActiveProfileName(Error *error);
+  std::string GetActiveProfileRpcIdentifier(Error *error);
 
   void HelpRegisterDerivedString(
       const std::string &name,