Refactor print-stale-test-expectations-entries.

Refactor print-stale-test-expectations-entries.

In this CL:

 - Extract a function check_expectations_line
 - Extract a function write_csv
 - Remove unused imports
 - Print errors to stderr and include bug links in message

BUG=597797
Committed: https://crrev.com/75c5d0912c9c8052ec04aa66a056f346394e8abe
Cr-Commit-Position: refs/heads/master@{#404707}

[qyearsley, 2016-07-07 18:02:53]

Description was changed from

==========
Refactor print-stale-test-expectations-entries.
==========

to

==========
Refactor print-stale-test-expectations-entries.

In this CL:
 
 - Extract a function check_expectations_line
 - Remove unused imports
 - Print errors to stderr and include bug links in message


Purpose:
==========

[qyearsley, 2016-07-07 18:25:32]

Description was changed from

==========
Refactor print-stale-test-expectations-entries.

In this CL:
 
 - Extract a function check_expectations_line
 - Remove unused imports
 - Print errors to stderr and include bug links in message


Purpose:
==========

to

==========
Refactor print-stale-test-expectations-entries.

In this CL:
 
 - Extract a function check_expectations_line
 - Extract a function write_csv
 - Remove unused imports
 - Print errors to stderr and include bug links in message

BUG=597797
==========

[qyearsley, 2016-07-07 18:25:32]

qyearsley@chromium.org changed reviewers:
+ dpranke@chromium.org, nainar@chromium.org

[nainar, 2016-07-11 02:03:08]

lgtm

[qyearsley, 2016-07-11 17:42:19]

The CQ bit was checked by qyearsley@chromium.org

[qyearsley, 2016-07-11 17:42:19]

The patchset sent to the CQ was uploaded after l-g-t-m from nainar@chromium.org
Link to the patchset: https://codereview.chromium.org/2130823002/#ps40001
(title: "Rebased")

[commit-bot: I haz the power, 2016-07-11 17:43:01]

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-07-11 19:22:50]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-11 19:23:04]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-11 19:25:51]

Description was changed from

==========
Refactor print-stale-test-expectations-entries.

In this CL:
 
 - Extract a function check_expectations_line
 - Extract a function write_csv
 - Remove unused imports
 - Print errors to stderr and include bug links in message

BUG=597797
==========

to

==========
Refactor print-stale-test-expectations-entries.

In this CL:

 - Extract a function check_expectations_line
 - Extract a function write_csv
 - Remove unused imports
 - Print errors to stderr and include bug links in message

BUG=597797

Committed: https://crrev.com/75c5d0912c9c8052ec04aa66a056f346394e8abe
Cr-Commit-Position: refs/heads/master@{#404707}
==========

[commit-bot: I haz the power, 2016-07-11 19:25:52]

Patchset 3 (id:??) landed as
https://crrev.com/75c5d0912c9c8052ec04aa66a056f346394e8abe
Cr-Commit-Position: refs/heads/master@{#404707}

[Dirk Pranke, 2016-08-09 03:41:48]

lgtm (very belatedly).