Change the host_debug sky_shell used for testing to build with assertions enabled

Change the host_debug sky_shell used for testing to build with assertions enabled

BUG=

Committed: https://chromium.googlesource.com/chromium/tools/build/+/814cb05c7b410646c0e58e4c3e2dfd697a03d96c

[jsimmons1, 2016-08-19 19:30:37]

Description was changed from

==========
Change the host_debug sky_shell used for testing to build with assertions
enabled

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

to

==========
Change the host_debug sky_shell used for testing to build with assertions
enabled

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

[jsimmons1, 2016-08-19 19:30:37]

jsimmons@google.com changed reviewers:
+ abarth@google.com, eseidel@chromium.org

[Chinmay, 2016-08-19 19:43:47]

chinmaygarde@google.com changed reviewers:
+ chinmaygarde@google.com

[Chinmay, 2016-08-19 19:43:48]

lgtm

[eseidel, 2016-08-19 19:44:39]

eseidel@chromium.org changed reviewers:
- chinmaygarde@google.com

[eseidel, 2016-08-19 19:44:40]

lgtm

https://codereview.chromium.org/2260133003/diff/1/scripts/slave/recipes/flutt...
File scripts/slave/recipes/flutter/engine.expected/linux.json (right):

https://codereview.chromium.org/2260133003/diff/1/scripts/slave/recipes/flutt...
scripts/slave/recipes/flutter/engine.expected/linux.json:145: "name": "gn
--unoptimized (2)"
When names are not unique they get (2) appended, etc.  This can be worked around
by explicitly naming the step when calling api.step, which RunGN does under the
covers I'm sure.

https://codereview.chromium.org/2260133003/diff/1/scripts/slave/recipes/flutt...
File scripts/slave/recipes/flutter/engine.py (right):

https://codereview.chromium.org/2260133003/diff/1/scripts/slave/recipes/flutt...
scripts/slave/recipes/flutter/engine.py:163: RunGN(api, '--unoptimized')
We should probably pass some sort of name to RunGN so it can make the name which
appears in the buildbot be unique.  But that can also be a later change.

[eseidel, 2016-08-19 19:44:45]

The CQ bit was checked by eseidel@chromium.org

[commit-bot: I haz the power, 2016-08-19 19:44: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-08-19 19:49:07]

Description was changed from

==========
Change the host_debug sky_shell used for testing to build with assertions
enabled

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

to

==========
Change the host_debug sky_shell used for testing to build with assertions
enabled

BUG=

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/814cb05c7b410646c0e5...
==========

[commit-bot: I haz the power, 2016-08-19 19:49:08]

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