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

Unified Diff: blimp/test/BUILD.gn

Issue 1929723002: [Blimp] Adds blimp engine browser test framework and LoadUrl test. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addresses comments and rebases Created 4 years, 8 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: blimp/test/BUILD.gn
diff --git a/blimp/test/BUILD.gn b/blimp/test/BUILD.gn
new file mode 100644
index 0000000000000000000000000000000000000000..241a5aa4edcfa0ee723f16ba1d835f72d4b86676
--- /dev/null
+++ b/blimp/test/BUILD.gn
@@ -0,0 +1,38 @@
+# Copyright 2016 The Chromium Authors. All rights reserved.
Kevin M 2016/05/02 21:32:39 We should probably put engine-specific tests under
haibinlu 2016/05/03 19:32:43 this browser test uses both engine and client modu
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import("//build/config/features.gni")
Kevin M 2016/05/02 21:32:39 Needed?
haibinlu 2016/05/03 19:32:43 removed
+import("//build/config/sanitizers/sanitizers.gni")
Kevin M 2016/05/02 21:32:39 ?
haibinlu 2016/05/03 19:32:43 removed
+import("//build/config/ui.gni")
Kevin M 2016/05/02 21:32:39 ?
haibinlu 2016/05/03 19:32:43 removed
+import("//testing/test.gni")
+
+test("engine_browsertests") {
+ sources = [
+ "blimp_browser_test.cc",
+ "blimp_browser_test.h",
+ "blimp_test_launcher.cc",
+ "engine_browsertest.cc",
+ "test_client.cc",
+ "test_client.h",
+ ]
+
+ defines = [ "HAS_OUT_OF_PROC_TEST_RUNNER" ]
Brian Goldman 2016/05/02 16:48:58 For my edification: what does this do?
haibinlu 2016/05/02 18:36:27 It is used in content/public/test/browser_test.h s
+
+ deps = [
+ "//base",
+ "//blimp/client:feature",
+ "//blimp/client:session",
+ "//blimp/client:switches",
+ "//blimp/engine:app",
+ "//blimp/engine:app_config",
+ "//blimp/engine:app_switches",
+ "//blimp/engine:pak",
+ "//blimp/net",
+ "//content/public/app:both",
+ "//content/test:browsertest_base",
+ "//content/test:test_support",
+ "//testing/gmock",
+ "//testing/gtest",
+ ]
+}

Powered by Google App Engine
This is Rietveld 408576698