Florent Fayolle Reporter
	Description • 10 years ago

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:35.0) Gecko/20100101 Firefox/35.0
Build ID: 20141201162954

Steps to reproduce:

1. Install this version of the Pentadactyl addon (latest nightly of pentadactyl with the maxVersion of the install.rdf file modified)
2. Put the .pentadatylrc this file in your home ($HOME on Linux)
3. Restart Firefox
4. Open two tabs
5. Use Shift-H


Actual results:

Firefox crashes


Expected results:

The user should switch to another tab.

Florent

	Florent Fayolle Reporter
	Comment 2 • 10 years ago

The crash report is given here:
https://crash-stats.mozilla.com/report/index/8e92421a-36f5-4be1-a3a3-e2c622141207

Kris: I took the liberty to cc you since I know you are one of the maintainer of the addon.

Florent

	Kevin Brosnan [Ex-Mozilla]
	Comment 3 • 10 years ago

Is this something that the JS engine should own?

	Brian Hackett [Laid off!]
	Comment 4 • 10 years ago

It's crashing in JS, yeah.  Can you reproduce this?  Is there a blame cset?

	Florent Fayolle Reporter
	Comment 5 • 10 years ago

Brian, who are you asking that question? (the needinfo flag is unset)
I can bisect the regression using mozregression to get a date of a nightly, but I can't bisect more if I have to compile. Are you interested?

Florent

	Andrew McCreight [:mccr8]
	Comment 6 • 10 years ago

Getting it down to a particular Nightly would be a good first step.

	Florent Fayolle Reporter
	Comment 7 • 10 years ago

> Getting it down to a particular Nightly would be a good first step.

OK, will do that.

Florent

	Florent Fayolle Reporter
	Comment 8 • 10 years ago

Oops, sorry, wrong modifications...

Florent

	Florent Fayolle Reporter
	Comment 9 • 10 years ago

The regression occurred in the nightly of 2014-09-16. I will continue bisecting tomorrow, but here is a regression range:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=d295c8ec&tochange=bb579e3d

Oddly, in that link, there are commits of Wed Sep 17. Is that normal?

Florent

	Florent Fayolle Reporter
	Comment 10 • 10 years ago

Here is a tinier range of commits:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=ed5f95178615&tochange=fada58fb0996

Florent

	Kevin Brosnan [Ex-Mozilla]
	Comment 11 • 10 years ago

[Tracking Requested - why for this release]: regression

	Kevin Brosnan [Ex-Mozilla]
	Comment 12 • 10 years ago

Needinfo is to look at this bug and see if this is something for the JS team to fix or is the extension doing something bad?

	Shu-yu Guo [:shu]
	Comment 13 • 10 years ago

I can't reproduce this with STR on Nightly, and I see this message during loading.

pentadactyl: Loading base: TypeError: iter[Symbol.iterator] is not a function

pentadactyl: bootstrap: TypeError: iter[Symbol.iterator] is not a function
map@resource://dactyl/bootstrap.jsm -> resource://dactyl/base.jsm:1622:1
toObject@resource://dactyl/bootstrap.jsm -> resource://dactyl/base.jsm:1563:17
iterProto[k]@resource://dactyl/bootstrap.jsm -> resource://dactyl/base.jsm:1869:19
deprecated@resource://dactyl/bootstrap.jsm -> resource://dactyl/base.jsm:331:31
@resource://dactyl/bootstrap.jsm -> resource://dactyl/base.jsm:1900:1
loadSubScript@resource://dactyl/bootstrap.jsm:20:26
_load@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/shu/moz/central/obj-dbg/tmp/scratch_user/extensions/pentadactyl@dactyl.googlecode.com.xpi!/bootstrap.js:137:17
load@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/shu/moz/central/obj-dbg/tmp/scratch_user/extensions/pentadactyl@dactyl.googlecode.com.xpi!/bootstrap.js:160:17
init@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/shu/moz/central/obj-dbg/tmp/scratch_user/extensions/pentadactyl@dactyl.googlecode.com.xpi!/bootstrap.js:261:25
startup@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/shu/moz/central/obj-dbg/tmp/scratch_user/extensions/pentadactyl@dactyl.googlecode.com.xpi!/bootstrap.js:366:13
XPI_callBootstrapMethod@resource://gre/modules/addons/XPIProvider.jsm:4436:9
XPI_startup@resource://gre/modules/addons/XPIProvider.jsm:2160:13
callProvider@resource://gre/modules/AddonManager.jsm:208:12
_startProvider@resource://gre/modules/AddonManager.jsm:667:5
AMI_startup@resource://gre/modules/AddonManager.jsm:821:9
AMP_startup@resource://gre/modules/AddonManager.jsm:2399:5
AMC_observe@file:///home/shu/moz/central/obj-dbg/dist/bin/components/addonManager.js:55:7

So I guess it's not loading the xpi. Let me build beta I guess...

	Shu-yu Guo [:shu]
	Comment 14 • 10 years ago

The crash is caused by not backporting bug 1077949. I rolled up a patch and requested backport in that bug.

Note that while that fixes the crash, the addon needs to be fixed as well. It has a temporal dead zone violation (accessing a 'let' binding before initializing it).

	Sylvestre Ledru [:Sylvestre]
	Comment 15 • 10 years ago

Regression plus crashing, tracking.

	Lukas Blakk [:lsblakk] use ?needinfo
	Comment 16 • 10 years ago

Fixed in bug 1077949, resolving this bug.  The issues with the addon should be tracked by that addon developer.