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

Unified Diff: chrome_elf/BUILD.gn

Issue 2123073002: Switch chrome_elf exception handling from breakpad to crashpad (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased to tip and git cl format Created 4 years, 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/install_static/install_util.cc ('k') | chrome_elf/DEPS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_elf/BUILD.gn
diff --git a/chrome_elf/BUILD.gn b/chrome_elf/BUILD.gn
index b11085621a95def88d6284a857399e8d8d44457a..5cc27bdf330d2cf70d2459ad86c098a2e294b632 100644
--- a/chrome_elf/BUILD.gn
+++ b/chrome_elf/BUILD.gn
@@ -42,10 +42,10 @@ shared_library("chrome_elf") {
]
deps = [
":blacklist",
- ":breakpad",
":chrome_elf_manifest",
":chrome_elf_resources",
":chrome_elf_security",
+ ":constants",
":hook_util",
"//base",
"//build/config/sanitizers:deps",
@@ -54,6 +54,7 @@ shared_library("chrome_elf") {
"//components/crash/content/app",
"//components/crash/core/common",
"//content/public/common:result_codes",
+ "//third_party/crashpad/crashpad/client:client",
]
configs += [ "//build/config/win:windowed" ]
configs -= [ "//build/config/win:console" ]
@@ -128,12 +129,13 @@ static_library("blacklist") {
"blacklist/blacklist.h",
"blacklist/blacklist_interceptions.cc",
"blacklist/blacklist_interceptions.h",
+ "blacklist/crashpad_helper.cc",
+ "blacklist/crashpad_helper.h",
]
public_deps = [
"//sandbox",
]
deps = [
- ":breakpad",
":constants",
":hook_util",
"//chrome/install_static:install_static_util",
@@ -141,20 +143,7 @@ static_library("blacklist") {
# Still uses base/win/pe_image.h
"//base",
- ]
-}
-
-static_library("breakpad") {
- include_dirs = [ "$target_gen_dir" ]
- sources = [
- "breakpad/breakpad.cc",
- "breakpad/breakpad.h",
- ]
- deps = [
- "//breakpad:breakpad_handler",
- "//chrome/common:version_header",
- "//chrome/install_static:install_static_util",
- "//chrome_elf/nt_registry:nt_registry",
+ "//third_party/crashpad/crashpad/client:client",
]
}
@@ -236,6 +225,18 @@ shared_library("blacklist_test_main_dll") {
"//chrome/install_static:install_static_util",
"//chrome_elf/nt_registry:nt_registry",
]
+ ldflags = [
+ "/NODEFAULTLIB:user32.lib",
+ "/DELAYLOAD:dbghelp.dll",
+ "/DELAYLOAD:ole32.dll",
+ "/DELAYLOAD:psapi.dll",
+ "/DELAYLOAD:rpcrt4.dll",
+ "/DELAYLOAD:shell32.dll",
+ "/DELAYLOAD:user32.dll",
+ "/DELAYLOAD:winhttp.dll",
+ "/DELAYLOAD:winmm.dll",
+ "/DELAYLOAD:ws2_32.dll",
+ ]
}
loadable_module("blacklist_test_dll_1") {
« no previous file with comments | « chrome/install_static/install_util.cc ('k') | chrome_elf/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698