Run Sizes on Windows perf builders (take 2)

Run Sizes on Windows perf builders (take 2)

(The first version of this patch was at https://codereview.chromium.org/1809783003/)

This is the same version as the first patch, but without passing --annotate=graphing
when running Sizes without a perf_id, as that apparently leads to failure (see crbug.com/596145).

BUG=457078
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299431

[hans, 2016-03-22 17:49:14]

Description was changed from

==========
Run Sizes on Windows perf builders (take 2)

(The first version of this patch was at
https://codereview.chromium.org/1809783003/)

This is the same version as the first patch, but without passing
--annotate=graphing
when running Sizes without a perf_id, as that apparently leds to failure.

BUG=457078
==========

to

==========
Run Sizes on Windows perf builders (take 2)

(The first version of this patch was at
https://codereview.chromium.org/1809783003/)

This is the same version as the first patch, but without passing
--annotate=graphing
when running Sizes without a perf_id, as that apparently leads to failure (see
crbug.com/596145).

BUG=457078
==========

[hans, 2016-03-22 17:52:33]

hans@chromium.org changed reviewers:
+ agable@chromium.org, qyearsley@chromium.org

[hans, 2016-03-22 17:52:34]

agable: Thanks for figuring out what was happening in crbug.com/596145

This removes the --annotate=graphic flag for this step when it doesn't have a
perf_id. Please take a look!

[agable, 2016-03-22 18:02:09]

This makes sense to me. LGTM.

[hans, 2016-03-22 18:05:10]

hans@chromium.org changed reviewers:
+ dtu@chromium.org, prasadv@chromium.org

[hans, 2016-03-22 18:05:11]

On 2016/03/22 18:02:09, agable wrote:
> This makes sense to me. LGTM.
Thanks!

+dtu, +prasadv: can you take an OWNERS look again?

[dtu, 2016-03-22 23:46:02]

lgtm

[hans, 2016-03-22 23:56:24]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2016-03-22 23:56:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1823183002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1823183002/1

[commit-bot: I haz the power, 2016-03-23 00:00:38]

Description was changed from

==========
Run Sizes on Windows perf builders (take 2)

(The first version of this patch was at
https://codereview.chromium.org/1809783003/)

This is the same version as the first patch, but without passing
--annotate=graphing
when running Sizes without a perf_id, as that apparently leads to failure (see
crbug.com/596145).

BUG=457078
==========

to

==========
Run Sizes on Windows perf builders (take 2)

(The first version of this patch was at
https://codereview.chromium.org/1809783003/)

This is the same version as the first patch, but without passing
--annotate=graphing
when running Sizes without a perf_id, as that apparently leads to failure (see
crbug.com/596145).

BUG=457078

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299431
==========

[commit-bot: I haz the power, 2016-03-23 00:00:39]

Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299431