749308 - Syntax errors don't show line numbers

Status: RESOLVED FIXED

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

Description

[Jeff Balogh (:jbalogh)]

Traceback (most recent call last):
  File "chrome://browser/content/browser.js", line 5211, in delayedStartu
    .init(window);
  File "resource:///components/nsSessionStore.js", line 440, in sss_ini
    this.onLoad(aWindow);
  File "resource:///components/nsSessionStore.js", line 627, in sss_onLoa
    Services.obs.notifyObservers(null, NOTIFY_WINDOWS_RESTORED, "");
  File "resource://push-at-jbalogh-dot-me/api-utils/lib/system/events.js", line 58, in 
    data: data
  File "resource://push-at-jbalogh-dot-me/api-utils/lib/addon/runner.js", line 57, in 
    startup(null, options);
  File "resource://push-at-jbalogh-dot-me/api-utils/lib/addon/runner.js", line 86, in startu
    let program = load(loader, loader.main).exports;
  File "resource://push-at-jbalogh-dot-me/api-utils/lib/cuddlefish.js", line 177, in loa
    evaluate(sandbox, module.uri);
  File "resource://push-at-jbalogh-dot-me/api-utils/lib/cuddlefish.js", line 142, in evaluat
    : scriptLoader.loadSubScript(uri, sandbox, encoding);
SyntaxError: missing ) after argument list

I make mistakes when I type and then I'm sad because I have no idea where it happened.

Comment 1

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

Jeff, could you try again with the current github master? Alex might have fixed this last week.

Comment 2

[Jeff Balogh (:jbalogh)]

Still no line numbers on master.

Comment 3

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

Can you put up a copy of your addon when it is giving you this error without the line numbers?

Comment 4

[Jeff Balogh (:jbalogh)]

Attachment: minimal jetpack with a syntax error in main

Comment 5

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

Hrm, when I run that from Builder on a recent Nightly build, I get the following three errors:

Timestamp: 5/4/2012 1:00:38 PM
Error: SyntaxError: missing ) after argument list
Source File: resource://jid0-2vpfeegebdndru2ba9tkyogsf68-at-jetpack/api-utils/lib/cuddlefish.js
Line: 250

Timestamp: 5/4/2012 1:00:38 PM
Error: An exception occurred.
Traceback (most recent call last):
  File "data:,let%20loader%20%3D%20Loader.new(%7B%22allTestModules%22%3A%5B%5D%2C%22enable_e10s%22%3Afalse%2C%22jetpackID%22%3A%22jid0-2vPfEeGebdnDrU2ba9tKyOgSf68%40jetpack%22%2C%22loader%22%3A%22api-utils%2Flib%2Fcuddlefish.js%22%2C%22main%22%3A%22main%22%2C%22mainPath%22%3A%22kwierso-117%2Flib%2Fmain.js%22%2C%22manifest%22%3A%7B%22api-utils%2Flib%2Fbase.js%22%3A%7B%22docsSHA256%22%3A%2292ddcf82c81cc72f046ec33da53b19e2ce67dd592b92aedad1f26b98a490c3cd%22%2C%22jsSHA256%22%3A%2259a13dee35dcb90d0805aaa895028f02b793448747637f02ecd75736131a6258%22%2C%22moduleName%22%3A%22base%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fbyte-streams.js%22%3A%7B%22docsSHA256%22%3A%220ed31570a32a427c27c1eed11a003dd9fdd9066d26cfc3a2e26e8b64bbdd7411%22%2C%22jsSHA256%22%3A%2270fb9a7ec33b9dda9b70e0d5c6b0234362e30c35f269027269b54a1d6f130eea%22%2C%22moduleName%22%3A%22byte-streams%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fchannel.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%22811b1c25c0220549e258e88057c93ade230d3a49dee7d8d6c72ee832b1ec7f9d%22%2C%22moduleName%22%3A%22channel%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%2C%22%40packaging%22%3A%7B%22path%22%3A%22%40packaging%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fcuddlefish.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%2296a0c54932b761ac7e30b61b6c716b862f69abcd58fa66c182f3cc3cab142aee%22%2C%22moduleName%22%3A%22cuddlefish%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22%40packaging%22%3A%7B%22path%22%3A%22%40packaging%22%7D%2C%22api-utils%2Fglobals!%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fglobals!.js%22%7D%2C%22api-utils%2Fprocess%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fprocess.js%22%7D%2C%22api-utils%2Fself!%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fself!.js%22%7D%2C%22api-utils%2Fsystem%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fsystem.js%22%7D%2C%22api-utils%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fenvironment.js%22%3A%7B%22docsSHA256%22%3A%221a8f64c8e3f83f13a9c1fa0e1196cf65555a63fd68eaf45c440792cf8bed7479%22%2C%22jsSHA256%22%3A%22d96f4609263ef09f541cbe4a9df8532594fbb01f642b0af632b71c95e70c4b0c%22%2C%22moduleName%22%3A%22environment%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Ferrors.js%22%3A%7B%22docsSHA256%22%3A%22e141cd31bd7d5a6fb64019470d4e24acaa7e98a449a88afd9c23b9b743e9dcc5%22%2C%22jsSHA256%22%3A%22a9f3c34b8f0ed4b45c08e3691f934610cf4568d0c7031073df4e8c792b4c211e%22%2C%22moduleName%22%3A%22errors%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fevents.js%22%3A%7B%22docsSHA256%22%3A%2240e48810b4d8d6926935abc2b7f7e49c4d1b7d51485f421bd43950bc3268d7f4%22%2C%22jsSHA256%22%3A%227d52b7d9931d4f94cb2da1745165f399f9ce637df0c25969259e14140a024b73%22%2C%22moduleName%22%3A%22events%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Flight-traits%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Flight-traits.js%22%7D%2C%22.%2Ftraits%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ftraits.js%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Ffile.js%22%3A%7B%22docsSHA256%22%3A%22a508f419ab1981dcd51439dec49f5833db3d4c7cfcc396fe90386d502f03bb0d%22%2C%22jsSHA256%22%3A%22d9f0ff832e14d93aa74150db5f69ad57c2f28248f69d726d164cb118c8717b29%22%2C%22moduleName%22%3A%22file%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Fbyte-streams%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fbyte-streams.js%22%7D%2C%22.%2Ftext-streams%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ftext-streams.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Ffunctional.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%22848e921e776f4bdd5c132bdf451442e8a7a1e482f91963bc8b6f63dec0b09135%22%2C%22moduleName%22%3A%22functional%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Ftimer%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ftimer.js%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fglobals!.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%22c4a4bd2124e12349cb5793fb50f64e2cc8f1ea2505f048e322d52714bda68c5e%22%2C%22moduleName%22%3A%22globals!%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Fmemory%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fmemory.js%22%7D%2C%22.%2Fplain-text-console%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fplain-text-console.js%22%7D%2C%22%40packaging%22%3A%7B%22path%22%3A%22%40packaging%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Flight-traits.js%22%3A%7B%22docsSHA256%22%3A%225d27c215d1f1104010c0c71f3f44f79b2530e9b1afa08db6e60ef5458a3a6a77%22%2C%22jsSHA256%22%3A%228d00d1661a149cf6e1ac50354418c4ac24e9d692b40ef8976ae481373d8bfb2c%22%2C%22moduleName%22%3A%22light-traits%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fmemory.js%22%3A%7B%22docsSHA256%22%3A%225bb4d443a8392684c9cd1e3ebf8d4c85b513a5927dd7214fdb2e631f73d01526%22%2C%22jsSHA256%22%3A%22f41b28860ac61fc69d05be491fc52d0babf109c25aee9a464db18bacd6b9bfd5%22%2C%22moduleName%22%3A%22memory%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fmessage-manager.js%22%3A%7B%22docsSHA256%22%3A%226af8abd732f40b9e3b19d8048d0bf32a66758420049ebfde74c69e21e7a8ae5a%22%2C%22jsSHA256%22%3A%2246158923434f5a0672d63967afa314f2a87470313efc15b719d4aa178639378c%22%2C%22moduleName%22%3A%22message-manager%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Ffunctional%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ffunctional.js%22%7D%2C%22.%2Fnamespace%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fnamespace.js%22%7D%2C%22.%2Fsandbox%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fsandbox.js%22%7D%2C%22.%2Ftimer%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ftimer.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fnamespace.js%22%3A%7B%22docsSHA256%22%3A%229039b94ab6b4c3300e71e805db3ffd83948d4f7e2d84ff3e9324a86a20585f83%22%2C%22jsSHA256%22%3A%22b00b2f4ea8af0839f70b003ac9ebcd845bc29f70f9a8a2bc4642152fd88ddf97%22%2C%22moduleName%22%3A%22namespace%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fplain-text-console.js%22%3A%7B%22docsSHA256%22%3A%221140c5a2ea5178be883254a2afb371e1550b863adeab5434a7cba9ce9f573d1c%22%2C%22jsSHA256%22%3A%2242c7855906a65e3b39451245c6e5f682494bac2c6557b6ace14faecb714cbff6%22%2C%22moduleName%22%3A%22plain-text-console%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Ftraceback%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ftraceback.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fprocess.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%222d1ccc5c3d9c01e221b6f9a8d13a156f8d71f50c441514a796eb1c925b071002%22%2C%22moduleName%22%3A%22process%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Fchannel%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fchannel.js%22%7D%2C%22.%2Fmessage-manager%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fmessage-manager.js%22%7D%2C%22.%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%2C%22.%2Fxul-app%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fxul-app.js%22%7D%2C%22%40packaging%22%3A%7B%22path%22%3A%22%40packaging%22%7D%2C%22api-utils%2Fwindow-utils%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fwindow-utils.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fruntime.js%22%3A%7B%22docsSHA256%22%3A%22da3d34aa242a4a1a78874ef3458f8288a8e16881679b7cfe9c7340a7af3783f6%22%2C%22jsSHA256%22%3A%22c890d46b04876a2bdebcc9fade057fea2ba0b484e3f0be6a8e27d6bb59e7eab3%22%2C%22moduleName%22%3A%22runtime.js%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fsandbox.js%22%3A%7B%22docsSHA256%22%3A%225b0f006528fc71bae1ce2b7a06b6b80f366d88b0f881c9de822f3ca47a39187f%22%2C%22jsSHA256%22%3A%225d5fcb3d381de91cf2a208d82f0d84eb353b0dd69daac86ef96cd1aaefa02c75%22%2C%22moduleName%22%3A%22sandbox%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fself!.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%228d19434a3d47c325ef3e54dbf383f7f56e7a1fe8328f36862d87e4d803504d91%22%2C%22moduleName%22%3A%22self!%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22%40packaging%22%3A%7B%22path%22%3A%22%40packaging%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fsystem.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%2277da3de84feb592f24d2bf326d72e45960ebcc4296f8cb49406048f87da718ea%22%2C%22moduleName%22%3A%22system%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Fenvironment%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fenvironment.js%22%7D%2C%22.%2Ffile%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ffile.js%22%7D%2C%22.%2Fruntime.js%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fruntime.js%22%7D%2C%22%40packaging%22%3A%7B%22path%22%3A%22%40packaging%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Ftext-streams.js%22%3A%7B%22docsSHA256%22%3A%228e19169ffafcda9c319bb1e12068bfe2612eff2268d02765d3ea506e22c93e4d%22%2C%22jsSHA256%22%3A%221290e60106ce88356cef2f78ee594082a711063351825732037542920a779225%22%2C%22moduleName%22%3A%22text-streams%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Ftimer.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%229d3680b71cb6e9e12ef982c98a17e0a846e8745ac12570e8b5b5d8212511ef4e%22%2C%22moduleName%22%3A%22timer%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%2C%22.%2Fxpcom%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fxpcom.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Ftraceback.js%22%3A%7B%22docsSHA256%22%3A%22e08c1fd5421087d68e608f7f179d4212a08ce7460fd6939834a2ace59badefa8%22%2C%22jsSHA256%22%3A%220487cabe5f96891764019d323a69bd5ef380f01ad88c8080194f474f36ca1bda%22%2C%22moduleName%22%3A%22traceback%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Furl%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Furl.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Ftraits.js%22%3A%7B%22docsSHA256%22%3A%227e6f1fb1d18a7abbde3a61f5db0ef61b24eb3adc77a07ce012fdef5a3db25ac0%22%2C%22jsSHA256%22%3A%22b6b6bebf59c46adf01bcb80f8a97aa354b7a6eb912733ae5521097f25bf08fc3%22%2C%22moduleName%22%3A%22traits%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Ftraits%2Fcore%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ftraits%2Fcore.js%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Ftraits%2Fcore.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%223a89ea0ad6095bd1e318549e622fd9a030373d04d329315a3a182e91c09614c5%22%2C%22moduleName%22%3A%22traits%2Fcore%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Funload.js%22%3A%7B%22docsSHA256%22%3A%223eac715867dc4fabccc115e6684cf05abdef4a0542274fdfce640d682c11b6b4%22%2C%22jsSHA256%22%3A%22f684707fd7ddaf2eafba0c1de2c6731792c60095bfec28deb03a0f202c725e3b%22%2C%22moduleName%22%3A%22unload%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Furl.js%22%3A%7B%22docsSHA256%22%3A%2240ece4f227368bea31efc12f5b050564b5ef0fec0102fe93edf2dc42ddda0c0d%22%2C%22jsSHA256%22%3A%221d7259d85252f34bc0a04adfae30cda97a2e7d49b2070087e9da72b746348ce2%22%2C%22moduleName%22%3A%22url%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fuuid.js%22%3A%7B%22docsSHA256%22%3A%227c1e57b8153ecb1c87de5fdd686684e0017d2ba16f43c647d629edf638a7f718%22%2C%22jsSHA256%22%3A%22f0dee5801e578994f6c69db12713062a103c380ed55aa0632a1f02c14a9d2a32%22%2C%22moduleName%22%3A%22uuid%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fwindow-utils.js%22%3A%7B%22docsSHA256%22%3A%226def0e8ecd7d26430c348350c7ba0fb523b0f634cba8a5a66496e6650fca7236%22%2C%22jsSHA256%22%3A%22cea11357da4d2d3c518ebe9a56c25957138ecc0c4eee8b20cc2cd6a8c80e7627%22%2C%22moduleName%22%3A%22window-utils%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Ferrors%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ferrors.js%22%7D%2C%22.%2Fevents%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fevents.js%22%7D%2C%22.%2Ftraits%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Ftraits.js%22%7D%2C%22.%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fxpcom.js%22%3A%7B%22docsSHA256%22%3A%2221651e6c0c6710f081701a1ad191a20ab67b7a86f758fbc1236ae39d12783524%22%2C%22jsSHA256%22%3A%2211a1e2e128a47388cf98662aa134858a5cb5edad9d90760e4eb81138920edaf2%22%2C%22moduleName%22%3A%22xpcom%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22.%2Fbase%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fbase.js%22%7D%2C%22.%2Fruntime%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fruntime.js%22%7D%2C%22.%2Funload%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Funload.js%22%7D%2C%22.%2Fuuid%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fuuid.js%22%7D%2C%22.%2Fxul-app%22%3A%7B%22path%22%3A%22api-utils%2Flib%2Fxul-app.js%22%7D%2C%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22api-utils%2Flib%2Fxul-app.js%22%3A%7B%22docsSHA256%22%3A%22a8a75f8986992cdfd4bb75e446bad60d01257a4df65dd07c9db050d90be536e6%22%2C%22jsSHA256%22%3A%22712b929399b525bf49eb95af4c2f91a5500d0c0a739ff87a7c5acf333dada3e9%22%2C%22moduleName%22%3A%22xul-app%22%2C%22packageName%22%3A%22api-utils%22%2C%22requirements%22%3A%7B%22chrome%22%3A%7B%22path%22%3A%22chrome%22%7D%7D%2C%22sectionName%22%3A%22lib%22%7D%2C%22kwierso-117%2Flib%2Fmain.js%22%3A%7B%22docsSHA256%22%3Anull%2C%22jsSHA256%22%3A%22a4a919ae28ac1de50a2068cb8602b1e47aee5541016b494801d56a8af95e72f8%22%2C%22moduleName%22%3A%22main%22%2C%22packageName%22%3A%22kwierso-117%22%2C%22requirements%22%3A%7B%7D%2C%22sectionName%22%3A%22lib%22%7D%7D%2C%22metadata%22%3A%7B%22addon-kit%22%3A%7B%22author%22%3A%22Atul%20Varma%20(http%3A%2F%2Ftoolness.com%2F)%20%3Catul%40mozilla.com%3E%22%2C%22contributors%22%3A%5B%22Myk%20Melez%20(http%3A%2F%2Fmelez.com%2F)%20%3Cmyk%40mozilla.org%3E%22%2C%22Daniel%20Aquino%20%3Cmr.danielaquino%40gmail.com%3E%22%5D%2C%22description%22%3A%22Add-on%20development%20made%20easy.%22%2C%22keywords%22%3A%5B%22javascript%22%2C%22engine%22%2C%22platform%22%2C%22xulrunner%22%2C%22jetpack-high-level%22%5D%2C%22license%22%3A%22MPL%202.0%22%2C%22name%22%3A%22addon-kit%22%7D%2C%22api-utils%22%3A%7B%22author%22%3A%22Atul%20Varma%20(http%3A%2F%2Ftoolness.com%2F)%20%3Catul%40mozilla.com%3E%22%2C%22contributors%22%3A%5B%22Myk%20Melez%20(http%3A%2F%2Fmelez.com%2F)%20%3Cmyk%40mozilla.org%3E%22%2C%22Daniel%20Aquino%20%3Cmr.danielaquino%40gmail.com%3E%22%5D%2C%22description%22%3A%22Foundational%20infrastructure%20and%20utilities.%22%2C%22keywords%22%3A%5B%22javascript%22%2C%22engine%22%2C%22platform%22%2C%22xulrunner%22%2C%22jetpack-low-level%22%5D%2C%22license%22%3A%22MPL%202.0%22%2C%22name%22%3A%22api-utils%22%7D%2C%22kwierso-117%22%3A%7B%22author%22%3A%22kwierso%22%2C%22main%22%3A%22main%22%2C%22name%22%3A%22kwierso-117%22%2C%22version%22%3A%22initial.rev1%22%7D%7D%2C%22name%22%3A%22kwierso-117%22%2C%22sdkVersion%22%3A%221.6.1%22%2C%22staticArgs%22%3A%7B%7D%2C%22verbose%22%3Afalse%2C%22loadReason%22%3A%22install%22%2C%22rootURI%22%3A%22jar%3Afile%3A%2F%2F%2FC%3A%2FUsers%2FKWierso%2FAppData%2FRoaming%2FMozilla%2FFirefox%2FProfiles%2Fcw0rrxh0.default%2Fextensions%2Fjid0-2vPfEeGebdnDrU2ba9tKyOgSf68%40jetpack.xpi!%2F%22%2C%22uriPrefix%22%3A%22resource%3A%2F%2Fjid0-2vpfeegebdndru2ba9tkyogsf68-at-jetpack%2F%22%7D)%3B%0Aloader.main(%22main%22%2C%20%22kwierso-117%2Flib%2Fmain.js%22)%3B", line 2, in 
    loader.main("main", "kwierso-117/lib/main.js");
  File "resource://jid0-2vpfeegebdndru2ba9tkyogsf68-at-jetpack/api-utils/lib/cuddlefish.js", line 237, in mai
    this.load(module); // this is where the addon's main.js finally runs
  File "resource://jid0-2vpfeegebdndru2ba9tkyogsf68-at-jetpack/api-utils/lib/cuddlefish.js", line 172, in loa
    sandbox.load(module.uri);
  File "resource://jid0-2vpfeegebdndru2ba9tkyogsf68-at-jetpack/api-utils/lib/cuddlefish.js", line 48, in loa
    scriptLoader.loadSubScript(uri, this.sandbox, 'UTF-8');
SyntaxError: missing ) after argument list

Timestamp: 5/4/2012 1:00:38 PM
Error: SyntaxError: missing ) after argument list
Source File: resource://jid0-2vpfeegebdndru2ba9tkyogsf68-at-jetpack/api-utils/lib/cuddlefish.js -> resource://jid0-2vpfeegebdndru2ba9tkyogsf68-at-jetpack/kwierso-117/lib/main.js
Line: 1

Comment 6

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

I also get the line numbers running from a recent Aurora build, but there are no numbers from Beta.

Comment 7

[Dave Townsend [:mossop]]

Perhaps the fix in bug 751149 didn't cover everything?

Comment 8

[Jeff Balogh (:jbalogh)]

(In reply to Wes Kocher (:KWierso) (Jetpack Bugmaster) from comment #6)
> I also get the line numbers running from a recent Aurora build, but there
> are no numbers from Beta.

I see line numbers in Beta, but not Aurora or Nightly.

Comment 9

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

So, when I run with github master branch, I see this logged using Jeff's sample addon on Beta, Aurora and Nightly:
http://pastebin.mozilla.org/1617011

When I run from Addon Builder using 1.6.1, I see this in the error console for Beta, Aurora and Nightly:
http://pastebin.mozilla.org/1617033

Comment 10

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

WONTFIXing because the version of Firefox that the versions of the SDK are targeting work correctly. (When 1.8 ships, we'll be on Firefox 13, which shows line numbers.)

Comment 11

[Jeff Griffiths (:canuckistani) (:⚡︎)]

Re-opening for this thread: 

https://groups.google.com/forum/#!topic/mozilla-labs-jetpack/2l6XIjk2Nuo

Comment 12

[Jeff Griffiths (:canuckistani) (:⚡︎)]

Here's a repro:

https://builder.addons.mozilla.org/package/158883/latest/

This is specifically when using contentScriptFile, contentScript: 'some code' behaves much more nicely. 

The error I get from this ( Fx Aurora 18, SDK 1.10 ), is:

[JavaScript Error: "An exception occurred.
Traceback (most recent call last):
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/l10n/loader.js", line 18, in request.onload
    resolve(request.responseText);
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 40, in then
    return { then: function then(resolve, reject) { reject(reason) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 113, in rejected
    function rejected(reason) { deferred.resolve(reject(reason)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 50, in effort
    try { return f(options) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/addon/runner.js", line 86, in onLocalizationReady
    run(options);
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/addon/runner.js", line 109, in run
    let program = main(options.loader, options.main);
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/loader.js", line 281, in main
    return load(loader, module).exports;
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/loader.js", line 193, in load
    evaluate(sandbox, module.uri);
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/loader.js", line 167, in evaluate
    : loadSubScript(uri, sandbox, encoding);
SyntaxError: syntax error
"]

Comment 13

[Myk Melez [:myk] [@mykmelez]]

(In reply to Jeff Griffiths (:canuckistani) from comment #12)
> Here's a repro:
> 
> https://builder.addons.mozilla.org/package/158883/latest/
> 
> This is specifically when using contentScriptFile, contentScript: 'some
> code' behaves much more nicely. 

That's presumably because `foo: 'bar'` is a syntactically valid `key: value` pair, while `foo:` is not (the value is missing), and your test case doesn't assign a value to contentScriptFile.  I bet it'd fail the same way if it didn't assign a value to contentScript (or any other key).

Comment 14

[Jeff Griffiths (:canuckistani) (:⚡︎)]

Right, I think I was multi-tasking (badly) and didn't complete the example. I've updated it so that it loads the content script and the content script indeed has a syntax error. I get this error in the JS console:

Timestamp: 12-10-27 6:34:42 PM
Error: An exception occurred.
Traceback (most recent call last):
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/canuckistani-32/data/cs.js", line 5, in null
    baz: 'else'
SyntaxError: missing } after property list

Did I speak too soon?

Also - in your case it looks like the syntax error is in a module, not a content script? If I try to load a module with the same error in the code ( see the example ) I get a very similar error:

Timestamp: 12-10-27 6:39:46 PM
Error: An exception occurred.
Traceback (most recent call last):
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/l10n/loader.js", line 18, in request.onload
    resolve(request.responseText);
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 40, in then
    return { then: function then(resolve, reject) { reject(reason) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 113, in rejected
    function rejected(reason) { deferred.resolve(reject(reason)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 138, in resolve
    while (pending.length) result.then.apply(result, pending.shift())
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 32, in then
    return { then: function then(resolve) { resolve(value) } }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 112, in resolved
    function resolved(value) { deferred.resolve(resolve(value)) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/promise.js", line 50, in effort
    try { return f(options) }
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/addon/runner.js", line 86, in onLocalizationReady
    run(options);
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/addon/runner.js", line 121, in run
    quit: exit
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/canuckistani-32/lib/main.js", line 6, in exports.main
    var foo = require('foo');
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/loader.js", line 267, in require
    freeze(load(loader, module));
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/loader.js", line 193, in load
    evaluate(sandbox, module.uri);
  File "resource://jid0-qbahlbnwkvfhx0s4nfoowwfrnj0-at-jetpack/api-utils/lib/loader.js", line 167, in evaluate
    : loadSubScript(uri, sandbox, encoding);
SyntaxError: missing } after property list

Comment 15

[Jeff Griffiths (:canuckistani) (:⚡︎)]

Cc'ing Irakli.

Irakli: the tracebacks produced by the loader when loading a module with a syntax error are huge, ugly and don't include enough info. Should I open a new bug for this? Feels like a separate issue.

Comment 16

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

I think different bug is a good idea! Yes we should create fix this and have test case that uses content-script with syntax error. I'm not sure we can get a proper
stack trace but for the very least we should not leak stack from the SDK code.

Comment 17

[Jeff Griffiths (:canuckistani) (:⚡︎)]

Ok then, closing. The new bug is bug 806509.

Comment 18

[Myk Melez [:myk] [@mykmelez]]

(In reply to Jeff Griffiths (:canuckistani) from comment #14)
> Also - in your case it looks like the syntax error is in a module, not a
> content script? If I try to load a module with the same error in the code (
> see the example ) I get a very similar error:

Right, my use case is an exception thrown by a module.  In the thread I started about exceptions in modules, Hernán raised the additional issue of exceptions in content scripts, but that is presumably a different bug.

Comment 19

[Jeff Griffiths (:canuckistani) (:⚡︎)]

Yeah, if you look at the first error in comment 14, that's what you see from a content script - it includes the line number.