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

Unified Diff: mash/BUILD.gn

Issue 2651953002: Revert of [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 142f775c8d838f9ec5afc6ea1bbbdfd48676a00d..79ed5e4d649125f829922d97caffde72879495f5 100644
--- a/mash/BUILD.gn
+++ b/mash/BUILD.gn
@@ -38,8 +38,38 @@
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",
@@ -48,41 +78,31 @@
"//services/ui/demo:manifest",
"//services/ui/ime/test_ime_driver:manifest",
- # 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",
+ # See the note above about dependencies on Chrome.
+ "//chrome/app:service_manifests",
]
- executable_overrides = [ "content_browser:@EXE_DIR/chrome" ]
+ if (is_chromeos) {
+ standalone_services += [ "ash" ]
- catalog_deps = [ "//mash/example:catalog" ]
-
- if (is_chromeos) {
- standalone_services += [ "//ash/mus:manifest" ]
+ deps += [ "//ash/mus:manifest" ]
}
if (is_linux && !is_android) {
- standalone_services += [ "//components/font_service:manifest" ]
+ standalone_services += [ "font_service" ]
+
+ deps += [ "//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",
]
}
@@ -113,15 +133,14 @@
"//base/test:test_support",
"//cc",
"//cc:test_support",
- "//mojo/edk/system",
- "//services/catalog:lib",
+ "//services/service_manager/background:main",
"//ui/aura",
"//ui/base",
"//ui/compositor:test_support",
"//ui/gl:test_support",
]
data_deps = [
- ":mash_unittests_catalog_copy",
+ ":unittests_manifest",
"//ash/mus:resources",
]
}
@@ -130,23 +149,4 @@
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