[Blimp] Adds blimp engine browser test framework and LoadUrl test.

blimp/test/BUILD.gn

+# Copyright 2016 The Chromium Authors. All rights reserved.

[Kevin M, 2016/05/02 21:32:39]

We should probably put engine-specific tests under engine/test.

This is a common location that will get super cluttered in no time with client
and engine stuff.

[haibinlu, 2016/05/03 19:32:43]

this browser test uses both engine and client module. We do not want engine to
depend on client. 

+# 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 so that we have
IN_PROC_BROWSER_TEST_F.

+
+  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",
+  ]
+}