[foozzie] Fix failure state for non-existing source output

[foozzie] Fix failure state for non-existing source output

BUG=chromium:673246
NOTRY=true
TBR=tandrii@chromium.org,mbarbella@chromium.org

Review-Url: https://codereview.chromium.org/2632623002
Cr-Commit-Position: refs/heads/master@{#42306}
Committed: https://chromium.googlesource.com/v8/v8/+/baf5a21751a837785718ed4d569599942fbb11e7

[Michael Achenbach, 2017-01-13 08:12:05]

Description was changed from

==========
[foozzie] Fix failure state for non-existing source output

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

to

==========
[foozzie] Fix failure state for non-existing source output

BUG=chromium:673246
NOTRY=true
TBR=tandrii@chromium.org,mbarbella@chromium.org
==========

[Michael Achenbach, 2017-01-13 08:12:06]

machenbach@chromium.org changed reviewers:
+ mbarbella@chromium.org, tandrii@chromium.org

[Michael Achenbach, 2017-01-13 08:13:40]

TBR

https://codereview.chromium.org/2632623002/diff/1/tools/foozzie/v8_suppressio...
File tools/foozzie/v8_suppressions.py (right):

https://codereview.chromium.org/2632623002/diff/1/tools/foozzie/v8_suppressio...
tools/foozzie/v8_suppressions.py:233: source_key =
hashlib.sha1(source).hexdigest()[:SOURCE_HASH_LENGTH]
Maybe not ideal that this is in this file. The hashing has nothing to do with
suppression. I'll probably refactor this soonish.

[Michael Achenbach, 2017-01-13 08:13:44]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2017-01-13 08:13:56]

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-01-13 08:15:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484295224756160, "parent_rev":
"00e98a39e3a3006e3c2ad593a3713a058fc40475", "commit_rev":
"baf5a21751a837785718ed4d569599942fbb11e7"}

[commit-bot: I haz the power, 2017-01-13 08:15:39]

Description was changed from

==========
[foozzie] Fix failure state for non-existing source output

BUG=chromium:673246
NOTRY=true
TBR=tandrii@chromium.org,mbarbella@chromium.org
==========

to

==========
[foozzie] Fix failure state for non-existing source output

BUG=chromium:673246
NOTRY=true
TBR=tandrii@chromium.org,mbarbella@chromium.org

Review-Url: https://codereview.chromium.org/2632623002
Cr-Commit-Position: refs/heads/master@{#42306}
Committed:
https://chromium.googlesource.com/v8/v8/+/baf5a21751a837785718ed4d569599942fb...
==========

[commit-bot: I haz the power, 2017-01-13 08:15:41]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/baf5a21751a837785718ed4d569599942fb...

[tandrii(chromium), 2017-01-13 08:20:31]

lgtm for short term fix.