Refactoring the SysInfoProvider.

chrome/browser/chromeos/system/commandline_fetcher.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 "chrome/browser/chromeos/system/commandline_fetcher.h"
+
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/command_line.h"
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/logging.h"
+#include "base/string_util.h"
+
+namespace chromeos {
+namespace system {
+
+void CommandLineFetcher::Fetch(SysLogsFetcherCallback request)
+{
+    FilePath temp_filename;
+    SysInfoResponse* response = new SysInfoResponse;
+    if (!file_util::CreateTemporaryFile(&temp_filename))

[rkc1, 2012/08/04 01:15:05]

Change this to use base::GetAppOutput.

[tudalex(Chromium), 2012/08/05 01:15:12]

Done.

+    {
+      request.Run(response);
+      return;
+    }
+    std::string cmd = command_ + " >> " +
+        temp_filename.value();
+
+    // Ignore the return value - if the script execution didn't work
+    // stderr won't go into the output file anyway.
+    if (::system(cmd.c_str()) == -1)
+      LOG(WARNING) << "Command " << cmd << " failed to run";
+    // Read logs from the temp file
+    std::string data;
+    bool read_success = file_util::ReadFileToString(temp_filename,
+                                                    &data);
+    // if we were using an internal temp file, the user does not need the
+    // logs to stay past the ReadFile call - delete the file
+    if (!read_success)
+    {
+      request.Run(response);
+      return;
+    }
+    file_util::Delete(temp_filename, false);
+    (*response)[key_] = data;
+    request.Run(response);
+}
+
+}  // namespace system
+}  // namespace chromeos