[V8] Report JSON parser script to DevTools

[V8] Report JSON parser script to DevTools

If JSON contains SyntaxError then V8 will report exception and won't report compile error.

LOG=Y
BUG=chromium:515382
R=yangguo@chromium.org,yurys@chromium.org

Committed: https://crrev.com/951f6b7a88fccbe14a6ddd8611e7bd98f9f7f7b1
Cr-Commit-Position: refs/heads/master@{#30426}

[kozy, 2015-08-27 00:03:49]

Yury and Yang, please take a look!

[yurys, 2015-08-27 00:06:57]

https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h
File src/json-parser.h (right):

https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h#newcode249
src/json-parser.h:249: isolate()->debug()->OnAfterCompile(script);
Can you add a test for this please?

[Yang, 2015-08-27 10:41:04]

https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h
File src/json-parser.h (right):

https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h#newcode249
src/json-parser.h:249: isolate()->debug()->OnAfterCompile(script);
And please add a comment on why this is necessary.

[kozy, 2015-08-27 18:21:41]

Patchset #2 (id:20001) has been deleted

[kozy, 2015-08-27 18:22:06]

All done. Please take a look.

https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h
File src/json-parser.h (right):

https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h#newcode249
src/json-parser.h:249: isolate()->debug()->OnAfterCompile(script);
On 2015/08/27 00:06:57, yurys wrote:
> Can you add a test for this please?

Done.

https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h#newcode249
src/json-parser.h:249: isolate()->debug()->OnAfterCompile(script);
On 2015/08/27 10:41:04, Yang wrote:
> And please add a comment on why this is necessary.

Done.

[Yang, 2015-08-27 18:43:46]

On 2015/08/27 18:22:06, kozyatinskiy wrote:
> All done. Please take a look.
> 
> https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h
> File src/json-parser.h (right):
> 
> https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h#newcode249
> src/json-parser.h:249: isolate()->debug()->OnAfterCompile(script);
> On 2015/08/27 00:06:57, yurys wrote:
> > Can you add a test for this please?
> 
> Done.
> 
> https://codereview.chromium.org/1308123006/diff/1/src/json-parser.h#newcode249
> src/json-parser.h:249: isolate()->debug()->OnAfterCompile(script);
> On 2015/08/27 10:41:04, Yang wrote:
> > And please add a comment on why this is necessary.
> 
> Done.

lgtm. thanks

[kozy, 2015-08-27 18:45:52]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2015-08-27 18:46:02]

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

[kozy, 2015-08-27 18:47:26]

The CQ bit was unchecked by kozyatinskiy@chromium.org

[yurys, 2015-08-27 18:47:37]

lgtm

[kozy, 2015-08-27 18:47:39]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2015-08-27 18:47:53]

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

[commit-bot: I haz the power, 2015-08-27 19:01:08]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-08-27 19:01:33]

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