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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: extensions/shell/BUILD.gn

Issue 922093002: Clean up app_shell GN build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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
« extensions/BUILD.gn ('K') | « extensions/BUILD.gn ('k') | extensions/shell/app_shell.gni » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/shell/BUILD.gn
diff --git a/extensions/shell/BUILD.gn b/extensions/shell/BUILD.gn
index a251765c0ee4aa54f1577f6b0a5b8d952f4083f5..1d9777b5929fad27f2ebbff70d0301e8cfb28a53 100644
--- a/extensions/shell/BUILD.gn
+++ b/extensions/shell/BUILD.gn
@@ -2,10 +2,13 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
+import("//extensions/shell/app_shell.gni")
+
# Technically, this directory should not depend on files from src/chrome, but
# that's where the VERSION file is. This should probably all be moved to
# src/build.
import("//chrome/version.gni")
+import("//testing/test.gni")
import("//tools/grit/grit_rule.gni")
assert(enable_extensions)
@@ -52,105 +55,54 @@ source_set("app_shell_lib") {
"//third_party/mojo/src/mojo/edk/system",
"//ui/base",
"//ui/base/ime",
- "//ui/wm",
"//v8",
]
- sources = [
- "app/shell_main_delegate.cc",
- "app/shell_main_delegate.h",
- "browser/api/identity/identity_api.cc",
- "browser/api/identity/identity_api.h",
- "browser/default_shell_browser_main_delegate.cc",
- "browser/default_shell_browser_main_delegate.h",
- "browser/desktop_controller.cc",
- "browser/desktop_controller.h",
- "browser/media_capture_util.cc",
- "browser/media_capture_util.h",
- "browser/shell_app_delegate.cc",
- "browser/shell_app_delegate.h",
- "browser/shell_app_view_guest_delegate.cc",
- "browser/shell_app_view_guest_delegate.h",
- "browser/shell_app_window_client.cc",
- "browser/shell_app_window_client.h",
- "browser/shell_app_window_client_aura.cc",
- "browser/shell_audio_controller_chromeos.cc",
- "browser/shell_audio_controller_chromeos.h",
- "browser/shell_browser_context.cc",
- "browser/shell_browser_context.h",
- "browser/shell_browser_context_keyed_service_factories.cc",
- "browser/shell_browser_context_keyed_service_factories.h",
- "browser/shell_browser_main_delegate.h",
- "browser/shell_browser_main_parts.cc",
- "browser/shell_browser_main_parts.h",
- "browser/shell_content_browser_client.cc",
- "browser/shell_content_browser_client.h",
- "browser/shell_desktop_controller_aura.cc",
- "browser/shell_desktop_controller_aura.h",
- "browser/shell_device_client.cc",
- "browser/shell_device_client.h",
- "browser/shell_display_info_provider.cc",
- "browser/shell_display_info_provider.h",
- "browser/shell_extension_host_delegate.cc",
- "browser/shell_extension_host_delegate.h",
- "browser/shell_extension_system.cc",
- "browser/shell_extension_system.h",
- "browser/shell_extension_system_factory.cc",
- "browser/shell_extension_system_factory.h",
- "browser/shell_extension_web_contents_observer.cc",
- "browser/shell_extension_web_contents_observer.h",
- "browser/shell_extensions_api_client.cc",
- "browser/shell_extensions_api_client.h",
- "browser/shell_extensions_browser_client.cc",
- "browser/shell_extensions_browser_client.h",
- "browser/shell_native_app_window.cc",
- "browser/shell_native_app_window.h",
- "browser/shell_native_app_window_aura.cc",
- "browser/shell_native_app_window_aura.h",
- "browser/shell_network_controller_chromeos.cc",
- "browser/shell_network_controller_chromeos.h",
- "browser/shell_network_delegate.cc",
- "browser/shell_network_delegate.h",
- "browser/shell_oauth2_token_service.cc",
- "browser/shell_oauth2_token_service.h",
- "browser/shell_prefs.cc",
- "browser/shell_prefs.h",
- "browser/shell_runtime_api_delegate.cc",
- "browser/shell_runtime_api_delegate.h",
- "browser/shell_screen.cc",
- "browser/shell_screen.h",
- "browser/shell_special_storage_policy.cc",
- "browser/shell_special_storage_policy.h",
- "browser/shell_speech_recognition_manager_delegate.cc",
- "browser/shell_speech_recognition_manager_delegate.h",
- "browser/shell_update_query_params_delegate.cc",
- "browser/shell_update_query_params_delegate.h",
- "browser/shell_url_request_context_getter.cc",
- "browser/shell_url_request_context_getter.h",
- "browser/shell_web_contents_modal_dialog_manager.cc",
- "common/shell_content_client.cc",
- "common/shell_content_client.h",
- "common/shell_extensions_client.cc",
- "common/shell_extensions_client.h",
- "common/switches.cc",
- "common/switches.h",
- "renderer/shell_content_renderer_client.cc",
- "renderer/shell_content_renderer_client.h",
- "renderer/shell_extensions_renderer_client.cc",
- "renderer/shell_extensions_renderer_client.h",
- "utility/shell_content_utility_client.cc",
- "utility/shell_content_utility_client.h",
- ]
+
+ sources = rebase_path(app_shell_gypi_values.app_shell_lib_sources,
+ ".",
+ "//extensions/shell")
+
+ if (use_aura) {
+ deps += [ "//ui/wm" ]
+
+ aura_sources = rebase_path(app_shell_gypi_values.app_shell_lib_sources_aura,
+ ".",
+ "//extensions/shell")
+ sources += aura_sources
+ }
+
if (is_chromeos) {
deps += [
"//chromeos",
"//ui/chromeos:ui_chromeos",
"//ui/display",
]
- sources += [
- "browser/api/shell_gcd/shell_gcd_api.cc",
- "browser/api/shell_gcd/shell_gcd_api.h",
- "browser/api/vpn_provider/vpn_service_factory.cc",
- ]
+ chromeos_sources =
+ rebase_path(app_shell_gypi_values.app_shell_lib_sources_chromeos,
+ ".",
+ "//extensions/shell")
+ sources += chromeos_sources
+ }
+
+ # TODO(GYP): Enable this when //components/nacl GN is done.
James Cook 2015/02/13 21:53:31 Can you maybe file a bug for this? Or reference th
Ken Rockot(use gerrit already) 2015/02/13 22:08:31 Added a ref to the nacl GN bug
+ if (false) {
+ if (enable_nacl) {
+ if (is_linux) {
+ deps += [ "//components/nacl:nacl_helper" ]
+ }
+ deps += [
+ "//components/nacl",
+ "//components/nacl:browser",
+ "//components/nacl:common",
+ "//components/nacl:renderer",
+ "//components/nacl:switches",
+ ]
+ nacl_sources =
+ rebase_path(app_shell_gypi_values.app_shell_lib_sources_nacl,
+ ".",
+ "//extensions/shell")
+ sources += nacl_sources
+ }
}
}
@@ -158,9 +110,9 @@ if (!(is_chromeos && !use_ozone) && (!is_win || link_chrome_on_windows)) {
executable("app_shell") {
# testonly because :app_shell_lib is testonly. See :app_shell_lib comment.
testonly = true
- sources = [
- "app/shell_main.cc",
- ]
+ sources = rebase_path(app_shell_gypi_values.app_shell_sources,
+ ".",
+ "//extensions/shell")
deps = [
":app_shell_lib",
"//extensions:shell_and_test_pak",
@@ -169,10 +121,66 @@ if (!(is_chromeos && !use_ozone) && (!is_win || link_chrome_on_windows)) {
if (is_win) {
configs += [ "//build/config/win:windowed" ]
configs -= [ "//build/config/win:console" ]
+ deps += [ "//base/allocator" ]
+ }
+
+ if (is_mac) {
+ # TODO(GYP): Mac bundling
}
}
}
+test("app_shell_unittests") {
+ sources = rebase_path(app_shell_gypi_values.app_shell_unittests_sources,
+ ".",
+ "//extensions/shell")
+
+ deps = [
+ ":app_shell_lib",
+ "//base",
+ "//base/test:test_support",
+ "//content/test:test_support",
+ "//extensions:shell_and_test_pak",
+ "//extensions:test_support",
+ "//testing/gtest",
+ ]
+
+ if (use_aura) {
+ deps += [ "//ui/aura:test_support" ]
+
+ aura_sources =
+ rebase_path(app_shell_gypi_values.app_shell_unittests_sources_aura,
+ ".",
+ "//extensions/shell")
+ sources += aura_sources
+ }
+
+ if (is_chromeos) {
+ deps += [ "//chromeos:test_support_without_gmock" ]
+
+ chromeos_sources =
+ rebase_path(app_shell_gypi_values.app_shell_unittests_sources_chromeos,
+ ".",
+ "//extensions/shell")
+ sources += chromeos_sources
+ }
+
+ # TODO(GYP): Enable this when //components/nacl GN is done.
+ if (false) {
+ if (use_nacl) {
+ nacl_sources =
+ rebase_path(app_shell_gypi_values.app_shell_unittests_sources_nacl,
+ ".",
+ "//extensions/shell")
+ sources += nacl_sources
+ }
+ }
+
+ if (is_win) {
+ deps += [ "//base/allocator" ]
+ }
+}
+
process_version("version_header") {
source = "common/version.h.in"
output = "$target_gen_dir/common/version.h"
« extensions/BUILD.gn ('K') | « extensions/BUILD.gn ('k') | extensions/shell/app_shell.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698