CSP: Add line numbers to reports whenever we have them.

CSP: Add line numbers to reports whenever we have them.

Following up on https://codereview.chromium.org/2020053002, we have
line numbers in a variety of cases today, but we're only dumping them
into the reports when a script file generated the error. This patch
ensures that line/column data is appended whenever it's available.

BUG=615862
Committed: https://crrev.com/5b144f48b84c94de36d4bb842f5a72387beb4612
Cr-Commit-Position: refs/heads/master@{#398006}

[Mike West, 2016-06-03 11:08:19]

mkwst@chromium.org changed reviewers:
+ philip@chromium.org

[Mike West, 2016-06-03 11:08:20]

Mind taking a look at this, Philip? Tiny change, I hope. :)

[Mike West, 2016-06-03 12:21:57]

The CQ bit was checked by mkwst@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-03 12:22:10]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2032793004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2032793004/1

[commit-bot: I haz the power, 2016-06-03 13:09:02]

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

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

Dry run: This issue passed the CQ dry run.

[Mike West, 2016-06-06 07:37:45]

On 2016/06/03 at 13:09:03, commit-bot wrote:
> Dry run: This issue passed the CQ dry run.

Friendly ping. :)

[Mike West, 2016-06-06 07:37:53]

The CQ bit was checked by mkwst@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-06 07:38:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2032793004/1

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

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

[commit-bot: I haz the power, 2016-06-06 09:19:54]

Dry run: This issue passed the CQ dry run.

[Mike West, 2016-06-06 11:08:46]

mkwst@chromium.org changed reviewers:
+ jochen@chromium.org
- philip@chromium.org

[Mike West, 2016-06-06 11:09:28]

mkwst@chromium.org changed reviewers:
+ yoav@yoav.ws

[Mike West, 2016-06-06 11:09:28]

Ah, Philip is OOO. Jochen or Yoav, would one of you mind taking a look at this
tiny change?

[Yoav Weiss, 2016-06-06 11:11:39]

On 2016/06/06 11:09:28, Mike West (OOO until 30th) wrote:
> Ah, Philip is OOO. Jochen or Yoav, would one of you mind taking a look at this
> tiny change?

LGTM

[Mike West, 2016-06-06 11:51:41]

The CQ bit was checked by mkwst@chromium.org

[commit-bot: I haz the power, 2016-06-06 11:51:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2032793004/1

[commit-bot: I haz the power, 2016-06-06 11:56:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-06 11:58:03]

Description was changed from

==========
CSP: Add line numbers to reports whenever we have them.

Following up on https://codereview.chromium.org/2020053002, we have
line numbers in a variety of cases today, but we're only dumping them
into the reports when a script file generated the error. This patch
ensures that line/column data is appended whenever it's available.

BUG=615862
==========

to

==========
CSP: Add line numbers to reports whenever we have them.

Following up on https://codereview.chromium.org/2020053002, we have
line numbers in a variety of cases today, but we're only dumping them
into the reports when a script file generated the error. This patch
ensures that line/column data is appended whenever it's available.

BUG=615862

Committed: https://crrev.com/5b144f48b84c94de36d4bb842f5a72387beb4612
Cr-Commit-Position: refs/heads/master@{#398006}
==========

[commit-bot: I haz the power, 2016-06-06 11:58:04]

Patchset 1 (id:??) landed as
https://crrev.com/5b144f48b84c94de36d4bb842f5a72387beb4612
Cr-Commit-Position: refs/heads/master@{#398006}