Move CrashService class to breakpad component

components/breakpad/tools/crash_service.h

 // Copyright (c) 2011 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.
 
-#ifndef CHROME_TOOLS_CRASH_SERVICE_CRASH_SERVICE_H_
-#define CHROME_TOOLS_CRASH_SERVICE_CRASH_SERVICE_H_
+#ifndef COMPONENTS_BREAKPAD_TOOLS_CRASH_SERVICE_H_
+#define COMPONENTS_BREAKPAD_TOOLS_CRASH_SERVICE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/files/file_path.h"
 #include "base/synchronization/lock.h"
 
 namespace google_breakpad {
 
 class CrashReportSender;
 class CrashGenerationServer;
 class ClientInfo;
 
 }
 
+namespace breakpad {
+
 // This class implements an out-of-process crash server. It uses breakpad's
 // CrashGenerationServer and CrashReportSender to generate and then send the
 // crash dumps. Internally, it uses OS specific pipe to allow applications to
 // register for crash dumps and later on when a registered application crashes
 // it will signal an event that causes this code to wake up and perform a
 // crash dump on the signaling process. The dump is then stored on disk and
 // possibly sent to the crash2 servers.
 class CrashService {
  public:
-  // The ctor takes a directory that needs to be writable and will create
-  // a subdirectory inside to keep logs, crashes and checkpoint files.
-  explicit CrashService(const std::wstring& report_dir);
+  explicit CrashService();

[Robert Sesek, 2013/10/17 13:16:05]

No |explicit|

[jochen (gone - plz use gerrit), 2013/10/17 16:47:39]

Done.

   ~CrashService();
 
-  // Starts servicing crash dumps. The command_line specifies various behaviors,
-  // see below for more information. Returns false if it failed. Do not use
+  // Starts servicing crash dumps. Returns false if it failed. Do not use
   // other members in that case.
-  bool Initialize(const std::wstring& command_line);
+  bool Initialize(const base::FilePath& operations_dir,

[Robert Sesek, 2013/10/17 13:16:05]

I don't know what an |operations_dir| is… comment?

This is also called |operating_dir| in the .cc file.

[jochen (gone - plz use gerrit), 2013/10/17 16:47:39]

Done.

+                  const base::FilePath& report_path,
+                  const base::FilePath& dumps_path);
 
   // Command line switches:
   //
   // --max-reports=<number>
   // Allows to override the maximum number for reports per day. Normally
   // the crash dumps are never sent so if you want to send any you must
   // specify a positive number here.
   static const char kMaxReports[];
   // --no-window
   // Does not create a visible window on the desktop. The window does not have
@@ skipping 63 matching lines @@
   // clients serviced statistics:
   int requests_handled_;
   int requests_sent_;
   volatile long clients_connected_;
   volatile long clients_terminated_;
   base::Lock sending_;
 
   DISALLOW_COPY_AND_ASSIGN(CrashService);
 };
 
-#endif  // CHROME_TOOLS_CRASH_SERVICE_CRASH_SERVICE_H_
+}  // namespace breakpad
+
+#endif  // COMPONENTS_BREAKPAD_TOOLS_CRASH_SERVICE_H_