Index: components/device_event_log/device_event_log_impl.cc |
diff --git a/chromeos/device_event_log_impl.cc b/components/device_event_log/device_event_log_impl.cc |
similarity index 91% |
rename from chromeos/device_event_log_impl.cc |
rename to components/device_event_log/device_event_log_impl.cc |
index 8df670b73d9f5fd89797710716b93eb7c0898168..b2609914e5687c683038a9f4d7e68cb9f3e401a0 100644 |
--- a/chromeos/device_event_log_impl.cc |
+++ b/components/device_event_log/device_event_log_impl.cc |
@@ -2,13 +2,13 @@ |
// Use of this source code is governed by a BSD-style license that can be |
// found in the LICENSE file. |
-#include "chromeos/device_event_log_impl.h" |
+#include "components/device_event_log/device_event_log_impl.h" |
#include <cmath> |
#include <list> |
#include <set> |
-#include "base/files/file_path.h" |
+#include "base/containers/adapters.h" |
#include "base/json/json_string_value_serializer.h" |
#include "base/json/json_writer.h" |
#include "base/logging.h" |
@@ -20,8 +20,6 @@ |
#include "base/values.h" |
#include "net/base/escape.h" |
-namespace chromeos { |
- |
namespace device_event_log { |
namespace { |
@@ -325,32 +323,30 @@ std::string DeviceEventLogImpl::GetAsString(StringOrder order, |
// determine the first entry to include. |
size_t shown_events = 0; |
size_t num_entries = 0; |
- for (LogEntryList::const_reverse_iterator riter = entries_.rbegin(); |
- riter != entries_.rend(); ++riter) { |
+ for (const LogEntry& entry : base::Reversed(entries_)) { |
++num_entries; |
- if (!LogEntryMatchesTypes(*riter, include_types, exclude_types)) |
+ if (!LogEntryMatchesTypes(entry, include_types, exclude_types)) |
continue; |
- if (riter->log_level > max_level) |
+ if (entry.log_level > max_level) |
continue; |
if (++shown_events >= max_events) |
break; |
} |
offset = entries_.size() - num_entries; |
} |
- for (LogEntryList::const_iterator iter = entries_.begin(); |
- iter != entries_.end(); ++iter) { |
+ for (const LogEntry& entry : entries_) { |
if (offset > 0) { |
--offset; |
continue; |
} |
- if (!LogEntryMatchesTypes(*iter, include_types, exclude_types)) |
+ if (!LogEntryMatchesTypes(entry, include_types, exclude_types)) |
continue; |
- if (iter->log_level > max_level) |
+ if (entry.log_level > max_level) |
continue; |
if (format_json) { |
- log_entries.AppendString(LogEntryAsJSON(*iter)); |
+ log_entries.AppendString(LogEntryAsJSON(entry)); |
} else { |
- result += LogEntryToString(*iter, show_time, show_file, show_type, |
+ result += LogEntryToString(entry, show_time, show_file, show_type, |
show_level, format_html); |
result += "\n"; |
} |
@@ -358,16 +354,15 @@ std::string DeviceEventLogImpl::GetAsString(StringOrder order, |
} else { |
size_t nlines = 0; |
// Iterate backwards through the list to show the most recent entries first. |
- for (LogEntryList::const_reverse_iterator riter = entries_.rbegin(); |
- riter != entries_.rend(); ++riter) { |
- if (!LogEntryMatchesTypes(*riter, include_types, exclude_types)) |
+ for (const LogEntry& entry : base::Reversed(entries_)) { |
+ if (!LogEntryMatchesTypes(entry, include_types, exclude_types)) |
continue; |
- if (riter->log_level > max_level) |
+ if (entry.log_level > max_level) |
continue; |
if (format_json) { |
- log_entries.AppendString(LogEntryAsJSON(*riter)); |
+ log_entries.AppendString(LogEntryAsJSON(entry)); |
} else { |
- result += LogEntryToString(*riter, show_time, show_file, show_type, |
+ result += LogEntryToString(entry, show_time, show_file, show_type, |
show_level, format_html); |
result += "\n"; |
} |
@@ -394,10 +389,13 @@ DeviceEventLogImpl::LogEntry::LogEntry(const char* filedesc, |
event(event), |
time(base::Time::Now()), |
count(1) { |
- if (filedesc) |
- file = base::FilePath(std::string(filedesc)).BaseName().value(); |
+ if (filedesc) { |
stevenjb
2015/02/17 19:24:35
nit: if (!filedesc) return;
|
+ file = filedesc; |
+ size_t last_slash_pos = file.find_last_of("\\/"); |
+ if (last_slash_pos != std::string::npos) { |
+ file.erase(0, last_slash_pos + 1); |
+ } |
+ } |
} |
} // namespace device_event_log |
- |
-} // namespace chromeos |