The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f11d62080a80c7de948ec4c4d7e9c1673becb257

commit f11d62080a80c7de948ec4c4d7e9c1673becb257
Author: sque <sque@chromium.org>
Date: Fri Jun 23 03:41:57 2017

Add GetLog method call to DebugDaemonClient

This gets an individual log from debugd.

BUG= 733846 

Review-Url: https://codereview.chromium.org/2944403002
Cr-Commit-Position: refs/heads/master@{#481805}

[modify] https://crrev.com/f11d62080a80c7de948ec4c4d7e9c1673becb257/chromeos/dbus/debug_daemon_client.cc
[modify] https://crrev.com/f11d62080a80c7de948ec4c4d7e9c1673becb257/chromeos/dbus/debug_daemon_client.h
[modify] https://crrev.com/f11d62080a80c7de948ec4c4d7e9c1673becb257/chromeos/dbus/fake_debug_daemon_client.cc
[modify] https://crrev.com/f11d62080a80c7de948ec4c4d7e9c1673becb257/chromeos/dbus/fake_debug_daemon_client.h

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/dea6441d2d8033ee73d5f746c34be09c03ed4758

commit dea6441d2d8033ee73d5f746c34be09c03ed4758
Author: sque <sque@chromium.org>
Date: Mon Jul 10 15:57:30 2017

Rename SingleLogSource to SingleLogFileLogSource

Since there is now a class SingleDebugDaemonLogSource, the old
name "SingleLogSource" is ambiguous. Rename it to indicate that the
logs come from a log file.

Also refactored the use of SingleLogSource in feedbackPrivate API:
- Use the base interface class SystemLogsSource instead, in
  LogSourceAccessManager.
- Convert from feedback_private::LogSource directly to a
  SystemLogSource in SingleLogSourceFactory. Conversion of enums is
  internal to SingleLogSourceFactory.

BUG= 733846 

Review-Url: https://codereview.chromium.org/2956513004
Cr-Commit-Position: refs/heads/master@{#485283}

[modify] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/chromeos/BUILD.gn
[rename] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/chromeos/system_logs/single_log_file_log_source.cc
[rename] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/chromeos/system_logs/single_log_file_log_source.h
[rename] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/chromeos/system_logs/single_log_file_log_source_unittest.cc
[modify] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/extensions/api/feedback_private/feedback_private_api_chromeos_unittest.cc
[modify] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/extensions/api/feedback_private/log_source_access_manager.cc
[modify] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/extensions/api/feedback_private/log_source_access_manager_chromeos_unittest.cc
[modify] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/extensions/api/feedback_private/log_source_resource.cc
[modify] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/extensions/api/feedback_private/log_source_resource.h
[modify] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/extensions/api/feedback_private/single_log_source_factory.cc
[modify] https://crrev.com/dea6441d2d8033ee73d5f746c34be09c03ed4758/chrome/browser/extensions/api/feedback_private/single_log_source_factory.h

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a135bf808de1da0ae1198d63b0f1901e42596c4f

commit a135bf808de1da0ae1198d63b0f1901e42596c4f
Author: sque <sque@chromium.org>
Date: Tue Jul 11 22:35:18 2017

Add SingleDebugDaemonLogSources to feedbackPrivate.readLogSource()

The two supported log sources are:
- modetest
- lsusb

BUG= 733846 

Review-Url: https://codereview.chromium.org/2956623003
Cr-Commit-Position: refs/heads/master@{#485710}

[modify] https://crrev.com/a135bf808de1da0ae1198d63b0f1901e42596c4f/chrome/browser/extensions/api/feedback_private/single_log_source_factory.cc
[modify] https://crrev.com/a135bf808de1da0ae1198d63b0f1901e42596c4f/chrome/common/extensions/api/feedback_private.idl

sque: Is it easy to expose "lscpi -n" the same way? Will adding another switch make things break? Alternatively making "lspci -nn" the default would work too.

Yes, it should be straightforward. Things to be done:
1. Add the "lspci -n" to src/platform2/debugd/src/log_tool.cc
2. Add it as a supported type to SingleDebugDaemonLogSource.
2. Add it to the feedbackPrivate.readLogSource API function.