Daily update script for performance numbers

Wrote a daily update script.
Committed: https://code.google.com/p/dart/source/detail?r=3623

[Emily Fortuna, 2012-01-26 00:45:21]

(this will eventually go on appengine with the rest of dartperf.googleplex.com)

[Siggi Cherem (dart-lang), 2012-01-26 02:33:18]

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
File tools/testing/perf_testing/calc_and_send_mail.py (right):

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:1: import create_graph
add header including:
- copyright
- #! line

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:21: #TODO Also, refactor
create_graph to make a separate
#TODO 
->
# TODO(efortuna):

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:46: browser_perf
BrowserPerformanceTestRunner object. Holds browser perf stats.
not sure if the style requires it or forbids it, but it would be nice to have a
symbol to divide the arg-name from the description (e.g. a ':')
[here and everywhere else in the CL]

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:134: # 
mail.send_mail(sender='"Emily Fortuna" <efortuna@google.com>',
Delete this entire comment

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
File tools/testing/perf_testing/create_graph.py (right):

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/create_graph.py:364:
self.style_and_save_perf_plot(title, y_axis, size_x, size_y, loc, filename,
80 col (here and below)

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
File tools/testing/perf_testing/get_current_stats.py (right):

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/get_current_stats.py:1: from create_graph import
BROWSER_CORRECTNESS
+ header here too

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/get_current_stats.py:41: timestamp =
timestamp_and_data[:timestamp_and_data.find('-')]
use index within this expression?
(timestamp = timestamp_and_data[:index]) ?

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/get_current_stats.py:43: while f[match.start():
match.start() + index] == timestamp:
space before ':' (symmetry)

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/get_current_stats.py:44: latest_files += [f]
->
latest_files.append(f)

[Emily Fortuna, 2012-01-26 18:04:07]

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
File tools/testing/perf_testing/calc_and_send_mail.py (right):

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:1: import create_graph
On 2012/01/26 02:33:18, sigmund wrote:
> add header including:
> - copyright
> - #! line

Done.

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:21: #TODO Also, refactor
create_graph to make a separate
On 2012/01/26 02:33:18, sigmund wrote:
> #TODO 
> ->
> # TODO(efortuna): 

Done.

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:46: browser_perf
BrowserPerformanceTestRunner object. Holds browser perf stats.
Done (added colons per the style guide).

On 2012/01/26 02:33:18, sigmund wrote:
> not sure if the style requires it or forbids it, but it would be nice to have
a
> symbol to divide the arg-name from the description (e.g. a ':')
> [here and everywhere else in the CL]

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:134: # 
mail.send_mail(sender='"Emily Fortuna" <efortuna@google.com>',
On 2012/01/26 02:33:18, sigmund wrote:
> Delete this entire comment

Done.

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/calc_and_send_mail.py:134: # 
mail.send_mail(sender='"Emily Fortuna" <efortuna@google.com>',
On 2012/01/26 02:33:18, sigmund wrote:
> Delete this entire comment

Done.

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
File tools/testing/perf_testing/create_graph.py (right):

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/create_graph.py:364:
self.style_and_save_perf_plot(title, y_axis, size_x, size_y, loc, filename,
On 2012/01/26 02:33:18, sigmund wrote:
> 80 col (here and below)

Done.

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
File tools/testing/perf_testing/get_current_stats.py (right):

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/get_current_stats.py:1: from create_graph import
BROWSER_CORRECTNESS
On 2012/01/26 02:33:18, sigmund wrote:
> + header here too

Done.

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/get_current_stats.py:41: timestamp =
timestamp_and_data[:timestamp_and_data.find('-')]
On 2012/01/26 02:33:18, sigmund wrote:
> use index within this expression?
> (timestamp = timestamp_and_data[:index]) ?

Done.

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/get_current_stats.py:43: while f[match.start():
match.start() + index] == timestamp:
On 2012/01/26 02:33:18, sigmund wrote:
> space before ':' (symmetry)

Done.

https://chromiumcodereview.appspot.com/9235046/diff/1/tools/testing/perf_test...
tools/testing/perf_testing/get_current_stats.py:44: latest_files += [f]
On 2012/01/26 02:33:18, sigmund wrote:
> ->
> latest_files.append(f)

Done.

[Siggi Cherem (dart-lang), 2012-01-26 19:19:53]

lgtm

https://chromiumcodereview.appspot.com/9235046/diff/1005/tools/testing/perf_t...
File tools/testing/perf_testing/calc_and_send_mail.py (right):

https://chromiumcodereview.appspot.com/9235046/diff/1005/tools/testing/perf_t...
tools/testing/perf_testing/calc_and_send_mail.py:127: values_tuple: the tuple of
values we are comparing the difference between."""
80

https://chromiumcodereview.appspot.com/9235046/diff/1005/tools/testing/perf_t...
File tools/testing/perf_testing/get_current_stats.py (right):

https://chromiumcodereview.appspot.com/9235046/diff/1005/tools/testing/perf_t...
tools/testing/perf_testing/get_current_stats.py:61: test_runner_dict: optional
agument to which we should """
extra period at the end of the sentence.