Updating the script to include sizes and resource sizes benchmarks

Updating the script to include sizes and resource sizes benchmarks

BUG=chromium:575318
Submitted via gerrit without problems here: https://chromium-review.googlesource.com/c/459438/

[ashleymarie1, 2017-03-23 17:04:59]

ashleymarie@chromium.org changed reviewers:
+ nednguyen@google.com

[nednguyen, 2017-03-23 17:15:28]

https://codereview.chromium.org/2773743002/diff/40001/tools/perf/core/perf_da...
File tools/perf/core/perf_data_generator.py (right):

https://codereview.chromium.org/2773743002/diff/40001/tools/perf/core/perf_da...
tools/perf/core/perf_data_generator.py:850: 'sizes (mac)':
BenchmarkMetadata('tapted', None),
all these owner should be @chromium.org?

Maybe we should add some assertion about owner's email.

[nednguyen, 2017-03-23 17:41:33]

https://codereview.chromium.org/2773743002/diff/40001/tools/perf/benchmark.csv
File tools/perf/benchmark.csv (left):

https://codereview.chromium.org/2773743002/diff/40001/tools/perf/benchmark.cs...
tools/perf/benchmark.csv:162: tab_switching.five_blank_pages,,
what happens to these?

[ashleymarie1, 2017-03-23 22:00:40]

On 2017/03/23 17:15:28, nednguyen wrote:
>
https://codereview.chromium.org/2773743002/diff/40001/tools/perf/core/perf_da...
> File tools/perf/core/perf_data_generator.py (right):
> 
>
https://codereview.chromium.org/2773743002/diff/40001/tools/perf/core/perf_da...
> tools/perf/core/perf_data_generator.py:850: 'sizes (mac)':
> BenchmarkMetadata('tapted', None),
> all these owner should be @chromium.org?
> 
> Maybe we should add some assertion about owner's email.

Whoops, fixed it
I'm thinking we should add that assertion when we require that an email address
exist for each benchmark; that way we can check everything in the same place

[ashleymarie1, 2017-03-23 22:00:47]

https://codereview.chromium.org/2773743002/diff/40001/tools/perf/benchmark.csv
File tools/perf/benchmark.csv (left):

https://codereview.chromium.org/2773743002/diff/40001/tools/perf/benchmark.cs...
tools/perf/benchmark.csv:162: tab_switching.five_blank_pages,,
On 2017/03/23 17:41:33, nednguyen wrote:
> what happens to these?

I believe it was out of sync when I was submitting the initial change

[nednguyen, 2017-03-23 22:02:19]

lgtm

[ashleymarie1, 2017-03-23 22:20:42]

The CQ bit was checked by ashleymarie@chromium.org

[commit-bot: I haz the power, 2017-03-23 22:21:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-23 22:25:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-23 22:25:02]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[ashleymarie1, 2017-03-23 22:37:32]

The CQ bit was checked by ashleymarie@chromium.org

[ashleymarie1, 2017-03-23 22:37:32]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2773743002/#ps80001
(title: "Handling merge from git pull")

[commit-bot: I haz the power, 2017-03-23 22:38:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-23 22:41:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-23 22:41:49]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ashleymarie1, 2017-03-24 13:26:19]

The CQ bit was checked by ashleymarie@chromium.org

[commit-bot: I haz the power, 2017-03-24 13:26:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-24 13:28:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-24 13:28:43]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[ashleymarie1, 2017-03-24 13:42:27]

The CQ bit was checked by ashleymarie@chromium.org

[ashleymarie1, 2017-03-24 13:42:28]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2773743002/#ps100001
(title: "Updating the script to include sizes and resource sizes benchmarks")

[commit-bot: I haz the power, 2017-03-24 13:42:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-24 13:44:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-24 13:44:37]

Try jobs failed on following builders:
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[ashleymarie1, 2017-03-24 13:52:10]

The CQ bit was checked by ashleymarie@chromium.org

[ashleymarie1, 2017-03-24 13:52:10]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2773743002/#ps120001
(title: "retrying upload after more git magic")

[commit-bot: I haz the power, 2017-03-24 13:52:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-24 13:54:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-24 13:54:48]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[ashleymarie1, 2017-03-24 16:28:46]

Description was changed from

==========
Updating the script to include sizes and resource sizes benchmarks

BUG=chromium:575318
==========

to

==========
Updating the script to include sizes and resource sizes benchmarks

BUG=chromium:575318

Submitted via gerrit without problems here:
https://chromium-review.googlesource.com/c/459438/
==========