609682 - slow script warning with javascript.options.methodjit.content=false

Status: RESOLVED DUPLICATE of bug 607856

(Core :: JavaScript Engine, defect)

Description

[Matthias Versen [:Matti]]

Build identifier: Mozilla/5.0 (Windows NT 6.1; rv:2.0b8pre) Gecko/20101104 Firefox/4.0b8pre SeaMonkey/2.1b2pre

1) set javascript.options.methodjit.content to false
2) load http://www.airliners.net/
3) you get a slow script warning for the Script: http://cdn-www.airliners.net/js/jquery.js:24
4) it works if methodjit is enabled.

This could be a regression from the last few days. I visit this page frequently but I'm not sure at which point I disabled methodjit during my triage work in bugzilla but it seems that I forgot to switch it on again. I will try to find the regression range if you want.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Regression range would be great.

Comment 2

[Matthias Versen [:Matti]]

bad nightly:  2010-10-27
Built from http://hg.mozilla.org/mozilla-central/rev/2f456d0310fa
good nightly: 2010-10-26
Built from http://hg.mozilla.org/mozilla-central/rev/d253c44465ae

-> http://hg.mozilla.org/mozilla-central/pushloghtml?startdate=2010-10-26&enddate=2010-10-27

there is a TM Merge in this timeframe

Comment 3

[Boris Zbarsky [:bzbarsky]]

Can you try narrowing down using TM nightlies?

Comment 4

[Matthias Versen [:Matti]]

I knew that you would ask that and my DSL speed sucks for such work

good 
2010-10-20-04-tracemonkey/
Built from http://hg.mozilla.org/tracemonkey/rev/449fea695930

bad 
2010-10-21-05-tracemonkey/
Built from http://hg.mozilla.org/tracemonkey/rev/bf01a9a4d35c

Comment 5

[Boris Zbarsky [:bzbarsky]]

Pushlog: http://hg.mozilla.org/tracemonkey/pushloghtml?fromchange=449fea695930&tochange=bf01a9a4d35c

Nicholas, the obvious tracer change in that range is bug 600127.  Could that have caused these symptoms?

Comment 6

[Nicholas Nethercote [inactive]]

(In reply to comment #5)
> 
> Nicholas, the obvious tracer change in that range is bug 600127.  Could that
> have caused these symptoms?

This one's come up a couple of times now.  (Actually, I could have sworn I replied to this bug earlier.)

Bug 600127 broke some stuff, bug 607856 diagnosed it and added a workaround, bug 609129 fixed it properly.

We'll keep getting reports until the patch from bug 609129 is merged to m-c.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Ah.  Matthias, can I bother you to test a tip TM build to see whether the problem is gone?

Comment 8

[Matthias Versen [:Matti]]

This is foxed in the latest TM nightly.
Thanks for your time, marking wfm