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

Unified Diff: mojo/BUILD.gn

Issue 1350023003: Add a Mojo EDK for Chrome that uses one OS pipe per message pipe. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: some review comments Created 5 years, 3 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
Index: mojo/BUILD.gn
diff --git a/mojo/BUILD.gn b/mojo/BUILD.gn
index 9e42c4ce8ea5057b6099e9b3c308aee297169add..c5568dad4416ccfd5b26bd27913e6cf5d62ed3b3 100644
--- a/mojo/BUILD.gn
+++ b/mojo/BUILD.gn
@@ -34,6 +34,7 @@ group("mojo") {
group("mojo_public_bindings_unittests_run") {
testonly = true
deps = [
+ # TODO(use_chrome_edk): "//mojo/edk/test:mojo_public_bindings_unittests"
"//third_party/mojo/src/mojo/edk/test:mojo_public_bindings_unittests",
]
}
@@ -43,7 +44,8 @@ group("mojo_public_bindings_unittests_run") {
group("mojo_public_environment_unittests_run") {
testonly = true
deps = [
- "//third_party/mojo/src/mojo/edk/test:mojo_public_environment_unittests",
+ # TODO(use_chrome_edk): "//mojo/edk/test:mojo_public_bindings_unittests"
+ "//third_party/mojo/src/mojo/edk/test:mojo_public_bindings_unittests",
]
}
@@ -51,7 +53,9 @@ group("mojo_public_environment_unittests_run") {
# The _run targets exist only for compatibility w/ GYP.
group("mojo_public_system_unittests_run") {
testonly = true
+
deps = [
+ # TODO(use_chrome_edk): "//mojo/edk/test:mojo_public_system_unittests"
"//third_party/mojo/src/mojo/edk/test:mojo_public_system_unittests",
]
}
@@ -61,6 +65,7 @@ group("mojo_public_system_unittests_run") {
group("mojo_public_utility_unittests_run") {
testonly = true
deps = [
+ # TODO(use_chrome_edk): "//mojo/edk/test:mojo_public_utility_unittests"
"//third_party/mojo/src/mojo/edk/test:mojo_public_utility_unittests",
]
}
@@ -72,6 +77,16 @@ group("tests") {
"//mojo/application/public/cpp/tests:mojo_public_application_unittests",
"//mojo/common:mojo_common_unittests",
"//mojo/converters/surfaces/tests:mojo_surfaces_lib_unittests",
+
+ # TODO(use_chrome_edk):
+ #"//mojo/edk/js/test:js_unittests",
+ #"//mojo/edk/system:mojo_message_pipe_perftests",
+ #"//mojo/edk/system:mojo_system_unittests",
+ #"//mojo/edk/test:mojo_public_bindings_unittests",
+ #"//mojo/edk/test:mojo_public_environment_unittests",
+ #"//mojo/edk/test:mojo_public_system_perftests",
+ #"//mojo/edk/test:mojo_public_system_unittests",
+ #"//mojo/edk/test:mojo_public_utility_unittests",
"//third_party/mojo/src/mojo/edk/js/test:js_unittests",
"//third_party/mojo/src/mojo/edk/system:mojo_message_pipe_perftests",
"//third_party/mojo/src/mojo/edk/system:mojo_system_unittests",

Powered by Google App Engine
This is Rietveld 408576698