Skip to content

GitLab

T
tangled
Commit a6085a6a authored by agrieve's avatar agrieve Committed by Commit bot
Browse files
Options

Reland #2 of Enable whitelisting for targets other than libchrome. 

Reverted in
https://codereview.chromium.org/2342243003/

Reason for reland:
Actually fixed gn gen failure.

This will allow monochrome to easily define its own set of the same
targets with a different resource whitelist

TBR=brettw@chromium.org,miu@chromium.org,yutak@chromium.org
BUG=641032

Review-Url: https://codereview.chromium.org/2342023004
Cr-Commit-Position: refs/heads/master@{#419209}
parent 92118751
Hide whitespace changes
Inline Side-by-side
Showing with 257 additions and 217 deletions
chrome/BUILD.gn
View file @ a6085a6a
...@@ -11,17 +11,13 @@ import("//build/config/sanitizers/sanitizers.gni") ...@@ -11,17 +11,13 @@ import("//build/config/sanitizers/sanitizers.gni")
import("//build/config/ui.gni") import("//build/config/ui.gni")
import("//build/config/win/console_app.gni") import("//build/config/win/console_app.gni")
import("//build/config/win/manifest.gni") import("//build/config/win/manifest.gni")
import("//chrome/chrome_repack_locales.gni") import("//chrome/chrome_paks.gni")
import("//chrome/common/features.gni") import("//chrome/common/features.gni")
import("//chrome/version.gni") import("//chrome/version.gni")
import("//third_party/widevine/cdm/widevine.gni") import("//third_party/widevine/cdm/widevine.gni")
import("//ui/base/ui_features.gni") import("//ui/base/ui_features.gni")
import("//v8/gni/v8.gni") import("//v8/gni/v8.gni")
if (enable_resource_whitelist_generation) {
android_resource_whitelist = "$root_gen_dir/chrome/resource_whitelist.txt"
}
if (is_android) { if (is_android) {
import("//build/config/android/rules.gni") import("//build/config/android/rules.gni")
} else if (is_mac) { } else if (is_mac) {
...@@ -36,6 +32,10 @@ if (is_android) { ...@@ -36,6 +32,10 @@ if (is_android) {
import("//third_party/icu/config.gni") import("//third_party/icu/config.gni")
} }
if (enable_resource_whitelist_generation) {
chrome_resource_whitelist = "$target_gen_dir/chrome_resource_whitelist.txt"
}
if (is_win) { if (is_win) {
action("reorder_imports") { action("reorder_imports") {
script = "//build/win/reorder-imports.py" script = "//build/win/reorder-imports.py"
...@@ -996,7 +996,7 @@ if (is_win) { ...@@ -996,7 +996,7 @@ if (is_win) {
":chrome_framework_helpers", ":chrome_framework_helpers",
":chrome_framework_plugins", ":chrome_framework_plugins",
":chrome_framework_resources", ":chrome_framework_resources",
":repack_locales_pack", ":packed_resources",
":widevine_cdm_library", ":widevine_cdm_library",
"//build/config/sanitizers:deps", "//build/config/sanitizers:deps",
"//chrome/app/nibs:chrome_xibs", "//chrome/app/nibs:chrome_xibs",
...@@ -1339,100 +1339,42 @@ if (is_chrome_branded && !is_android) { ...@@ -1339,100 +1339,42 @@ if (is_chrome_branded && !is_android) {
} }
} }
group("packed_resources") { chrome_paks("packed_resources") {
public_deps = [ if (is_mac) {
":repack_chrome_100_percent", output_dir = "$root_gen_dir/repack"
":repack_locales_pack", copy_data_to_bundle = true
] } else {
output_dir = root_out_dir
if (is_chrome_branded && !is_mac && !is_android) {
public_deps += [ ":default_apps" ]
}
if (enable_hidpi) {
public_deps += [ ":repack_chrome_200_percent" ]
} }
}
repack("packed_extra_resources") {
visibility = [ "./*" ]
sources = [
"$root_gen_dir/blink/public/resources/blink_resources.pak",
"$root_gen_dir/chrome/browser_resources.pak",
"$root_gen_dir/chrome/chrome_unscaled_resources.pak",
"$root_gen_dir/chrome/common_resources.pak",
"$root_gen_dir/chrome/invalidations_resources.pak",
"$root_gen_dir/chrome/net_internals_resources.pak",
"$root_gen_dir/chrome/password_manager_internals_resources.pak",
"$root_gen_dir/chrome/policy_resources.pak",
"$root_gen_dir/chrome/translate_internals_resources.pak",
"$root_gen_dir/components/components_resources.pak",
"$root_gen_dir/content/browser/tracing/tracing_resources.pak",
"$root_gen_dir/content/content_resources.pak",
"$root_gen_dir/net/net_resources.pak",
"$root_gen_dir/ui/resources/webui_resources.pak",
]
deps = [
"//chrome/app/theme:chrome_unscaled_resources",
"//chrome/browser:resources",
"//chrome/browser/resources:invalidations_resources",
"//chrome/browser/resources:net_internals_resources",
"//chrome/browser/resources:password_manager_internals_resources",
"//chrome/browser/resources:policy_resources",
"//chrome/browser/resources:translate_internals_resources",
"//chrome/common:resources",
"//components/resources",
"//content:resources",
"//content/browser/tracing:resources",
"//net:net_resources",
"//third_party/WebKit/public:resources",
"//ui/resources",
]
if (is_android) { if (enable_resource_whitelist_generation) {
sources += [ "$root_gen_dir/android_webview/aw_resources.pak" ] repack_whitelist = chrome_resource_whitelist
deps += [ "//android_webview:generate_aw_resources" ] deps = [
} else { # Non-Android. "//chrome:resource_whitelist",
# New paks should be added here by default.
sources += [
"$root_gen_dir/blink/devtools_resources.pak",
"$root_gen_dir/chrome/component_extension_resources.pak",
"$root_gen_dir/chrome/options_resources.pak",
"$root_gen_dir/chrome/quota_internals_resources.pak",
"$root_gen_dir/chrome/settings_resources.pak",
"$root_gen_dir/chrome/sync_file_system_internals_resources.pak",
]
deps += [
"//chrome/browser/resources:component_extension_resources",
"//chrome/browser/resources:options_resources",
"//chrome/browser/resources:quota_internals_resources",
"//chrome/browser/resources:settings_resources",
"//chrome/browser/resources:sync_file_system_internals_resources",
"//content/browser/devtools:devtools_resources",
] ]
} }
if (is_chromeos) {
sources += [ "$root_gen_dir/ui/file_manager/file_manager_resources.pak" ] if (is_chrome_branded && !is_mac && !is_android) {
deps += [ "//ui/file_manager:resources" ] public_deps = [
} ":default_apps",
if (enable_extensions) {
sources += [
"$root_gen_dir/extensions/extensions_renderer_resources.pak",
"$root_gen_dir/extensions/extensions_resources.pak",
] ]
deps += [ "//extensions:extensions_resources" ]
} }
}
chrome_extra_paks("packed_extra_resources") {
if (is_mac) { if (is_mac) {
output = "$root_gen_dir/repack/resources.pak" output_dir = "$root_gen_dir/repack"
copy_data_to_bundle = true
} else { } else {
output = "$root_out_dir/resources.pak" output_dir = root_out_dir
} }
copy_data_to_bundle = is_mac copy_data_to_bundle = is_mac
if (enable_resource_whitelist_generation) { if (enable_resource_whitelist_generation) {
repack_whitelist = android_resource_whitelist repack_whitelist = chrome_resource_whitelist
deps += [ ":resource_whitelist" ] deps = [
":resource_whitelist",
]
} }
} }
...@@ -1449,127 +1391,6 @@ repack("browser_tests_pak") { ...@@ -1449,127 +1391,6 @@ repack("browser_tests_pak") {
] ]
} }
# Collects per-locale grit files from many sources into global per-locale files.
chrome_repack_locales("repack_locales_pack") {
visibility = [ ":*" ]
input_locales = locales
copy_data_to_bundle = is_mac
if (is_mac) {
output_locales = locales_as_mac_outputs
} else {
output_locales = locales
}
if (enable_resource_whitelist_generation) {
repack_whitelist = android_resource_whitelist
deps = [
":resource_whitelist",
]
}
}
# Generates a rule to repack a set of resources, substituting a given string
# in for the percentage (e.g. "100", "200"). It generates the repacked files in
# the "gen" directory, and then introduces a copy rule to copy it to the root
# build directory.
#
# It's not clear why this two-step dance is necessary as opposed to just
# generating the file in the destination. However, this is what the GYP build
# does, and for maintenance purposes, this keeps the same files in the same
# place between the two builds when possible.
#
# Argument:
# percent [required]
# String to substitute for the percentage.
template("chrome_repack_percent") {
percent = invoker.percent
repack_name = "${target_name}_repack"
repack_output_file = "$root_gen_dir/repack/chrome_${percent}_percent.pak"
copy_name = target_name
repack(repack_name) {
visibility = [ ":$copy_name" ]
# All sources should also have deps for completeness.
sources = [
"$root_gen_dir/blink/public/resources/blink_image_resources_${percent}_percent.pak",
"$root_gen_dir/chrome/renderer_resources_${percent}_percent.pak",
"$root_gen_dir/chrome/theme_resources_${percent}_percent.pak",
"$root_gen_dir/components/components_resources_${percent}_percent.pak",
"$root_gen_dir/content/app/resources/content_resources_${percent}_percent.pak",
"$root_gen_dir/ui/resources/ui_resources_${percent}_percent.pak",
]
deps = [
"//chrome/app/theme:theme_resources",
"//chrome/renderer:resources",
"//components/resources",
"//components/strings",
"//content/app/resources",
"//net:net_resources",
"//third_party/WebKit/public:image_resources",
"//ui/resources",
]
if (use_ash) {
sources +=
[ "$root_gen_dir/ash/resources/ash_resources_${percent}_percent.pak" ]
deps += [ "//ash/resources" ]
}
if (toolkit_views) {
sources += [ "$root_gen_dir/ui/views/resources/views_resources_${percent}_percent.pak" ]
deps += [ "//ui/views/resources" ]
}
if (is_chromeos) {
sources += [ "$root_gen_dir/ui/chromeos/resources/ui_chromeos_resources_${percent}_percent.pak" ]
deps += [ "//ui/chromeos/resources" ]
}
if (enable_extensions) {
sources += [ "$root_gen_dir/extensions/extensions_browser_resources_${percent}_percent.pak" ]
deps += [ "//extensions:extensions_browser_resources" ]
}
if (enable_app_list) {
sources += [ "$root_gen_dir/ui/app_list/resources/app_list_resources_${percent}_percent.pak" ]
deps += [ "//ui/app_list/resources" ]
}
output = repack_output_file
copy_data_to_bundle = is_mac
if (enable_resource_whitelist_generation) {
repack_whitelist = android_resource_whitelist
deps += [ ":resource_whitelist" ]
}
}
copy(copy_name) {
visibility = [ ":*" ]
deps = [
":$repack_name",
]
sources = [
repack_output_file,
]
outputs = [
"$root_build_dir/chrome_${percent}_percent.pak",
]
}
}
chrome_repack_percent("repack_chrome_100_percent") {
percent = "100"
}
if (enable_hidpi) {
chrome_repack_percent("repack_chrome_200_percent") {
percent = "200"
}
}
# GYP version: chrome/chrome_resources.gyp:chrome_strings # GYP version: chrome/chrome_resources.gyp:chrome_strings
group("strings") { group("strings") {
public_deps = [ public_deps = [
...@@ -1659,6 +1480,9 @@ if (is_android) { ...@@ -1659,6 +1480,9 @@ if (is_android) {
if (enable_resource_whitelist_generation) { if (enable_resource_whitelist_generation) {
action("resource_whitelist") { action("resource_whitelist") {
assert(is_android,
"Resource whitelist currently implemented only on Android")
deps = [ deps = [
"//chrome/android:chrome", "//chrome/android:chrome",
] ]
...@@ -1669,7 +1493,7 @@ if (enable_resource_whitelist_generation) { ...@@ -1669,7 +1493,7 @@ if (enable_resource_whitelist_generation) {
_infile, _infile,
] ]
_outfile = android_resource_whitelist _outfile = chrome_resource_whitelist
outputs = [ outputs = [
_outfile, _outfile,
] ]
......
chrome/chrome_paks.gni 0 → 100644
View file @ a6085a6a
# Copyright 2016 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
import("//build/config/locales.gni")
import("chrome_repack_locales.gni")
# Generates a rule to repack a set of resources, substituting a given string
# in for the percentage (e.g. "100", "200"). It generates the repacked files in
# the "gen" directory, and then introduces a copy rule to copy it to the root
# build directory.
#
# Argument:
# percent [required]
# String to substitute for the percentage.
template("chrome_repack_percent") {
percent = invoker.percent
repack(target_name) {
forward_variables_from(invoker,
[
"copy_data_to_bundle",
"repack_whitelist",
"visibility",
])
# All sources should also have deps for completeness.
sources = [
"$root_gen_dir/blink/public/resources/blink_image_resources_${percent}_percent.pak",
"$root_gen_dir/chrome/renderer_resources_${percent}_percent.pak",
"$root_gen_dir/chrome/theme_resources_${percent}_percent.pak",
"$root_gen_dir/components/components_resources_${percent}_percent.pak",
"$root_gen_dir/content/app/resources/content_resources_${percent}_percent.pak",
"$root_gen_dir/ui/resources/ui_resources_${percent}_percent.pak",
]
deps = [
"//chrome/app/theme:theme_resources",
"//chrome/renderer:resources",
"//components/resources",
"//components/strings",
"//content/app/resources",
"//net:net_resources",
"//third_party/WebKit/public:image_resources",
"//ui/resources",
]
if (defined(invoker.deps)) {
deps += invoker.deps
}
if (use_ash) {
sources +=
[ "$root_gen_dir/ash/resources/ash_resources_${percent}_percent.pak" ]
deps += [ "//ash/resources" ]
}
if (toolkit_views) {
sources += [ "$root_gen_dir/ui/views/resources/views_resources_${percent}_percent.pak" ]
deps += [ "//ui/views/resources" ]
}
if (is_chromeos) {
sources += [ "$root_gen_dir/ui/chromeos/resources/ui_chromeos_resources_${percent}_percent.pak" ]
deps += [ "//ui/chromeos/resources" ]
}
if (enable_extensions) {
sources += [ "$root_gen_dir/extensions/extensions_browser_resources_${percent}_percent.pak" ]
deps += [ "//extensions:extensions_browser_resources" ]
}
if (enable_app_list) {
sources += [ "$root_gen_dir/ui/app_list/resources/app_list_resources_${percent}_percent.pak" ]
deps += [ "//ui/app_list/resources" ]
}
output = "${invoker.output_dir}/chrome_${percent}_percent.pak"
}
}
# Paramters:
# output_dir:
#
template("chrome_paks") {
chrome_repack_percent("${target_name}_100_percent") {
percent = "100"
forward_variables_from(invoker,
[
"copy_data_to_bundle",
"deps",
"output_dir",
"repack_whitelist",
"visibility",
])
}
if (invoker.enable_hidpi) {
chrome_repack_percent("${target_name}_200_percent") {
percent = "200"
forward_variables_from(invoker,
[
"copy_data_to_bundle",
"deps",
"output_dir",
"repack_whitelist",
"visibility",
])
}
}
chrome_repack_locales("${target_name}_locales") {
forward_variables_from(invoker,
[
"copy_data_to_bundle",
"deps",
"repack_whitelist",
"visibility",
])
input_locales = locales
output_dir = "${invoker.output_dir}/locales"
if (is_mac) {
output_locales = locales_as_mac_outputs
} else {
output_locales = locales
}
}
group(target_name) {
forward_variables_from(invoker, [ "deps" ])
public_deps = [
":${target_name}_100_percent",
":${target_name}_locales",
]
if (invoker.enable_hidpi) {
public_deps += [ ":${target_name}_200_percent" ]
}
if (defined(invoker.public_deps)) {
public_deps += invoker.public_deps
}
}
}
template("chrome_extra_paks") {
repack(target_name) {
forward_variables_from(invoker,
[
"copy_data_to_bundle",
"repack_whitelist",
"visibility",
])
output = "${invoker.output_dir}/resources.pak"
sources = [
"$root_gen_dir/blink/public/resources/blink_resources.pak",
"$root_gen_dir/chrome/browser_resources.pak",
"$root_gen_dir/chrome/chrome_unscaled_resources.pak",
"$root_gen_dir/chrome/common_resources.pak",
"$root_gen_dir/chrome/invalidations_resources.pak",
"$root_gen_dir/chrome/net_internals_resources.pak",
"$root_gen_dir/chrome/password_manager_internals_resources.pak",
"$root_gen_dir/chrome/policy_resources.pak",
"$root_gen_dir/chrome/translate_internals_resources.pak",
"$root_gen_dir/components/components_resources.pak",
"$root_gen_dir/content/browser/tracing/tracing_resources.pak",
"$root_gen_dir/content/content_resources.pak",
"$root_gen_dir/net/net_resources.pak",
"$root_gen_dir/ui/resources/webui_resources.pak",
]
deps = [
"//chrome/app/theme:chrome_unscaled_resources",
"//chrome/browser:resources",
"//chrome/browser/resources:invalidations_resources",
"//chrome/browser/resources:net_internals_resources",
"//chrome/browser/resources:password_manager_internals_resources",
"//chrome/browser/resources:policy_resources",
"//chrome/browser/resources:translate_internals_resources",
"//chrome/common:resources",
"//components/resources",
"//content:resources",
"//content/browser/tracing:resources",
"//net:net_resources",
"//third_party/WebKit/public:resources",
"//ui/resources",
]
if (defined(invoker.deps)) {
deps += invoker.deps
}
if (is_android) {
sources += [ "$root_gen_dir/android_webview/aw_resources.pak" ]
deps += [ "//android_webview:generate_aw_resources" ]
} else { # Non-Android.
# New paks should be added here by default.
sources += [
"$root_gen_dir/blink/devtools_resources.pak",
"$root_gen_dir/chrome/component_extension_resources.pak",
"$root_gen_dir/chrome/options_resources.pak",
"$root_gen_dir/chrome/quota_internals_resources.pak",
"$root_gen_dir/chrome/settings_resources.pak",
"$root_gen_dir/chrome/sync_file_system_internals_resources.pak",
]
deps += [
"//chrome/browser/resources:component_extension_resources",
"//chrome/browser/resources:options_resources",
"//chrome/browser/resources:quota_internals_resources",
"//chrome/browser/resources:settings_resources",
"//chrome/browser/resources:sync_file_system_internals_resources",
"//content/browser/devtools:devtools_resources",
]
}
if (is_chromeos) {
sources += [ "$root_gen_dir/ui/file_manager/file_manager_resources.pak" ]
deps += [ "//ui/file_manager:resources" ]
}
if (enable_extensions) {
sources += [
"$root_gen_dir/extensions/extensions_renderer_resources.pak",
"$root_gen_dir/extensions/extensions_resources.pak",
]
deps += [ "//extensions:extensions_resources" ]
}
}
}
chrome/chrome_repack_locales.gni
View file @ a6085a6a
...@@ -36,6 +36,9 @@ template("chrome_repack_locales") { ...@@ -36,6 +36,9 @@ template("chrome_repack_locales") {
"//components/strings:components_locale_settings", "//components/strings:components_locale_settings",
"//components/strings:components_strings", "//components/strings:components_strings",
] ]
if (defined(invoker.deps)) {
deps += invoker.deps
}
if (use_ash) { if (use_ash) {
source_patterns += [ "${root_gen_dir}/ash/common/strings/ash_strings_" ] source_patterns += [ "${root_gen_dir}/ash/common/strings/ash_strings_" ]
...@@ -99,13 +102,5 @@ template("chrome_repack_locales") { ...@@ -99,13 +102,5 @@ template("chrome_repack_locales") {
if (defined(invoker.additional_source_patterns)) { if (defined(invoker.additional_source_patterns)) {
source_patterns += invoker.additional_source_patterns source_patterns += invoker.additional_source_patterns
} }
if (!defined(output_dir)) {
if (is_mac || is_ios) {
output_dir = "${root_gen_dir}/repack/locales"
} else {
output_dir = "${root_out_dir}/locales"
}
}
} }
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment