Small MessageLocation related refactoring.

Small MessageLocation related refactoring.

R=cbruni@chromium.org

Committed: https://crrev.com/1cdcae94c199a450110a776edf65fd1fea700459
Cr-Commit-Position: refs/heads/master@{#30305}

[Camillo Bruni, 2015-08-21 12:12:21]

lgtm

https://codereview.chromium.org/1309673003/diff/1/src/messages.cc
File src/messages.cc (right):

https://codereview.chromium.org/1309673003/diff/1/src/messages.cc#newcode49
src/messages.cc:49: script_handle =
Script::GetWrapper(isolate->factory()->empty_script());
The case used to default to end=-1 and start=-1 for the empty script.

[Yang, 2015-08-21 12:45:13]

On 2015/08/21 12:12:21, cbruni wrote:
> lgtm
> 
> https://codereview.chromium.org/1309673003/diff/1/src/messages.cc
> File src/messages.cc (right):
> 
> https://codereview.chromium.org/1309673003/diff/1/src/messages.cc#newcode49
> src/messages.cc:49: script_handle =
> Script::GetWrapper(isolate->factory()->empty_script());
> The case used to default to end=-1 and start=-1 for the empty script.

As discussed offline, I don't think this is an issue.

[Yang, 2015-08-21 12:47:00]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2015-08-21 12:47:00]

The patchset sent to the CQ was uploaded after l-g-t-m from cbruni@chromium.org
Link to the patchset: https://codereview.chromium.org/1309673003/#ps20001
(title: "keep old default values")

[commit-bot: I haz the power, 2015-08-21 12:47:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1309673003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1309673003/20001

[commit-bot: I haz the power, 2015-08-21 13:18:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-21 13:19:17]

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