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

Unified Diff: mash/BUILD.gn

Issue 2645973006: [Service Manager] Get rid of dynamic service discovery (Closed)
Patch Set: . Created 3 years, 11 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 | « content/test/run_all_unittests.cc ('k') | mash/example/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mash/BUILD.gn
diff --git a/mash/BUILD.gn b/mash/BUILD.gn
index 79ed5e4d649125f829922d97caffde72879495f5..142f775c8d838f9ec5afc6ea1bbbdfd48676a00d 100644
--- a/mash/BUILD.gn
+++ b/mash/BUILD.gn
@@ -38,38 +38,8 @@ catalog("catalog") {
testonly = true
standalone_services = [
- "catalog_viewer",
- "leveldb",
- "mash_session",
- "mus_demo",
- "quick_launch",
- "task_viewer",
- "test_ime_driver",
- "tracing",
- "ui",
-
- # TODO(rockot): It seems like a layering violation to have these service
- # names referenced here, but it's the most convenient thing to do for now,
- # and this is only for developers.
- #
- # Note that there are no build-time dependencies on the actual Chrome
- # binary implied by mash:all or other targets here. This only ensures that
- # if you have such a binary built, the runner will be able to locate it.
- "chrome_content_browser",
- "chrome_content_gpu",
- "chrome_content_plugin",
- "chrome_content_renderer",
- "chrome_content_utility",
- ]
-
- executable_overrides = [ "chrome_content_browser:@EXE_DIR/chrome" ]
-
- included_catalogs = [ "${root_gen_dir}/mash/example/catalog.json" ]
-
- deps = [
"//components/leveldb:manifest",
"//mash/catalog_viewer:manifest",
- "//mash/example:catalog",
"//mash/quick_launch:manifest",
"//mash/session:manifest",
"//mash/task_viewer:manifest",
@@ -78,32 +48,42 @@ catalog("catalog") {
"//services/ui/demo:manifest",
"//services/ui/ime/test_ime_driver:manifest",
- # See the note above about dependencies on Chrome.
- "//chrome/app:service_manifests",
+ # TODO(rockot): It seems like a layering violation to have these service
+ # names referenced here, but it's the most convenient thing to do for now,
+ # and this is only for developers.
+ #
+ # Note that there are no build-time dependencies on the actual Chrome
+ # binary implied by mash:all or other targets here. This only ensures that
+ # if you have such a binary built, the runner will be able to locate it.
+ "//chrome/app:chrome_content_browser_manifest",
+ "//chrome/app:chrome_content_gpu_manifest",
+ "//chrome/app:chrome_content_plugin_manifest",
+ "//chrome/app:chrome_content_renderer_manifest",
+ "//chrome/app:chrome_content_utility_manifest",
]
- if (is_chromeos) {
- standalone_services += [ "ash" ]
+ executable_overrides = [ "content_browser:@EXE_DIR/chrome" ]
+
+ catalog_deps = [ "//mash/example:catalog" ]
- deps += [ "//ash/mus:manifest" ]
+ if (is_chromeos) {
+ standalone_services += [ "//ash/mus:manifest" ]
}
if (is_linux && !is_android) {
- standalone_services += [ "font_service" ]
-
- deps += [ "//components/font_service:manifest" ]
+ standalone_services += [ "//components/font_service:manifest" ]
}
}
copy("mash_catalog") {
testonly = true
- deps = [
- ":catalog",
- ]
sources = get_target_outputs(":catalog")
outputs = [
"$root_out_dir/mash_catalog.json",
]
+ deps = [
+ ":catalog",
+ ]
}
group("browser") {
@@ -133,14 +113,15 @@ if (is_chromeos) {
"//base/test:test_support",
"//cc",
"//cc:test_support",
- "//services/service_manager/background:main",
+ "//mojo/edk/system",
+ "//services/catalog:lib",
"//ui/aura",
"//ui/base",
"//ui/compositor:test_support",
"//ui/gl:test_support",
]
data_deps = [
- ":unittests_manifest",
+ ":mash_unittests_catalog_copy",
"//ash/mus:resources",
]
}
@@ -149,4 +130,23 @@ if (is_chromeos) {
name = "mash_unittests"
source = "unittests_manifest.json"
}
+
+ catalog("mash_unittests_catalog") {
+ testonly = true
+
+ embedded_services = [ ":unittests_manifest" ]
+
+ catalog_deps = [ ":catalog" ]
+ }
+
+ copy("mash_unittests_catalog_copy") {
+ testonly = true
+ sources = get_target_outputs(":mash_unittests_catalog")
+ outputs = [
+ "${root_out_dir}/mash_unittests_catalog.json",
+ ]
+ deps = [
+ ":mash_unittests_catalog",
+ ]
+ }
}
« no previous file with comments | « content/test/run_all_unittests.cc ('k') | mash/example/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698