Re-factor network_event_log

chromeos/network/managed_state.cc

 // 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.
 
 #include "chromeos/network/managed_state.h"
 
 #include "base/logging.h"
 #include "base/values.h"
 #include "chromeos/network/device_state.h"
+#include "chromeos/network/network_event_log.h"
 #include "chromeos/network/network_state.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 namespace chromeos {
 
 ManagedState::ManagedState(ManagedType type, const std::string& path)
     : managed_type_(type),
       path_(path),
       is_observed_(false) {
 }
 
 ManagedState::~ManagedState() {
 }
 
 ManagedState* ManagedState::Create(ManagedType type, const std::string& path) {
-  switch(type) {
+  switch (type) {
     case MANAGED_TYPE_NETWORK:
       return new NetworkState(path);
     case MANAGED_TYPE_DEVICE:
       return new DeviceState(path);
   }
   return NULL;
 }
 
 NetworkState* ManagedState::AsNetworkState() {
   if (managed_type() == MANAGED_TYPE_NETWORK)
@@ skipping 18 matching lines @@
     return GetStringValue(key, value, &type_);
   }
   return false;
 }
 
 bool ManagedState::GetBooleanValue(const std::string& key,
                                    const base::Value& value,
                                    bool* out_value) {
   bool new_value;
   if (!value.GetAsBoolean(&new_value)) {
-    LOG(WARNING) << "Failed to parse boolean value for:" << key;
+    NET_LOG_ERROR("Error parsing state value", path() + "." + key);
     return false;
   }
   if (*out_value == new_value)
     return false;
   *out_value = new_value;
   return true;
 }
 
 bool ManagedState::GetIntegerValue(const std::string& key,
                                    const base::Value& value,
                                    int* out_value) {
   int new_value;
   if (!value.GetAsInteger(&new_value)) {
-    LOG(WARNING) << "Failed to parse integer value for:" << key;
+    NET_LOG_ERROR("Error parsing state value", path() + "." + key);
     return false;
   }
   if (*out_value == new_value)
     return false;
   *out_value = new_value;
   return true;
 }
 
 bool ManagedState::GetStringValue(const std::string& key,
                                   const base::Value& value,
                                   std::string* out_value) {
   std::string new_value;
   if (!value.GetAsString(&new_value)) {
-    LOG(WARNING) << "Failed to parse string value for:" << key;
+    NET_LOG_ERROR("Error parsing state value", path() + "." + key);
     return false;
   }
   if (*out_value == new_value)
     return false;
   *out_value = new_value;
   return true;
 }
 
 }  // namespace chromeos