Fix generate_policy_source.py to honour supported_on ranges.

components/policy/BUILD.gn

Index: components/policy/BUILD.gn
diff --git a/components/policy/BUILD.gn b/components/policy/BUILD.gn
index 632cbe673d6991128a72d59849ed5ecfff43f5bf..0eea25c8fa928e9b80b5e9e4ee6d3b9ff2ad8a84 100644
--- a/components/policy/BUILD.gn
+++ b/components/policy/BUILD.gn
@@ -67,6 +67,19 @@ if (enable_configuration_policy) {
   app_resources_path =
       "$android_resources_gen_dir/values-v21/restriction_values.xml"
 
+  # Obtain Chromium version.
+  version_py_abspath = "//build/util/version.py"
+  chrome_version_abspath = "//chrome/VERSION"
+  chrome_version_path = rebase_path(chrome_version_abspath, root_build_dir)
+  version_full = exec_script(version_py_abspath,
+                             [
+                               "-f",
+                               chrome_version_path,
+                               "-t @MAJOR@.@MINOR@.@BUILD@.@PATCH@",
+                             ],
+                             "trim string",
+                             [ chrome_version_abspath ])

[Mattias Nissler (ping if slow), 2015/02/18 10:34:42]

I have no idea where this came from, but I assume you copied it from somewhere?
If so, can you please CC the respective OWNER/reviewer to perform a sanity
check?

[Thiemo Nagel, 2015/02/18 17:31:56]

Done.

[brettw, 2015/02/23 19:19:07]

I have tried very hard not to add this kind of thing. We did it a lot in GYP
because everything is so slow you don't notice that everything grinds to a halt
while we run python and do a bunch of disk I/O. But this really adds up.

For this version thing in particular, the GYP build runs version.py a *lot* of
times over and over in the build. We've always been able to avoid doing the same
in the GN build.

Your use of this is somewhat different than the others. Usually version.py is
called to generate some header so my standard instructions won't work. But since
this is just being used to call generate_policy_source.py, that script itself
can just compute what it needs at build time (rather than GN/GYP time).

So your action below can just run version.py, or you can have the script load
the VERSION. Either way, be sure to add any files the script depends on
(VERSION/version.py or both) to the inputs of the action below.

[Thiemo Nagel, 2015/04/22 17:26:47]

Done.

+
   action("cloud_policy_code_generate") {
     script = "tools/generate_policy_source.py"
 
@@ -111,6 +124,7 @@ if (enable_configuration_policy) {
           rebase_path(app_restrictions_path, root_build_dir),
       "--app-restrictions-resources=" +
           rebase_path(app_resources_path, root_build_dir),
+      version_full,
       os,
       chromeos_flag,
       rebase_path("resources/policy_templates.json", root_build_dir),