Revert of [Chromoting] Add GN build support for ar_sample_app target.…

… (patchset #1 id:1 of https://codereview.chromium.org/1135933002/) Reason for revert: Broke GN bots, eg.: http://build.chromium.org/p/chromium.linux/builders/Linux%20GN/builds/28281 Original issue's description: > [Chromoting] Add GN build support for ar_sample_app target. > > This adds GN build rules for the AppRemoting sample app. > Some DesktopRemoting build rules were prettified as part of this > process (to make the App- and Desktop- remoting build rules more > similar). > > This re-lands crrev.com/1126393002 without any additional changes > required. The issues with the GN clobber builds are fixed in > crrev.com/1137753002. > > BUG=471916 > > Committed: https://crrev.com/581797d8f76d5855b7f0bed1fe912a038d1d7d25 > Cr-Commit-Position: refs/heads/master@{#329206} TBR=jamiewalch@chromium.org,garykac@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=471916 Review URL: https://codereview.chromium.org/1136673003 Cr-Commit-Position: refs/heads/master@{#329228}
FairyWorld · May 11, 2015 · 0374f26 · 0374f26
1 parent 85bad9e
commit 0374f26
Show file tree

Hide file tree

Showing 8 changed files with 137 additions and 450 deletions.
diff --git a/remoting/app_remoting_webapp.gyp b/remoting/app_remoting_webapp.gyp
@@ -92,7 +92,6 @@
 
   'targets': [
     {
-      # GN version: //remoting/webapp:ar_sample_app
       # Sample AppRemoting app.
       'target_name': 'ar_sample_app',
       'app_key': 'Sample_App',

diff --git a/remoting/app_remoting_webapp_build.gypi b/remoting/app_remoting_webapp_build.gypi
@@ -11,7 +11,6 @@
     'app_remoting_webapp_files.gypi',
   ],
 
-  # GN version: See remoting/webapp/build_template.gni
   'target_defaults': {
     'type': 'none',
 
@@ -120,6 +119,7 @@
           '<(ar_app_manifest_app)', # Manifest template
           'app_remoting',  # Web app type
           '<@(ar_webapp_files)',
+          '<@(ar_generated_html_files)',
           '--locales_listfile',
           '<(ar_webapp_locales_listfile)',
           '--jinja_paths',

diff --git a/remoting/remoting_client.gypi b/remoting/remoting_client.gypi
@@ -49,11 +49,12 @@
     },  # end of target 'remoting_client'
 
     {
-      # GN version: See remoting/webapp/build_template.gni
+      # GN version: //remoting/webapp:html
       'target_name': 'remoting_webapp_html',
       'type': 'none',
       'actions': [
         {
+          # GN version: //remoting/webapp:main_html
           'action_name': 'Build Remoting Webapp main.html',
           'inputs': [
             'webapp/build-html.py',
@@ -74,6 +75,7 @@
           ],
         },
         {
+          # GN version: //remoting/webapp:wcs_sandbox_html
           'action_name': 'Build Remoting Webapp wcs_sandbox.html',
           'inputs': [
             'webapp/build-html.py',
@@ -90,6 +92,7 @@
           ],
         },
         {
+          # GN version: //remoting/webapp:background_html
           'action_name': 'Build Remoting Webapp background.html',
           'inputs': [
             'webapp/build-html.py',
@@ -125,7 +128,6 @@
     },  # end of target 'remoting_webapp_html'
 
     {
-      # GN version: //remoting/webapp:webapp
       'target_name': 'remoting_webapp',
       'type': 'none',
       'dependencies': [
@@ -141,7 +143,6 @@
     },  # end of target 'remoting_webapp'
 
     {
-      # GN version: //remoting/webapp:webapp_v1
       'target_name': 'remoting_webapp_v1',
       'type': 'none',
       'variables': {
@@ -157,7 +158,6 @@
     ['disable_nacl==0 and disable_nacl_untrusted==0', {
       'targets': [
         {
-          # GN version: //remoting/webapp:webapp_v1
           'target_name': 'remoting_webapp_v2',
           'type': 'none',
           'variables': {

diff --git a/remoting/remoting_version.gni b/remoting/remoting_version.gni
@@ -15,7 +15,7 @@ if (is_chrome_branded) {
 
 # Set these files as being input dependencies to the scripts, so the build will
 # be re-run if the files change.
-remoting_version_files = [
+_script_deps = [
   _remoting_version_abspath,
   _chrome_version_abspath,
   _remoting_branding_abspath,
@@ -35,7 +35,7 @@ version_major = exec_script(_version_py_abspath,
                               "-t \"@MAJOR@\"",
                             ],
                             "value",
-                            remoting_version_files)
+                            _script_deps)
 
 version_minor = exec_script(_version_py_abspath,
                             [
@@ -44,7 +44,7 @@ version_minor = exec_script(_version_py_abspath,
                               "-t \"@REMOTING_PATCH@\"",
                             ],
                             "value",
-                            remoting_version_files)
+                            _script_deps)
 
 version_short =
     "${version_major}.${version_minor}." + exec_script(_version_py_abspath,
@@ -56,7 +56,7 @@ version_short =
                                                          "-t \"@BUILD@\"",
                                                        ],
                                                        "value",
-                                                       remoting_version_files)
+                                                       _script_deps)
 
 version_full = "${version_short}." + exec_script(_version_py_abspath,
                                                  [
@@ -67,7 +67,7 @@ version_full = "${version_short}." + exec_script(_version_py_abspath,
                                                    "-t \"@PATCH@\"",
                                                  ],
                                                  "value",
-                                                 remoting_version_files)
+                                                 _script_deps)
 
 prefpane_bundle_name = exec_script(_version_py_abspath,
                                    [
@@ -76,7 +76,7 @@ prefpane_bundle_name = exec_script(_version_py_abspath,
                                      "-t \"@MAC_PREFPANE_BUNDLE_NAME@\"",
                                    ],
                                    "string",
-                                   remoting_version_files)
+                                   _script_deps)
 
 host_bundle_name = exec_script(_version_py_abspath,
                                [
@@ -85,4 +85,4 @@ host_bundle_name = exec_script(_version_py_abspath,
                                  "-t \"@MAC_HOST_BUNDLE_NAME@\"",
                                ],
                                "string",
-                               remoting_version_files)
+                               _script_deps)
diff --git a/remoting/webapp/BUILD.gn b/remoting/webapp/BUILD.gn
@@ -10,38 +10,116 @@ import("//remoting/webapp/build_template.gni")
 group("webapp") {
   deps = [
     ":webapp_v1",
-    ":ar_sample_app",
   ]
 
   if (enable_nacl) {
     deps += [ ":webapp_v2" ]
   }
 }
 
-desktop_remoting_webapp("webapp_v1") {
+remoting_webapp("webapp_v1") {
   webapp_type = "v1"
-  output_dir = "remoting/remoting-webapp"
-  zip_path = "remoting/remoting-webapp.zip"
+  output_dir = "remoting.webapp"
+  zip_path = "remoting-webapp.zip"
   extra_files = []
 }
 
-desktop_remoting_webapp("webapp_v2") {
+remoting_webapp("webapp_v2") {
   webapp_type = "v2_pnacl"
-  output_dir = "remoting/remoting-webapp.v2"
-  zip_path = "remoting/remoting-webapp.v2.zip"
+  output_dir = "remoting.webapp.v2"
+  zip_path = "remoting-webapp.v2.zip"
   extra_files = [
     "crd/remoting_client_pnacl.nmf.jinja2",
     # TODO(garykac): Get correct path to this.
     #"<(PRODUCT_DIR)/remoting_client_plugin_newlib.pexe",
   ]
 }
 
-app_remoting_webapp("ar_sample_app") {
-  app_key = "Sample_App"
-  app_id = "ljacajndfccfgnfohlgkdphmbnpkjflk"
-  app_client_id = "sample_client_id"
-  app_name = "App Remoting Client"
-  app_description = "App Remoting client"
-  app_capabilities = [ "GOOGLE_DRIVE" ]
-  manifest_key = "remotingdevbuild"
+group("html") {
+  deps = [
+    ":background_html",
+    ":main_html",
+    ":message_window_html",
+    ":wcs_sandbox_html",
+  ]
+}
+
+action("main_html") {
+  script = "build-html.py"
+
+  inputs = [ remoting_webapp_template_main ] + remoting_webapp_template_files +
+           remoting_webapp_crd_main_html_all_js_files
+
+  outputs = [
+    "$target_gen_dir/main.html",
+  ]
+
+  args = [
+    rebase_path("$target_gen_dir/main.html", root_build_dir),
+    rebase_path(remoting_webapp_template_main, root_build_dir),
+  ]
+  args += [
+    "--template-dir",
+    rebase_path(remoting_dir, root_build_dir),
+  ]
+  args += [ "--templates" ] +
+          rebase_path(remoting_webapp_template_files, remoting_dir)
+  args += [ "--js" ] +
+          rebase_path(remoting_webapp_crd_main_html_all_js_files, remoting_dir)
+}
+
+action("wcs_sandbox_html") {
+  script = "build-html.py"
+
+  inputs = [ remoting_webapp_template_wcs_sandbox ] +
+           remoting_webapp_wcs_sandbox_html_all_js_files
+
+  outputs = [
+    "$target_gen_dir/wcs_sandbox.html",
+  ]
+
+  args = [
+    rebase_path("$target_gen_dir/wcs_sandbox.html", root_build_dir),
+    rebase_path(remoting_webapp_template_wcs_sandbox, root_build_dir),
+  ]
+  args +=
+      [ "--js" ] +
+      rebase_path(remoting_webapp_wcs_sandbox_html_all_js_files, remoting_dir)
+}
+
+action("background_html") {
+  script = "build-html.py"
+
+  inputs = [ remoting_webapp_template_background ] +
+           remoting_webapp_background_html_all_js_files
+
+  outputs = [
+    "$target_gen_dir/background.html",
+  ]
+
+  args = [
+    rebase_path("$target_gen_dir/background.html", root_build_dir),
+    rebase_path(remoting_webapp_template_background, root_build_dir),
+  ]
+  args += [ "--js" ] + rebase_path(remoting_webapp_background_html_all_js_files,
+                                   remoting_dir)
+}
+
+action("message_window_html") {
+  script = "build-html.py"
+
+  inputs = [ remoting_webapp_template_message_window ] +
+           remoting_webapp_message_window_html_all_js_files
+
+  outputs = [
+    "$target_gen_dir/message_window.html",
+  ]
+
+  args = [
+    rebase_path("$target_gen_dir/message_window.html", root_build_dir),
+    rebase_path(remoting_webapp_template_message_window, root_build_dir),
+  ]
+  args +=
+      [ "--js" ] + rebase_path(remoting_webapp_message_window_html_all_js_files,
+                               remoting_dir)
 }
diff --git a/remoting/webapp/build-webapp.py b/remoting/webapp/build-webapp.py
@@ -169,7 +169,7 @@ def buildWebApp(buildtype, version, destination, zip_path,
       raise
     else:
       pass
-  os.makedirs(destination, 0775)
+  os.mkdir(destination, 0775)
 
   if buildtype != 'Official' and buildtype != 'Release' and buildtype != 'Dev':
     raise Exception('Unknown buildtype: ' + buildtype)