Mac: Fix missing safe_browsing dependency on ntp_background_proto

Fixes the below build error with macOS official builds.

FAILED: obj/chrome/browser/safe_browsing/safe_browsing/settings_reset_prompt_model.o
In file included from ../../chrome/browser/safe_browsing/settings_reset_prompt/settings_reset_prompt_model.cc:5:
In file included from ../../chrome/browser/safe_browsing/settings_reset_prompt/settings_reset_prompt_model.h:19:
In file included from ../../chrome/browser/profile_resetter/profile_resetter.h:22:
In file included from ../../chrome/browser/search/instant_service.h:20:
In file included from ../../chrome/browser/search/background/ntp_background_service.h:15:
../../chrome/browser/search/background/ntp_background_data.h:10:10: fatal error: 'chrome/browser/search/background/ntp_background.pb.h' file not found
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
This commit is contained in:
Marshall Greenblatt
2020-10-29 14:44:32 -04:00
parent 9c5fca54c5
commit f2039ae1e6
2 changed files with 11 additions and 3 deletions

View File

@@ -256,7 +256,7 @@ patches = [
'name': 'chrome_browser_product_override',
},
{
# Fix Jumbo/component build dependency issue.
# Fix build dependency issues.
'name': 'chrome_browser_safe_browsing',
},
{

View File

@@ -1,8 +1,16 @@
diff --git chrome/browser/safe_browsing/BUILD.gn chrome/browser/safe_browsing/BUILD.gn
index 7f54cacab005..b4f509b766ca 100644
index 7f54cacab005..db469e514100 100644
--- chrome/browser/safe_browsing/BUILD.gn
+++ chrome/browser/safe_browsing/BUILD.gn
@@ -251,6 +251,7 @@ static_library("safe_browsing") {
@@ -17,6 +17,7 @@ static_library("safe_browsing") {
deps = [
"//chrome/app:generated_resources",
+ "//chrome/browser:ntp_background_proto",
"//chrome/common",
"//chrome/common:constants",
"//components/browser_sync",
@@ -251,6 +252,7 @@ static_library("safe_browsing") {
"//chrome/common/safe_browsing:download_type_util",
"//chrome/services/file_util/public/cpp",
"//components/content_settings/core/browser",