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

Side by Side Diff: tools/testing/perf_testing/run_perf_tests.py

Issue 20847003: Fix Dromaeo performance tests. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 4 months 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 | Annotate | Revision Log
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 2
3 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 3 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
4 # for details. All rights reserved. Use of this source code is governed by a 4 # for details. All rights reserved. Use of this source code is governed by a
5 # BSD-style license that can be found in the LICENSE file. 5 # BSD-style license that can be found in the LICENSE file.
6 6
7 import datetime 7 import datetime
8 import math 8 import math
9 import optparse 9 import optparse
10 import os 10 import os
(...skipping 409 matching lines...) Expand 10 before | Expand all | Expand 10 after
420 self.revision_dict[platform][f][val] = [] 420 self.revision_dict[platform][f][val] = []
421 self.values_dict[platform][f][val] = [] 421 self.values_dict[platform][f][val] = []
422 for extra_metric in extra_metrics: 422 for extra_metric in extra_metrics:
423 self.revision_dict[platform][f][extra_metric] = [] 423 self.revision_dict[platform][f][extra_metric] = []
424 self.values_dict[platform][f][extra_metric] = [] 424 self.values_dict[platform][f][extra_metric] = []
425 425
426 def IsValidCombination(self, platform, variant): 426 def IsValidCombination(self, platform, variant):
427 """Check whether data should be captured for this platform/variant 427 """Check whether data should be captured for this platform/variant
428 combination. 428 combination.
429 """ 429 """
430 if variant == 'dart_html' and platform != 'dartium':
431 return False
430 if platform == 'dartium' and (variant == 'js' or variant == 'dart2js_html'): 432 if platform == 'dartium' and (variant == 'js' or variant == 'dart2js_html'):
431 # Testing JavaScript performance on Dartium is a waste of time. Should be 433 # Testing JavaScript performance on Dartium is a waste of time. Should be
432 # same as Chrome. 434 # same as Chrome.
433 return False 435 return False
434 if (platform == 'safari' and variant == 'dart2js' and 436 if (platform == 'safari' and variant == 'dart2js' and
435 int(self.test_runner.current_revision_num) < 10193): 437 int(self.test_runner.current_revision_num) < 10193):
436 # In revision 10193 we fixed a bug that allows Safari 6 to run dart2js 438 # In revision 10193 we fixed a bug that allows Safari 6 to run dart2js
437 # code. Since we can't change the Safari version on the machine, we're 439 # code. Since we can't change the Safari version on the machine, we're
438 # just not running 440 # just not running
439 # for this case. 441 # for this case.
(...skipping 689 matching lines...) Expand 10 before | Expand all | Expand 10 after
1129 else: 1131 else:
1130 if runner.backfill: 1132 if runner.backfill:
1131 results_set = FillInBackHistory(results_set, runner) 1133 results_set = FillInBackHistory(results_set, runner)
1132 else: 1134 else:
1133 time.sleep(200) 1135 time.sleep(200)
1134 else: 1136 else:
1135 runner.RunTestSequence() 1137 runner.RunTestSequence()
1136 1138
1137 if __name__ == '__main__': 1139 if __name__ == '__main__':
1138 main() 1140 main()
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698