[source-positions] make the SourcePositionInfo constructor actually compute position info

[source-positions] make the SourcePositionInfo constructor actually compute position info

R=danno@chromium.org

BUG=

Committed: https://crrev.com/859414da4c413da6732475ce92d6b83fb1bb8fb3
Cr-Commit-Position: refs/heads/master@{#41693}

[Tobias Tebbi, 2016-12-14 11:07:25]

This should fix a regression for fullcodegen source positions introduced by your
usage of the formerly only partially initializing SourcePositionInfo constructor
in https://codereview.chromium.org/2558283002/.
Sorry for the unclear code review.

[danno, 2016-12-14 11:42:31]

Thanks a lot for fixing this! Very cool. lgtm.

[Tobias Tebbi, 2016-12-14 12:22:58]

The CQ bit was checked by tebbi@chromium.org

[commit-bot: I haz the power, 2016-12-14 12:23:11]

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-12-14 12:57:46]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481718178919920, "parent_rev":
"e625be69866dc5b1d1da1aa0217bd769a56d2b3f", "commit_rev":
"9af923c81c1d78df50b921183e77c247def181a2"}

[commit-bot: I haz the power, 2016-12-14 12:57:49]

Description was changed from

==========
[source-positions] make the SourcePositionInfo constructor actually compute
position info

R=danno@chromium.org

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

to

==========
[source-positions] make the SourcePositionInfo constructor actually compute
position info

R=danno@chromium.org

BUG=

Review-Url: https://codereview.chromium.org/2568303008
==========

[commit-bot: I haz the power, 2016-12-14 12:57:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-14 12:58:28]

Description was changed from

==========
[source-positions] make the SourcePositionInfo constructor actually compute
position info

R=danno@chromium.org

BUG=

Review-Url: https://codereview.chromium.org/2568303008
==========

to

==========
[source-positions] make the SourcePositionInfo constructor actually compute
position info

R=danno@chromium.org

BUG=

Committed: https://crrev.com/859414da4c413da6732475ce92d6b83fb1bb8fb3
Cr-Commit-Position: refs/heads/master@{#41693}
==========

[commit-bot: I haz the power, 2016-12-14 12:58:29]

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