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

Side by Side Diff: client/simulate_crash_mac.h

Issue 1368023004: Use references instead of pointers for CPU context in SimulateCrash (Closed) Base URL: https://chromium.googlesource.com/crashpad/crashpad@master
Patch Set: Created 5 years, 2 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
« no previous file with comments | « no previous file | client/simulate_crash_mac.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Crashpad Authors. All rights reserved. 1 // Copyright 2014 The Crashpad Authors. All rights reserved.
2 // 2 //
3 // Licensed under the Apache License, Version 2.0 (the "License"); 3 // Licensed under the Apache License, Version 2.0 (the "License");
4 // you may not use this file except in compliance with 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 5 // You may obtain a copy of the License at
6 // 6 //
7 // http://www.apache.org/licenses/LICENSE-2.0 7 // http://www.apache.org/licenses/LICENSE-2.0
8 // 8 //
9 // Unless required by applicable law or agreed to in writing, software 9 // Unless required by applicable law or agreed to in writing, software
10 // distributed under the License is distributed on an "AS IS" BASIS, 10 // distributed under the License is distributed on an "AS IS" BASIS,
(...skipping 27 matching lines...) Expand all
38 //! This function returns normally after the exception message is processed. If 38 //! This function returns normally after the exception message is processed. If
39 //! no valid handler was found, or no handler processed the exception 39 //! no valid handler was found, or no handler processed the exception
40 //! successfully, a warning will be logged, but these conditions are not 40 //! successfully, a warning will be logged, but these conditions are not
41 //! considered fatal. 41 //! considered fatal.
42 //! 42 //!
43 //! \param[in] cpu_context The thread state to pass to the exception handler as 43 //! \param[in] cpu_context The thread state to pass to the exception handler as
44 //! the exception context, provided that it is compatible with the thread 44 //! the exception context, provided that it is compatible with the thread
45 //! state flavor that the exception handler accepts. If it is not 45 //! state flavor that the exception handler accepts. If it is not
46 //! compatible, the correct thread state for the handler will be obtained by 46 //! compatible, the correct thread state for the handler will be obtained by
47 //! calling `thread_get_state()`. 47 //! calling `thread_get_state()`.
48 void SimulateCrash(const NativeCPUContext* cpu_context); 48 void SimulateCrash(const NativeCPUContext& cpu_context);
49 49
50 } // namespace crashpad 50 } // namespace crashpad
51 51
52 //! \brief Captures the CPU context and simulates an exception without crashing. 52 //! \brief Captures the CPU context and simulates an exception without crashing.
53 #define CRASHPAD_SIMULATE_CRASH() \ 53 #define CRASHPAD_SIMULATE_CRASH() \
54 do { \ 54 do { \
55 crashpad::NativeCPUContext cpu_context; \ 55 crashpad::NativeCPUContext cpu_context; \
56 crashpad::CaptureContext(&cpu_context); \ 56 crashpad::CaptureContext(&cpu_context); \
57 crashpad::SimulateCrash(&cpu_context); \ 57 crashpad::SimulateCrash(cpu_context); \
58 } while (false) 58 } while (false)
59 59
60 #endif // CRASHPAD_CLIENT_SIMULATE_CRASH_MAC_H_ 60 #endif // CRASHPAD_CLIENT_SIMULATE_CRASH_MAC_H_
OLDNEW
« no previous file with comments | « no previous file | client/simulate_crash_mac.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698