Dump touchpad event logs for touch log source

chrome/browser/chromeos/system_logs/touch_log_source_ozone.cc

 // Copyright (c) 2013 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 "chrome/browser/chromeos/system_logs/touch_log_source.h"
 
 #include "ash/touch/touch_hud_debug.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/command_line.h"
+#include "base/files/file_path.h"
+#include "base/files/file_util.h"
 #include "base/json/json_string_value_serializer.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/process/launch.h"
 #include "components/feedback/feedback_util.h"
 #include "content/public/browser/browser_thread.h"
 #include "ui/ozone/public/input_controller.h"
 #include "ui/ozone/public/ozone_platform.h"
 
 using content::BrowserThread;
 
 namespace {
 
 const char kHUDLogDataKey[] = "hud_log";
 
 // The prefix "hack-33025" was historically chosen in http://crbug.com/139715.
 // We continue to go with it in order to be compatible with the existing touch
 // log processing toolchain.
 const char kDeviceStatusLogDataKey[] = "hack-33025-touchpad";
+const char kTouchpadEventLogDataKey[] = "hack-33025-touchpad_activity";
 
-// Callback for handing the outcome of GetTouchDeviceStatus(). Appends the
-// collected log to the SystemLogsResponse map.
-void OnStatusLogCollected(scoped_ptr<system_logs::SystemLogsResponse> response,
-                          const system_logs::SysLogsSourceCallback& callback,
-                          scoped_ptr<std::string> log) {
+// Directory for temp touch event logs.
+const char kTouchEventLogDir[] = "/home/chronos/user/log";
+
+// Binary paths.
+const char kShellCommand[] = "/bin/sh";
+const char kTarCommand[] = "/bin/tar";
+const char kUuencodeCommand[] = "/usr/bin/uuencode";
+
+void CleanupEventLog(scoped_ptr<std::vector<base::FilePath>> log_paths) {
+  for (size_t i = 0; i < log_paths->size(); ++i)
+    base::DeleteFile((*log_paths)[i], false);
+}
+
+void PackEventLog(system_logs::SystemLogsResponse* response,
+                  scoped_ptr<std::vector<base::FilePath>> log_paths) {
+  // Combine logs with a command line call that tars them up and uuencode the
+  // result in one string. This is to be compatible with the X11 behavior.
+  std::vector<std::pair<std::string, base::CommandLine>> commands;
+  base::CommandLine command = base::CommandLine(base::FilePath(kShellCommand));
+  command.AppendArg("-c");
+  std::string log_dir(kTouchEventLogDir);

[spang, 2015/02/02 21:43:16]

Don't need this, you can concatenate a const char* directly.

[Shecky Lin, 2015/02/03 08:20:14]

It doesn't look like so. I tried it before and got a compile error. Just wrote a
small c++ file to verify that it really doesn't work.

[spang, 2015/02/03 16:31:20]

Ah, right, you're building a new string rather than appending onto a
std::string. One of the arguments to + must be a std::string to get the
overload.

+  std::string get_touchpad_gesture_log =
+      "ls -1t " + log_dir + "/touchpad_activity_* | head -n 7";
+  std::string get_touchpad_evdev_log =
+      "ls -1t " + log_dir + "/cmt_input_events_* | head -n 7";
+  command.AppendArg(std::string(kTarCommand) + " cf - $(" +
+                    get_touchpad_gesture_log + " ) $(" +
+                    get_touchpad_evdev_log + " ) 2>/dev/null | " +
+                    kUuencodeCommand + " -m touchpad_activity_log.tar");

[spang, 2015/02/02 21:43:16]

Can't this use log_paths instead of doing the $(ls ...) command substitution?

[Shecky Lin, 2015/02/03 08:20:14]

I will have to use two separate vectors in the callback to separate touchpad
logs and touchscreen logs because they will need to go to different files. Does
that sound like a better plan?

[spang, 2015/02/03 16:31:20]

Sure, I think we should really minimize use of shell scripting in here.

[Shecky Lin, 2015/02/10 08:21:22]

Done.

+  commands.push_back(std::make_pair(kTouchpadEventLogDataKey, command));
+
+  // For now only touchpad (and mouse) logs are actually collected.
+  for (size_t i = 0; i < commands.size(); ++i) {
+    std::string output;
+    base::GetAppOutput(commands[i].second, &output);
+    (*response)[commands[i].first] = output;
+  }
+
+  // Cleanup these temporary log files.
+  BrowserThread::PostBlockingPoolTask(
+      FROM_HERE, base::Bind(CleanupEventLog, base::Passed(&log_paths)));
+}
+
+void OnEventLogCollected(scoped_ptr<system_logs::SystemLogsResponse> response,
+                         const system_logs::SysLogsSourceCallback& callback,
+                         scoped_ptr<std::vector<base::FilePath>> log_paths) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  (*response)[kDeviceStatusLogDataKey] = *log;
 
-  BrowserThread::PostTask(
-      BrowserThread::UI, FROM_HERE,
-      base::Bind(callback, base::Owned(response.release())));
+  base::Closure pack_closure =
+      base::Bind(&PackEventLog, base::Unretained(response.get()),
+                 base::Passed(&log_paths));
+  base::Closure callback_closure =
+      base::Bind(callback, base::Owned(response.release()));
+  BrowserThread::PostBlockingPoolTaskAndReply(FROM_HERE, pack_closure,
+                                              callback_closure);
+}
+
+// Callback for handing the outcome of GetTouchDeviceStatus().
+//
+// Appends the collected log to the SystemLogsResponse map. Also goes on to
+// collect touch event logs.
+ void OnStatusLogCollected(scoped_ptr<system_logs::SystemLogsResponse> response,
+                           const system_logs::SysLogsSourceCallback& callback,
+                           scoped_ptr<std::string> log) {
+   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+   (*response)[kDeviceStatusLogDataKey] = *log;
+
+  // Collect touch event logs.
+  const base::FilePath kBaseLogPath(kTouchEventLogDir);
+  ui::OzonePlatform::GetInstance()->GetInputController()->GetTouchEventLog(
+      kBaseLogPath,
+      base::Bind(&OnEventLogCollected, base::Passed(&response), callback));
 }
 
 // Collect touch HUD debug logs. This needs to be done on the UI thread.
 void CollectTouchHudDebugLog(system_logs::SystemLogsResponse* response) {
   scoped_ptr<base::DictionaryValue> dictionary =
       ash::TouchHudDebug::GetAllAsDictionary();
   if (!dictionary->empty()) {
     std::string touch_log;
     JSONStringValueSerializer json(&touch_log);
     json.set_pretty_print(true);
@@ skipping 12 matching lines @@
 
   scoped_ptr<SystemLogsResponse> response(new SystemLogsResponse);
   CollectTouchHudDebugLog(response.get());
 
   // Collect touch device status logs.
   ui::OzonePlatform::GetInstance()->GetInputController()->GetTouchDeviceStatus(
       base::Bind(&OnStatusLogCollected, base::Passed(&response), callback));
 }
 
 }  // namespace system_logs