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

Side by Side Diff: handler/win/crash_report_exception_handler.h

Issue 1314093002: Refactor handler/main for Windows, implement CrashHandlerExceptionServer (Closed) Base URL: https://chromium.googlesource.com/crashpad/crashpad@crash-handler-exe
Patch Set: fixes2 Created 5 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
« no previous file with comments | « handler/main.cc ('k') | handler/win/crash_report_exception_handler.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 2015 The Crashpad Authors. All rights reserved. 1 // Copyright 2015 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,
11 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 11 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12 // See the License for the specific language governing permissions and 12 // See the License for the specific language governing permissions and
13 // limitations under the License. 13 // limitations under the License.
14 14
15 #ifndef CRASHPAD_HANDLER_MAC_CRASH_REPORT_EXCEPTION_HANDLER_H_ 15 #ifndef CRASHPAD_HANDLER_WIN_CRASH_REPORT_EXCEPTION_HANDLER_H_
16 #define CRASHPAD_HANDLER_MAC_CRASH_REPORT_EXCEPTION_HANDLER_H_ 16 #define CRASHPAD_HANDLER_WIN_CRASH_REPORT_EXCEPTION_HANDLER_H_
17 17
18 #include <mach/mach.h> 18 #include <windows.h>
19 19
20 #include <map> 20 #include <map>
21 #include <string> 21 #include <string>
22 22
23 #include "base/basictypes.h" 23 #include "util/win/exception_handler_server.h"
24 #include "client/crash_report_database.h"
25 #include "handler/crash_report_upload_thread.h"
26 #include "util/mach/exc_server_variants.h"
27 24
28 namespace crashpad { 25 namespace crashpad {
29 26
27 class CrashReportDatabase;
28 class CrashReportUploadThread;
29
30 //! \brief An exception handler that writes crash reports for exception messages 30 //! \brief An exception handler that writes crash reports for exception messages
31 //! to a CrashReportDatabase. 31 //! to a CrashReportDatabase.
32 class CrashReportExceptionHandler : public UniversalMachExcServer::Interface { 32 class CrashReportExceptionHandler : public ExceptionHandlerServer::Delegate {
33 public: 33 public:
34 //! \brief Creates a new object that will store crash reports in \a database. 34 //! \brief Creates a new object that will store crash reports in \a database.
35 //! 35 //!
36 //! \param[in] database The database to store crash reports in. Weak. 36 //! \param[in] database The database to store crash reports in. Weak.
37 //! \param[in] upload_thread The upload thread to notify when a new crash 37 //! \param[in] upload_thread The upload thread to notify when a new crash
38 //! report is written into \a database. 38 //! report is written into \a database.
39 //! \param[in] process_annotations A map of annotations to insert as 39 //! \param[in] process_annotations A map of annotations to insert as
40 //! process-level annotations into each crash report that is written. Do 40 //! process-level annotations into each crash report that is written. Do
41 //! not confuse this with module-level annotations, which are under the 41 //! not confuse this with module-level annotations, which are under the
42 //! control of the crashing process, and are used to implement Chromes 42 //! control of the crashing process, and are used to implement Chrome's
43 //! crash keys. Process-level annotations are those that are beyond the 43 //! "crash keys." Process-level annotations are those that are beyond the
44 //! control of the crashing process, which must reliably be set even if 44 //! control of the crashing process, which must reliably be set even if
45 //! the process crashes before its able to establish its own annotations. 45 //! the process crashes before it's able to establish its own annotations.
46 //! To interoperate with Breakpad servers, the recommended practice is to 46 //! To interoperate with Breakpad servers, the recommended practice is to
47 //! specify values for the `"prod"` and `"ver"` keys as process 47 //! specify values for the `"prod"` and `"ver"` keys as process
48 //! annotations. 48 //! annotations.
49 CrashReportExceptionHandler( 49 CrashReportExceptionHandler(
50 CrashReportDatabase* database, 50 CrashReportDatabase* database,
51 CrashReportUploadThread* upload_thread, 51 CrashReportUploadThread* upload_thread,
52 const std::map<std::string, std::string>* process_annotations); 52 const std::map<std::string, std::string>* process_annotations);
53 53
54 ~CrashReportExceptionHandler(); 54 ~CrashReportExceptionHandler() override;
55 55
56 // UniversalMachExcServer::Interface: 56 // ExceptionHandlerServer::Delegate:
57 57
58 //! \brief Processes an exception message by writing a crash report to this 58 //! \brief Processes an exception message by writing a crash report to this
59 //! object’s CrashReportDatabase. 59 //! object's CrashReportDatabase.
60 kern_return_t CatchMachException( 60 void ExceptionHandlerServerStarted() override;
61 exception_behavior_t behavior, 61 unsigned int ExceptionHandlerServerException(
62 exception_handler_t exception_port, 62 HANDLE process,
63 thread_t thread, 63 WinVMAddress exception_information_address) override;
64 task_t task,
65 exception_type_t exception,
66 const mach_exception_data_type_t* code,
67 mach_msg_type_number_t code_count,
68 thread_state_flavor_t* flavor,
69 ConstThreadState old_state,
70 mach_msg_type_number_t old_state_count,
71 thread_state_t new_state,
72 mach_msg_type_number_t* new_state_count,
73 const mach_msg_trailer_t* trailer,
74 bool* destroy_complex_request) override;
75 64
76 private: 65 private:
77 CrashReportDatabase* database_; // weak 66 CrashReportDatabase* database_; // weak
78 CrashReportUploadThread* upload_thread_; // weak 67 CrashReportUploadThread* upload_thread_; // weak
79 const std::map<std::string, std::string>* process_annotations_; // weak 68 const std::map<std::string, std::string>* process_annotations_; // weak
80 69
81 DISALLOW_COPY_AND_ASSIGN(CrashReportExceptionHandler); 70 DISALLOW_COPY_AND_ASSIGN(CrashReportExceptionHandler);
82 }; 71 };
83 72
84 } // namespace crashpad 73 } // namespace crashpad
85 74
86 #endif // CRASHPAD_HANDLER_MAC_CRASH_REPORT_EXCEPTION_HANDLER_H_ 75 #endif // CRASHPAD_HANDLER_WIN_CRASH_REPORT_EXCEPTION_HANDLER_H_
OLDNEW
« no previous file with comments | « handler/main.cc ('k') | handler/win/crash_report_exception_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698