910653 - Errors thrown by loadSubScript do not get proper stack property

Status: RESOLVED INCOMPLETE

(Add-on SDK Graveyard :: General, defect, P3)

Description

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Here is a reproducible test case:

https://gist.github.com/Gozala/6376195

Comment 1

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Cc-ing dcamp as I was pointed out he's working on it

Comment 2

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Attachment: Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1227

Pointer to Github pull-request

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/7db663a31c5017a5ad5740070d4856bfa272b9c0
Merge pull request #1227 from Gozala/bug/workaround@910653

Bug 910653 - Workaround issues with syntax errors r=@zer0

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Irakli, is that workaround enough to call this fixed, or should it stay open for something else?

Comment 5

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

(In reply to Wes Kocher (:KWierso) from comment #4)
> Irakli, is that workaround enough to call this fixed, or should it stay open
> for something else?

I think we should ask our platform guys to take a look at the issue to do a proper fix on a subscriptloader.

Comment 6

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1399562