Changed sequence of package_build step

Changed sequence of package_build step to be before package_build_for_bisect

BUG=645662
Committed: https://chromium.googlesource.com/chromium/tools/build/+/2f0a7c064259dc0e0bf375c2d1771544f5647a1c

[miimnk, 2016-09-13 21:40:55]

Description was changed from

==========
merged


moved sequence of package_build steps


resolved merge conflicts


fixed bug with previous land


merged


lightweight builds archiving for mac and win64


mac,win,win64 support

BUG=
==========

to

==========
Changed sequence of package_build step to be before package_build_for_bisect

BUG=645662
==========

[miimnk, 2016-09-13 21:40:55]

miimnk@google.com changed reviewers:
+ dtu@chromium.org, prasadv@chromium.org

[miimnk, 2016-09-13 21:44:17]

Hi Dave and Prasad. 

This issue is related to
https://bugs.chromium.org/p/chromium/issues/detail?id=645662

Please take a look and review.

Best,

Min

[prasadv, 2016-09-13 21:49:47]

lgtm

[dtu, 2016-09-13 21:53:58]

lgtm

[miimnk, 2016-09-13 21:57:29]

miimnk@google.com changed reviewers:
+ mmoss@chromium.org, stip@chromium.org

[miimnk, 2016-09-13 21:57:58]

miimnk@google.com changed reviewers:
+ mmoss@google.com
- mmoss@chromium.org

[miimnk, 2016-09-13 21:59:59]

The CQ bit was checked by miimnk@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 22:00:11]

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

[Michael Moss, 2016-09-13 22:04:29]

mmoss@chromium.org changed reviewers:
+ mmoss@chromium.org

[Michael Moss, 2016-09-13 22:04:30]

lgtm

[commit-bot: I haz the power, 2016-09-13 22:06:52]

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

[commit-bot: I haz the power, 2016-09-13 22:06:53]

Dry run: This issue passed the CQ dry run.

[ghost stip (do not use), 2016-09-13 22:31:34]

lgtm

[miimnk, 2016-09-15 21:11:46]

The CQ bit was checked by miimnk@google.com

[commit-bot: I haz the power, 2016-09-15 21:11:53]

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, 2016-09-15 21:18:49]

Description was changed from

==========
Changed sequence of package_build step to be before package_build_for_bisect

BUG=645662
==========

to

==========
Changed sequence of package_build step to be before package_build_for_bisect

BUG=645662

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/2f0a7c064259dc0e0bf3...
==========

[commit-bot: I haz the power, 2016-09-15 21:18:50]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/2f0a7c064259dc0e0bf3...

[dtu, 2016-09-19 19:08:40]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2352833002/ by dtu@chromium.org.

The reason for reverting is: Turns out this had an unintended side effect. The
"package build" step sets the "build_archive_url" property, so now the perf
testers are using the wrong build archive.

First failing tests:
https://build.chromium.org/p/chromium.perf/builders/Win%207%20ATI%20GPU%20Per...
https://build.chromium.org/p/chromium.perf/builders/Win%207%20Nvidia%20GPU%20....