shill: logging improvements

- in sorted service list, include the reason that service A
  comes before service B
- set friendly name for ethernet service
  (so they no longer log as "0")
- log argument to Connection::SetIsDefault

BUG=None
TEST=unit tests

Change-Id: I34f893cad0ba76be988171868540327105e9dd50
Reviewed-on: https://gerrit.chromium.org/gerrit/16401
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Tested-by: mukesh agrawal <quiche@chromium.org>
Commit-Ready: mukesh agrawal <quiche@chromium.org>
diff --git a/service_sorter.h b/service_sorter.h
index 7707a17..34887df 100644
--- a/service_sorter.h
+++ b/service_sorter.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium OS Authors. All rights reserved.
+// Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
@@ -23,7 +23,8 @@
   explicit ServiceSorter(const std::vector<Technology::Identifier> &tech_order)
       : technology_order_(tech_order) {}
   bool operator() (ServiceRefPtr a, ServiceRefPtr b) {
-    return Service::Compare(a, b, technology_order_);
+    const char *reason;
+    return Service::Compare(a, b, technology_order_, &reason);
   }
 
  private: