SuperSize: Set no-cache headers for hosted version

Turns out google storage sets a one hour public cache header by default,
which means a bad push results in the page being broken for an hour.

This leaves the default header on for .size files, but sets all other
files (which are tiny) to no-cache.

Note: the .wasm file is big, and should probably be cached, but that is handled
by the service worker already.

Bug: None
Change-Id: Ia3c03e4533c37338b924d5ace3f7b5fd4ac8d6f9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1981263
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Eric Stevenson <estevenson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#727618}

tools/binary_size/generate_milestone_reports.py

@@ -231,6 +231,10 @@ def main():
 
   if args.sync:
     subprocess.check_call(cmd)
+    subprocess.check_call([
+        'gsutil.py', 'setmeta', '-h', 'Cache-Control:no-cache',
+        _PUSH_URL + 'milestones.json'
+    ])
   else:
     print()
     print('Sync files by running:')

tools/binary_size/generate_official_build_report.py

@@ -71,7 +71,10 @@ def main():
 
   with tempfile.NamedTemporaryFile() as f:
     _WriteReportsJson(f)
-    cmd = ['gsutil.py', 'cp', '-a', 'public-read', f.name, _REPORTS_JSON_GS_URL]
+    cmd = [
+        'gsutil.py', '--', '-h', 'Cache-Control:no-cache', 'cp', '-a',
+        'public-read', f.name, _REPORTS_JSON_GS_URL
+    ]
     logging.warning(' '.join(cmd))
     subprocess.check_call(cmd)
 

tools/binary_size/libsupersize/upload_html_viewer.py

@@ -18,7 +18,8 @@ def _SyncStatic():
   """Upload static files from the static directory."""
   static_files = os.path.join(os.path.dirname(__file__), 'static')
   subprocess.check_call([
-    'gsutil.py', '-m', 'rsync', '-r', static_files, GS_BUCKET
+      'gsutil.py', '--', '-m', '-h'
+      'Cache-Control:no-cache', 'rsync', '-r', static_files, GS_BUCKET
   ])
 
 
@@ -28,8 +29,10 @@ def _SyncTemplates():
   cache_hash = uuid.uuid4().hex
 
   p = subprocess.Popen([
-    'gsutil.py', 'cp', '-p', '-', '%s/sw.js' % GS_BUCKET
-  ], stdin=subprocess.PIPE)
+      'gsutil.py', '--', '-h'
+      'Cache-Control:no-cache', 'cp', '-p', '-', GS_BUCKET + '/sw.js'
+  ],
+                       stdin=subprocess.PIPE)
   with open(template_file, 'r') as in_file:
     p.communicate(in_file.read().replace('{{cache_hash}}', cache_hash))
 
@@ -42,9 +45,8 @@ def _SetMetaAndPermissions():
   ])
 
   # All files in the root of the bucket are user readable
-  subprocess.check_call([
-    'gsutil.py', '-m', 'acl', 'ch', '-u', 'AllUsers:R', '%s/*' % GS_BUCKET
-  ])
+  subprocess.check_call(
+      ['gsutil.py', '-m', 'acl', 'ch', '-u', 'AllUsers:R', GS_BUCKET + '/*'])
 
 
 def main():