Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(146)

Side by Side Diff: scripts/slave/zip_build.py

Issue 110373008: Remove Chrome Frame tests everywhere. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """ Creates a zip file in the staging dir with the result of a compile. 6 """ Creates a zip file in the staging dir with the result of a compile.
7 It can be sent to other machines for testing. 7 It can be sent to other machines for testing.
8 """ 8 """
9 9
10 import csv 10 import csv
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 return ordered_asc_by_mtime_list[-prune_limit:] 103 return ordered_asc_by_mtime_list[-prune_limit:]
104 104
105 105
106 def FileRegexWhitelist(options): 106 def FileRegexWhitelist(options):
107 if chromium_utils.IsWindows() and options.target is 'Release': 107 if chromium_utils.IsWindows() and options.target is 'Release':
108 # Special case for chrome. Add back all the chrome*.pdb files to the list. 108 # Special case for chrome. Add back all the chrome*.pdb files to the list.
109 # Also add browser_test*.pdb, ui_tests.pdb and ui_tests.pdb. 109 # Also add browser_test*.pdb, ui_tests.pdb and ui_tests.pdb.
110 # TODO(nsylvain): This should really be defined somewhere else. 110 # TODO(nsylvain): This should really be defined somewhere else.
111 return (r'^(chrome[_.]dll|chrome[_.]exe' 111 return (r'^(chrome[_.]dll|chrome[_.]exe'
112 # r'|browser_test.+|unit_tests' 112 # r'|browser_test.+|unit_tests'
113 # r'|chrome_frame_.*tests'
114 r')\.pdb$') 113 r')\.pdb$')
115 114
116 return '$NO_FILTER^' 115 return '$NO_FILTER^'
117 116
118 117
119 def FileRegexBlacklist(options): 118 def FileRegexBlacklist(options):
120 if chromium_utils.IsWindows(): 119 if chromium_utils.IsWindows():
121 # Remove all .ilk/.7z and maybe PDB files 120 # Remove all .ilk/.7z and maybe PDB files
122 # TODO(phajdan.jr): Remove package_pdb_files when nobody uses it. 121 # TODO(phajdan.jr): Remove package_pdb_files when nobody uses it.
123 include_pdbs = options.factory_properties.get('package_pdb_files', True) 122 include_pdbs = options.factory_properties.get('package_pdb_files', True)
(...skipping 303 matching lines...) Expand 10 before | Expand all | Expand 10 after
427 426
428 if options.path_filter: 427 if options.path_filter:
429 options.path_filter = PATH_FILTERS[options.path_filter]( 428 options.path_filter = PATH_FILTERS[options.path_filter](
430 build_directory.GetBuildOutputDirectory(), options.target) 429 build_directory.GetBuildOutputDirectory(), options.target)
431 430
432 return Archive(options) 431 return Archive(options)
433 432
434 433
435 if '__main__' == __name__: 434 if '__main__' == __name__:
436 sys.exit(main(sys.argv)) 435 sys.exit(main(sys.argv))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698