Emil Ghitta, QA [:emilghitta] Reporter
	Description • 6 years ago

This bug was filed from the Socorro interface and is
report bp-dcf0a23a-8c1e-49e9-8157-eb6b10180928.
=============================================================

Top 10 frames of crashing thread:

0 mozglue.dll mozalloc_abort memory/mozalloc/mozalloc_abort.cpp:35
1 xul.dll static void Abort xpcom/base/nsDebugImpl.cpp:471
2 xul.dll NS_DebugBreak xpcom/base/nsDebugImpl.cpp:453
3 xul.dll nsDebugImpl::Abort xpcom/base/nsDebugImpl.cpp:146
4 xul.dll XPTC__InvokebyIndex xpcom/reflect/xptcall/md/win32/xptcinvoke_asm_x86_64.asm:97
5  @0x1c93b4fdccf 
6 xul.dll XPCWrappedNative::CallMethod js/xpconnect/src/XPCWrappedNative.cpp:1233
7 xul.dll static bool XPC_WN_CallMethod js/xpconnect/src/XPCWrappedNativeJSOps.cpp:963
8 xul.dll js::InternalCallOrConstruct js/src/vm/Interpreter.cpp:553
9 xul.dll static bool InternalCall js/src/vm/Interpreter.cpp:607

=============================================================

[Steps to reproduce]:
1. Launch Firefox with ./firefox -no-remote -p -migration.
2. Import from Edge or Chrome.
3. Close Firefox.

[Expected results]:
Firefox successfully closes.

[Actual results]:
Several Firefox processes remain open and eventually, Firefox Crashes.

[Additional Information]:
For further information regarding this issue please observe the attached screencast.

	Emil Ghitta, QA [:emilghitta] Reporter
	Comment 1 • 6 years ago

Not sure which Bug caused this regression. Had to find the regression manually.

Last good 2018-06-14
First bad 2018-06-15

Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=684094d78fb4&tochange=e51f8dbf0397

	Pascal Chevrel:pascalc
	Comment 2 • 6 years ago

Matthew, could you have a look at this 63 regression? Is that something we can fix for 63?

	Matthew N. [:MattN]
	Comment 3 • 6 years ago

From crash-stats and the regression range I can see that this was also in 62.

Can you reproduce this using the normal user flow (not using the -migration command line argument) by moving your profiles.ini file aside?

	Matthew N. [:MattN]
	Comment 4 • 6 years ago

After discussing on Slack, I realized that we disabled the "no profiles.ini" case a year ago in bug 1403402.

	Emil Ghitta, QA [:emilghitta] Reporter
	Comment 5 • 6 years ago

Just a note. This seems to be reproducible only by using the -migration command line argument. I didn't manage to reproduce this after importing via the "Import Now" button from homepage.

	Mike Taylor [:miketaylr]
	Comment 6 • 6 years ago

This feels like an edge case, if it requires the CL argument.

	Matthew N. [:MattN]
	Comment 7 • 6 years ago

I agree that since this requires the command line argument it's not a high priority.