commit 764538d1bcb07883dab5355ff556f6f87b985ba2
author Christopher Wiley <wiley@chromium.org> Fri Nov 09 10:58:23 2012 -0800
committer Gerrit <chrome-bot@google.com> Mon Nov 12 10:08:44 2012 -0800
tree 894bee1b43ebc50b89df4e8b71360a788070163a
parent 393b93f3a7684828f32a481b34cc2e94c6825f50

shill: Refactor message types into UserBoundNlMessage

Since all UserBoundNlMessage subclasses have a particular message type
and type string, pull that functionality into the abstract super class.

BUG=None
TEST=Unit tests

Change-Id: Ib44db2acabab33c48de2ef1143de6be3e1df0acf
Reviewed-on: https://gerrit.chromium.org/gerrit/37731
Tested-by: Christopher Wiley <wiley@chromium.org>
Reviewed-by: Wade Guthrie <wdg@chromium.org>
Commit-Ready: Christopher Wiley <wiley@chromium.org>

callback80211_metrics.cc

diff --git a/callback80211_metrics.cc b/callback80211_metrics.cc
index 9b20966..9acc501 100644
--- a/callback80211_metrics.cc
+++ b/callback80211_metrics.cc
@@ -29,9 +29,10 @@
 
 void Callback80211Metrics::Config80211MessageCallback(
     const UserBoundNlMessage &message) {
-  SLOG(WiFi, 3) << "Received " << message.GetMessageTypeString()
-                << " (" << + message.GetMessageType() << ")";
-  if (metrics_ && message.GetMessageType() == DeauthenticateMessage::kCommand) {
+  SLOG(WiFi, 3) << "Received " << message.message_type_string()
+                << " (" << + message.message_type() << ")";
+  if (metrics_ &&
+      message.message_type() == DeauthenticateMessage::kCommand) {
     Metrics::WiFiDisconnectByWhom by_whom =
         message.AttributeExists(NL80211_ATTR_DISCONNECTED_BY_AP) ?
                     Metrics::kDisconnectedByAp : Metrics::kDisconnectedNotByAp;