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

Unified Diff: components/crash/content/app/BUILD.gn

Issue 1423043002: Fix missing dependencies for //components/crash. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: double-sigh. Add v8 config to content_descriptors target 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « .gn ('k') | components/crash/content/browser/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/crash/content/app/BUILD.gn
diff --git a/components/crash/content/app/BUILD.gn b/components/crash/content/app/BUILD.gn
index 315c28194ea3f0528ccd3b53636798fe8ea87180..d43804b7cff63498012a0b1d4ea5b8bbe7865338 100644
--- a/components/crash/content/app/BUILD.gn
+++ b/components/crash/content/app/BUILD.gn
@@ -15,12 +15,16 @@ source_set("lib") {
]
include_dirs = [ "../../../../breakpad/src" ]
+
+ deps = [
+ "//base",
+ ]
+
+ if (is_win) {
+ deps += [ "//breakpad:breakpad_handler" ]
+ }
}
-# Note: if you depend on this target, you need to either link in
-# content.gyp:content_common, or add content/public/common/content_switches.cc
-# to your sources.
-#
# GYP version: components/crash.gypi:crash_component
source_set("app") {
sources = [
@@ -38,7 +42,10 @@ source_set("app") {
]
if (is_mac) {
- deps += [ "//third_party/crashpad/crashpad/client" ]
+ deps += [
+ ":lib",
+ "//third_party/crashpad/crashpad/client",
+ ]
}
}
@@ -65,7 +72,10 @@ source_set("app") {
# all platforms, or preferably, depend on crash_component to get Breakpad
# everywhere except for Mac, where you will get Crashpad.
source_set("app_non_mac") {
- visibility = [ ":*" ]
+ visibility = [
+ ":*",
+ "//components/crash/content/browser",
+ ]
sources = [
"breakpad_linux_impl.h",
@@ -94,6 +104,9 @@ source_set("app_non_mac") {
":lib",
"//base",
"//base:base_static",
+ "//components/crash/core/common",
+ "//content/public/common:content_descriptors",
+ "//content/public/common:result_codes",
]
if (is_android) {
@@ -133,10 +146,12 @@ source_set("app_breakpad_mac_to_be_deleted") {
defines = [ "CRASH_IMPLEMENTATION" ]
deps += [
+ ":lib",
"//base",
"//base:base_static",
"//breakpad",
"//breakpad:client",
+ "//components/crash/core/common",
]
}
}
« no previous file with comments | « .gn ('k') | components/crash/content/browser/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698