Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(245)

Side by Side Diff: util/test/multiprocess_exec.h

Issue 531203002: Add the MultiprocessExec test and its test (Closed) Base URL: https://chromium.googlesource.com/crashpad/crashpad@master
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2014 The Crashpad Authors. All rights reserved.
2 //
3 // Licensed under the Apache License, Version 2.0 (the "License");
4 // you may not use this file except in compliance with the License.
5 // You may obtain a copy of the License at
6 //
7 // http://www.apache.org/licenses/LICENSE-2.0
8 //
9 // Unless required by applicable law or agreed to in writing, software
10 // distributed under the License is distributed on an "AS IS" BASIS,
11 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12 // See the License for the specific language governing permissions and
13 // limitations under the License.
14
15 #ifndef CRASHPAD_UTIL_TEST_MULTIPROCESS_EXEC_H_
16 #define CRASHPAD_UTIL_TEST_MULTIPROCESS_EXEC_H_
17
18 #include <string>
19 #include <vector>
20
21 #include "base/basictypes.h"
22 #include "util/test/multiprocess.h"
23
24 namespace crashpad {
25 namespace test {
26
27 //! \brief Manages an `exec()`-based multiprocess test.
28 //!
29 //! These tests are based on `fork()` and `exec()`. The parent process is able
30 //! to communicate with the child in the same manner as a base-class
31 //! Multiprocess parent. The read and write pipes appear in the child process on
32 //! stdin and stdout, respectively.
33 //!
34 //! Subclasses are expected to implement the parent in the same was as a
35 //! base-class Multiprocess parent. The child must be implemented in an
36 //! executable to be set by SetChildCommand().
37 class MultiprocessExec : public Multiprocess {
Robert Sesek 2014/09/02 21:45:15 Will there be no need for a MachMultiprocesseExec?
Mark Mentovai 2014/09/02 22:47:47 rsesek wrote:
Robert Sesek 2014/09/03 20:56:24 Right, I figured this required some refactoring to
38 public:
39 MultiprocessExec();
40
41 //! \brief Sets the command to `exec()` in the child.
Robert Sesek 2014/09/02 21:45:15 Do I have to call this? What if I don't?
42 //!
43 //! \param[in] command The executable’s pathname.
44 //! \param[in] arguments The command-line arguments to pass to the child
45 //! process in its `argv[]` vector. This vector must begin at `argv[1]`,
46 //! as \a command is implicitly used as `argv[0]`. This argument may be
47 //! `NULL` if no command-line arguments are to be passed.
48 void SetChildCommand(const std::string& command,
49 const std::vector<std::string>* arguments);
50
51 protected:
52 ~MultiprocessExec();
53
54 // Multiprocess:
55 virtual void PreFork() override;
56
57 private:
58 // Multiprocess:
59 virtual void MultiprocessChild() override;
60
61 std::string command_;
62 std::vector<std::string> arguments_;
63 std::vector<const char*> argv_;
64
65 DISALLOW_COPY_AND_ASSIGN(MultiprocessExec);
66 };
67
68 } // namespace test
69 } // namespace crashpad
70
71 #endif // CRASHPAD_UTIL_TEST_MULTIPROCESS_EXEC_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698