Hook up test runner to new layout_test_html_viewer

components/html_viewer/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/module_args/v8.gni")
 import("//mojo/public/mojo_application.gni")
 import("//testing/test.gni")
 import("//tools/grit/grit_rule.gni")
 import("//tools/grit/repack.gni")
 
@@ skipping 49 matching lines @@
     "ax_provider_impl.h",
     "blink_basic_type_converters.cc",
     "blink_basic_type_converters.h",
     "blink_input_events_type_converters.cc",
     "blink_input_events_type_converters.h",
     "blink_platform_impl.cc",
     "blink_platform_impl.h",
     "blink_resource_constants.h",
     "blink_url_request_type_converters.cc",
     "blink_url_request_type_converters.h",
+    "content_handler_impl.cc",
+    "content_handler_impl.h",
     "devtools_agent_impl.cc",
     "devtools_agent_impl.h",
     "discardable_memory_allocator.cc",
     "discardable_memory_allocator.h",
     "document_resource_waiter.cc",
     "document_resource_waiter.h",
     "frame.cc",
     "frame.h",
     "frame_tree_manager.cc",
     "frame_tree_manager.h",
     "frame_tree_manager_delegate.h",
     "global_state.cc",
     "global_state.h",
     "html_document.cc",
     "html_document.h",
+    "html_document_application_delegate.cc",
+    "html_document_application_delegate.h",
     "html_document_oopif.cc",
     "html_document_oopif.h",
+    "html_viewer.cc",
+    "html_viewer.h",
+    "html_viewer_switches.cc",
+    "html_viewer_switches.h",
     "media_factory.cc",
     "media_factory.h",
     "mock_web_blob_registry_impl.cc",
     "mock_web_blob_registry_impl.h",
     "test_html_viewer_impl.cc",
     "test_html_viewer_impl.h",
     "touch_handler.cc",
     "touch_handler.h",
     "web_clipboard_impl.cc",
     "web_clipboard_impl.h",
@@ skipping 76 matching lines @@
   }
 
   data_deps = [
     "//components/clipboard",
     "//components/view_manager",
   ]
 }
 
 mojo_native_application("html_viewer") {
   sources = [
-    "content_handler_impl.cc",
-    "content_handler_impl.h",
-    "html_document_application_delegate.cc",
-    "html_document_application_delegate.h",
-    "html_viewer.cc",
-    "html_viewer.h",
     "html_viewer_main.cc",
-    "html_viewer_switches.cc",
-    "html_viewer_switches.h",
   ]
   deps = [
     ":html_viewer_resources_grit",
     ":lib",
     "//mojo/common",
     "//mojo/platform_handle",
     "//third_party/icu:icudata",
     "//third_party/mojo/src/mojo/public/c/system:for_shared_library",
     "//ui/resources:ui_test_pak",
   ]
   data_deps = [ "//mojo/services/network:network" ]
   resources = [
     "$root_out_dir/icudtl.dat",
     "$root_out_dir/ui_test.pak",
     "$target_gen_dir/html_viewer_resources.pak",
   ]
 
   if (v8_use_external_startup_data) {
     resources += [
       "$root_build_dir/natives_blob.bin",
       "$root_build_dir/snapshot_blob.bin",
     ]
     deps += [ "//gin" ]
   }
 }
 
+mojo_native_application("layout_test_html_viewer") {
+  testonly = true
+  sources = [
+    "layout_test_content_handler_impl.cc",
+    "layout_test_content_handler_impl.h",
+    "layout_test_html_viewer.cc",
+    "layout_test_html_viewer.h",
+    "layout_test_html_viewer_main.cc",
+    "web_test_delegate_impl.cc",
+    "web_test_delegate_impl.h",
+  ]
+  deps = [

[jam, 2015/07/09 16:48:27]

many of these are shared. can you move them up to lib?

[jochen (gone - plz use gerrit), 2015/07/10 07:53:43]

the dependencies on the resources need to stay, but I moved what is possible

+    ":html_viewer_resources_grit",
+    ":lib",
+    "//components/test_runner",
+    "//mojo/common",
+    "//mojo/platform_handle",
+    "//third_party/WebKit/public:blink",
+    "//third_party/WebKit/public:test_support",
+    "//third_party/icu:icudata",
+    "//third_party/mojo/src/mojo/public/c/system:for_shared_library",
+    "//ui/resources:ui_test_pak",
+    "//url",
+  ]
+  data_deps = [ "//mojo/services/network:network" ]

[jam, 2015/07/09 16:48:27]

can we put this in the lib section since it's shared? (there are other data_deps
there)

[jochen (gone - plz use gerrit), 2015/07/10 07:53:43]

done

+  resources = [
+    "$root_out_dir/icudtl.dat",
+    "$root_out_dir/ui_test.pak",
+    "$target_gen_dir/html_viewer_resources.pak",
+  ]
+
+  if (v8_use_external_startup_data) {
+    resources += [
+      "$root_build_dir/natives_blob.bin",
+      "$root_build_dir/snapshot_blob.bin",
+    ]
+    deps += [ "//gin" ]
+  }
+}
+
 test("html_viewer_unittests") {
   sources = [
     "ax_provider_impl_unittest.cc",
     "discardable_memory_allocator_unittest.cc",
     "run_all_unittests.cc",
   ]
   deps = [
     ":lib",
     "//base/test:test_support",
     "//third_party/mojo/src/mojo/edk/system",
@@ skipping 20 matching lines @@
     "//testing/gtest",
     "//third_party/mojo_services/src/accessibility/public/interfaces",
     "//third_party/mojo/src/mojo/public/c/system:for_shared_library",
   ]
 
   data_deps = [
     ":html_viewer",
     "//components/view_manager",
   ]
 }