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

Side by Side Diff: android_webview/browser/aw_browser_terminator.h

Issue 2393853002: Refactor CrashDump*Manager to use a shared CrashDumpObserver singleton. (Closed)
Patch Set: fix initialization in ShellBrowserMainParts Created 4 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
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef COMPONENTS_CRASH_CONTENT_BROWSER_CRASH_DUMP_BROWSER_TERMINATOR_H_
6 #define COMPONENTS_CRASH_CONTENT_BROWSER_CRASH_DUMP_BROWSER_TERMINATOR_H_
7
8 #include <map>
9
10 #include "base/synchronization/lock.h"
11 #include "components/crash/content/browser/crash_dump_observer_android.h"
12
13 namespace base {
14 class SyncSocket;
15 }
16
17 namespace android_webview {
18
19 // This class manages behavior of the browser on renderer crashes when
20 // microdumps are used for capturing the crash stack. Normally, in this case
21 // the browser doesn't need to do much, because a microdump is written into
22 // Android log by the renderer process itself. However, the browser may need to
23 // crash itself on a renderer crash. Since on Android renderers are not child
24 // processes of the browser, it can't access the exit code. Instead, the browser
25 // uses a dedicated pipe in order to receive the information about the renderer
26 // crash status.
27 class AwBrowserTerminator : public breakpad::CrashDumpObserver::Client {
28 public:
29 AwBrowserTerminator();
30
31 // breakpad::CrashDumpObserver::Client implementation.
32 void OnChildStart(int child_process_id,
33 content::FileDescriptorInfo* mappings) override;
34 void OnChildExit(int child_process_id,
35 base::ProcessHandle pid,
36 content::ProcessType process_type,
37 base::TerminationStatus termination_status,
38 base::android::ApplicationState app_state) override;
39
40 private:
41 ~AwBrowserTerminator() override;
42
43 // This map should only be accessed with its lock aquired as it is accessed
44 // from the PROCESS_LAUNCHER, FILE, and UI threads.
45 base::Lock child_process_id_to_pipe_lock_;
46 std::map<int, std::unique_ptr<base::SyncSocket>> child_process_id_to_pipe_;
47
48 DISALLOW_COPY_AND_ASSIGN(AwBrowserTerminator);
49 };
50
51 } // namespace breakpad
52
53 #endif // COMPONENTS_CRASH_CONTENT_BROWSER_CRASH_DUMP_BROWSER_TERMINATOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698