Blink-in-JS: Output a correct JS file name and line number when an exception is thrown

Blink-in-JS: Output a correct JS file name and line number when an exception is thrown

Currently there is a bug on V8 APIs on Message objects, and they don't return valid values
when they are called at the first time if V8 has a pending exception.
We need to call twice in order to get valid values.

This is a temporary fix in the Blink side until we fix the V8 side.

BUG=341031
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180339

[haraken, 2014-08-15 04:51:33]

PTAL

[Jens Widell, 2014-08-15 04:55:30]

LGTM

https://codereview.chromium.org/478623002/diff/20001/Source/bindings/core/v8/...
File Source/bindings/core/v8/PrivateScriptRunner.cpp (right):

https://codereview.chromium.org/478623002/diff/20001/Source/bindings/core/v8/...
Source/bindings/core/v8/PrivateScriptRunner.cpp:29: // This is a bug of V8.
Is there a bug report?

[haraken, 2014-08-15 04:57:23]

https://codereview.chromium.org/478623002/diff/20001/Source/bindings/core/v8/...
File Source/bindings/core/v8/PrivateScriptRunner.cpp (right):

https://codereview.chromium.org/478623002/diff/20001/Source/bindings/core/v8/...
Source/bindings/core/v8/PrivateScriptRunner.cpp:29: // This is a bug of V8.
On 2014/08/15 04:55:29, Jens Widell wrote:
> Is there a bug report?

Not yet, I'm writing a fix :)

[haraken, 2014-08-15 04:57:26]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2014-08-15 04:58:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/haraken@chromium.org/478623002/20001

[haraken, 2014-08-15 05:20:31]

On 2014/08/15 04:57:23, haraken wrote:
>
https://codereview.chromium.org/478623002/diff/20001/Source/bindings/core/v8/...
> File Source/bindings/core/v8/PrivateScriptRunner.cpp (right):
> 
>
https://codereview.chromium.org/478623002/diff/20001/Source/bindings/core/v8/...
> Source/bindings/core/v8/PrivateScriptRunner.cpp:29: // This is a bug of V8.
> On 2014/08/15 04:55:29, Jens Widell wrote:
> > Is there a bug report?
> 
> Not yet, I'm writing a fix :)

I realized that a fix isn't that easy; filed a bug:
https://code.google.com/p/v8/issues/detail?id=3514

[commit-bot: I haz the power, 2014-08-15 06:16:28]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/22877)

[commit-bot: I haz the power, 2014-08-15 07:34:33]

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

[commit-bot: I haz the power, 2014-08-15 07:34:34]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/22887)

[haraken, 2014-08-15 07:35:06]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2014-08-15 07:36:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/haraken@chromium.org/478623002/20001

[commit-bot: I haz the power, 2014-08-15 08:09:08]

Committed patchset #2 (20001) as 180339