[Chromoting] Enable remoting webapp main.html GN build.

BUILD.gn

Index: BUILD.gn
diff --git a/BUILD.gn b/BUILD.gn
index a5d7cfd058981929a0984a8b9e0187350d2a22d0..e634d634c7748486f357302d0f50ec7a6c67bdf7 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -10,7 +10,6 @@
 
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
-import("//remoting/remoting_host.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
@@ -102,6 +101,7 @@ group("gn_all") {
     "//ppapi/examples/video_effects",
     "//ppapi/examples/video_encode",
     "//printing:printing_unittests",
+    "//remoting:all_r",

[garykac, 2015/04/15 00:44:07]

I attempted to use 'all' here (since duplicate names are supposed to be ok), but
ninja complained about 'all' being defined multiple times.

'all_r' is a placeholder, what naming conventions do you suggest for this?

[Dirk Pranke, 2015/04/15 01:08:40]

I'm not 100% convinced that we should move the files into a subdirectory. The
advantage to having everything defined in the top-level file is that you can
easily see what all is built in one place; you don't have to jump around to N
different files like you do in GYP.

However, remoting is complicated, and I can see the counterargument, so let's
take remoting as an experiment for now about creating roll-up targets in
subdirectories.

You should use '*_all' for a name, e.g. 'remoting_all'.

However, I would also like the structure of this target to mirror the structure
of the 'gn_all' target in build/gn_migration.gypi. Can you please update that as
well?

[garykac, 2015/04/16 02:11:57]

Done.

     "//skia:skia_unittests",
     "//sql:sql_unittests",
     "//sync:sync_unit_tests",
@@ -136,20 +136,6 @@ group("gn_all") {
     deps += [ "//extensions/shell:app_shell_unittests" ]
   }
 
-  if (enable_remoting_host) {

[garykac, 2015/04/15 00:44:07]

Moving these into the remoting GN build files since it makes sense to keep all
that info in one place.

-    deps += [
-      "//remoting:remoting_unittests",
-      "//remoting:remoting_perftests",
-      "//remoting/host",
-      "//remoting/host:remoting_start_host",
-      "//remoting/host/it2me:remote_assistance_host",
-    ]
-  }
-
-  if (enable_me2me_host) {
-    deps += [ "//remoting/host:remoting_me2me_host" ]
-  }
-
   if (!is_win) {
     deps += [ "//breakpad:symupload" ]
   }