Handle exceptions in check_gn_headers.py

Handle exceptions in check_gn_headers.py

If exceptions are thrown, the main process would wait forever if they
are not handled. This could potentially cause timeout on the bots.

BUG=661774
Review-Url: https://codereview.chromium.org/2903063006
Cr-Commit-Position: refs/heads/master@{#474873}
Committed: https://chromium.googlesource.com/chromium/src/+/09692cdc03a2723afadf7109b2b8404a9eb497ed

[wychen, 2017-05-25 21:03:29]

wychen@chromium.org changed reviewers:
+ dpranke@chromium.org

[wychen, 2017-05-25 21:03:29]

PTAL

[wychen, 2017-05-25 22:26:09]

Description was changed from

==========
Handle CalledProcessError in check_gn_headers.py

If CalledProcessError is thrown, the main process would wait forever
if it's not handled. This could potentially cause timeout on the bots.

BUG=661774
==========

to

==========
Handle exceptions in check_gn_headers.py

If exceptions are thrown, the main process would wait forever if they
are not handled. This could potentially cause timeout on the bots.

BUG=661774
==========

[wychen, 2017-05-25 22:51:08]

Description was changed from

==========
Handle exceptions in check_gn_headers.py

If exceptions are thrown, the main process would wait forever if they
are not handled. This could potentially cause timeout on the bots.

BUG=661774
==========

to

==========
Handle exceptions in check_gn_headers.py

If exceptions are thrown, the main process would wait forever if they
are not handled. This could potentially cause timeout on the bots.

BUG=661774
==========

[Dirk Pranke, 2017-05-25 22:56:00]

lgtm

[wychen, 2017-05-26 00:41:22]

The CQ bit was checked by wychen@chromium.org

[commit-bot: I haz the power, 2017-05-26 00:43:10]

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-05-26 01:57:19]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495759282770140,
"parent_rev": "4f62b873dc5532897db517f451f47b6053d652b2", "commit_rev":
"09692cdc03a2723afadf7109b2b8404a9eb497ed"}

[commit-bot: I haz the power, 2017-05-26 01:57:35]

Description was changed from

==========
Handle exceptions in check_gn_headers.py

If exceptions are thrown, the main process would wait forever if they
are not handled. This could potentially cause timeout on the bots.

BUG=661774
==========

to

==========
Handle exceptions in check_gn_headers.py

If exceptions are thrown, the main process would wait forever if they
are not handled. This could potentially cause timeout on the bots.

BUG=661774

Review-Url: https://codereview.chromium.org/2903063006
Cr-Commit-Position: refs/heads/master@{#474873}
Committed:
https://chromium.googlesource.com/chromium/src/+/09692cdc03a2723afadf7109b2b8...
==========

[commit-bot: I haz the power, 2017-05-26 01:57:36]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/09692cdc03a2723afadf7109b2b8...