Chromium Code Reviews| Index: chrome/browser/extensions/api/log_private/syslog_parser.cc |
| diff --git a/chrome/browser/extensions/api/log_private/syslog_parser.cc b/chrome/browser/extensions/api/log_private/syslog_parser.cc |
| index 4f26653554dc0458430c3f899f30155167adb8c3..d5915334194aabb8d8ab6e37258bf8acd60ae9ea 100644 |
| --- a/chrome/browser/extensions/api/log_private/syslog_parser.cc |
| +++ b/chrome/browser/extensions/api/log_private/syslog_parser.cc |
| @@ -23,7 +23,7 @@ namespace extensions { |
| namespace { |
| -const char kProcessInfoDelimiters[] = "[]"; |
| +const char kProcessInfoDelimiters[] = "[]:"; |
| } // namespace |
| @@ -40,22 +40,16 @@ SyslogParser::Error SyslogParser::ParseEntry( |
| base::StringTokenizer tokenizer(input, " "); |
| if (!tokenizer.GetNext()) { |
| LOG(ERROR) |
| - << "Error when parsing data. Expect: At least 3 tokens. Actual: 0"; |
| + << "Error when parsing data. Expect: At least 2 tokens. Actual: 0"; |
| return TOKENIZE_ERROR; |
| } |
| std::string time = tokenizer.token(); |
| if (ParseTime(time, &(entry->timestamp)) != SyslogParser::SUCCESS) { |
| return SyslogParser::PARSE_ERROR; |
| } |
| - // Skips "localhost" field. |
|
Jeffrey Yasskin
2014/08/26 18:44:27
Does this parser need to keep working before the c
mukesh agrawal
2014/08/26 22:41:15
Updated commit message to reflect that ChromeOS sy
|
| if (!tokenizer.GetNext()) { |
| LOG(ERROR) |
| - << "Error when parsing data. Expect: At least 3 tokens. Actual: 1"; |
| - return TOKENIZE_ERROR; |
| - } |
| - if (!tokenizer.GetNext()) { |
| - LOG(ERROR) |
| - << "Error when parsing data. Expect: At least 3 tokens. Actual: 2"; |
| + << "Error when parsing data. Expect: At least 2 tokens. Actual: 1"; |
| return TOKENIZE_ERROR; |
| } |
| ParseProcess(tokenizer.token(), entry.get()); |