Closed Bug 1121773 Opened 9 years ago Closed 9 years ago

JavaScript Error: "uncaught exception: 2147746132" in Firefox for Android on recent fx-team

Categories

(Firefox for Android Graveyard :: General, defect)

All
Android
defect
Not set
normal

Tracking

(firefox38.0.5 fixed, firefox39 fixed, firefox40 fixed, firefox41 fixed)

RESOLVED FIXED
Firefox 41
Tracking Status
firefox38.0.5 --- fixed
firefox39 --- fixed
firefox40 --- fixed
firefox41 --- fixed

People

(Reporter: nalexander, Assigned: nalexander)

References

Details

Attachments

(1 file)

GeckoConsole(18913)         [JavaScript Error: "uncaught exception: 2147746132"]

at start-up and page load.  Per http://james-ross.co.uk/mozilla/misc/nserror?0x80040154, this is NS_ERROR_FACTORY_NOT_REGISTERED.

Interferes with the remote debugger, I think because browser.js doesn't complete loading.  This is happening with

@    changeset:   253669:565179dcd634
|\   tag:         fx-team
| |  parent:      253605:f3285709bf99
| |  parent:      253668:c1f6345f2803
| |  user:        Ryan VanderMeulen <ryanvm@gmail.com>
| |  date:        Wed Jan 14 16:33:44 2015 -0500
| |  summary:     Merge m-c to fx-team. a=merge

and with an earlier fx-team.
OS: Mac OS X → Android
Sorry for the noise, folks.  This didn't repro on the TBPL build and was fixed by a clobber build.  Sigh.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
Aaron just saw this in release 37.0.1.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
I see this in local builds all over the place, but my local builds tend to be a little non-standard so I can't be sure this is not just my issue.
Running with

adb shell am start -a android.activity.MAIN -n org.mozilla.fennec_$USER/.App --es env0 NSPR_LOG_MODULES=nsComponentManager:5

I see that this is:

E GeckoConsole(21263)         [JavaScript Error: "uncaught exception: 2147746132"]
I PRLog(21263)                1930373960[75b43100]: nsComponentManager: CreateInstanceByContractID(@mozilla.org/dom/site-specific-user-agent;1) FAILED

It appears that this is a false alarm: I surmise the invocation must be at [1].

https://dxr.mozilla.org/mozilla-central/source/dom/base/Navigator.cpp#2728

I would like to understand how to catch and ignore the error rather than failing.
This is a real issue.  It's fall-out from a bad patch (that I reviewed :/) from Bug 1154960.  The patch conditionally included the SiteSpecificUserAgent files but didn't remove the unconditional include.
Depends on: 1154960
https://hg.mozilla.org/mozilla-central/rev/c250ce4656d0
Assignee: nobody → nalexander
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Blocks: 1167208
Comment on attachment 8608931 [details] [diff] [review]
Patch that landed on fx-team

Approval Request Comment
[Feature/regressing bug #]: oversight from Bug 1154960.
[User impact if declined]: none; manual testing should uncover such issues.
[Describe test coverage new/current, TreeHerder]: none.
[Risks and why]: extremely low -- a small build system change.
[String/UUID change made/needed]: none.
Attachment #8608931 - Flags: approval-mozilla-beta?
Attachment #8608931 - Flags: approval-mozilla-aurora?
Comment on attachment 8608931 [details] [diff] [review]
Patch that landed on fx-team

Approved for uplift to aurora and beta. Small fix for a regression in the remote debugger for android
Attachment #8608931 - Flags: approval-mozilla-beta?
Attachment #8608931 - Flags: approval-mozilla-beta+
Attachment #8608931 - Flags: approval-mozilla-aurora?
Attachment #8608931 - Flags: approval-mozilla-aurora+
Nick, in order to fix bug 1167208 in 38.0.5, we should uplift that to release too, right?
Flags: needinfo?(nalexander)
(In reply to Sylvestre Ledru [:sylvestre] from comment #11)
> Nick, in order to fix bug 1167208 in 38.0.5, we should uplift that to
> release too, right?

There are enough tickets that this is tricky.  This patch should uplift; there is no particular patch for Bug 1167208; and this patch completes the patch from Bug 1154960, which should be uplifted.  That is, the two patches

https://hg.mozilla.org/mozilla-central/rev/bef3ac32b648 (original from Bug 1154960)
https://hg.mozilla.org/mozilla-central/rev/c250ce4656d0 (fix from Bug 1121773)

should make it all the way up to 38.0.5.  (In that order.)
Flags: needinfo?(nalexander)
Comment on attachment 8608931 [details] [diff] [review]
Patch that landed on fx-team

Let's take in 38.0.5 release too.
Ryan, could you land that please? Thanks
Flags: needinfo?(ryanvm)
Attachment #8608931 - Flags: approval-mozilla-release+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.