557720 - Textarea filled by using this JS hangs the browser

Status: RESOLVED WORKSFORME

(Core :: Layout: Form Controls, defect)

Description

[Alice0775 White]

Attachment: testcase

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a4pre)
Gecko/20100406 Minefield/3.7a4pre ID:20100406040007
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a4pre)
Gecko/20100406 Minefield/3.7a4pre ID:20100406040007

This javascript script puts some text in a textarea.
Firefox3.6.3 execute this script as expected and it takes 1-5 sec in C2Q@2.5GHz CPU. 
Minefield3.7a4pre hangs with 100% CPU-core.

It is related but different to Bug504784 .

Reproducible: Always

Steps to Reproduce:
1. Start Minefield with New profile
2. Open testcase

Actual Results:  
Minefield3.7a4pre hangs with 100% CPU core.

Expected Results: 
Minefield3.7a4pre should not hang,
and execute this script as expected and it takes 1-5 sec as same as Firefox3.6.3. 

http://hg.mozilla.org/mozilla-central/rev/736619af9701
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20091101 Minefield/3.7a1pre ID:20091101045105

Regression rwindow:

Works:
http://hg.mozilla.org/mozilla-central/rev/3405d57427a5
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20091115 Minefield/3.7a1pre ID:20091115042134

Fails:
http://hg.mozilla.org/mozilla-central/rev/74e3158a9fc5
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20091116 Minefield/3.7a1pre ID:20091116050530

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=3405d57427a5&tochange=74e3158a9fc5

This issue does not happens on Branch(Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.4pre) Gecko/20100406 Namoroka/3.6.4pre ID:20100406042106).

Comment 1

[Alice0775 White]

Sorry , please ignore the following sentence in comment #0.
>http://hg.mozilla.org/mozilla-central/rev/736619af9701
>Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20091101
>Minefield/3.7a1pre ID:20091101045105

Comment 2

[Boris Zbarsky [:bzbarsky]]

If you take out the display = "none" thing, isn't this bug also present in the older builds?

All that happened here is that the display change got processed before the value sets due to the flush that bug 527306 took out, and now it's not processed until after all the value-setting is done....

I would also assume that the lazy editor changes that Ehsan is working on will help here.

Comment 3

[Alice0775 White]

(In reply to comment #2)
> If you take out the display = "none" thing, isn't this bug also present in the
> older builds?
testcase( https://bugzilla.mozilla.org/attachment.cgi?id=389100 )of Bug504784 which is not including the display = "none" thing,When loading it,The latest Namoroka3.6.4pre hangs.and the old trunk build also hangs.but latest Minfield3.7a4pre does not hang.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Right, but that only goes up to 1500, not 7000.  That difference should be a factor of 20 or more in the time required....

Comment 5

[(no longer active)]

(In reply to comment #3)
> (In reply to comment #2)
> > If you take out the display = "none" thing, isn't this bug also present in the
> > older builds?
> testcase( https://bugzilla.mozilla.org/attachment.cgi?id=389100 )of Bug504784
> which is not including the display = "none" thing,When loading it,The latest
> Namoroka3.6.4pre hangs.and the old trunk build also hangs.but latest
> Minfield3.7a4pre does not hang.

I think the difference which you see here is because of bug 518122.

Also, this should depend on the textarea lazy initialization work.

Comment 6

[Alice0775 White]

https://hg.mozilla.org/mozilla-central/rev/70dfd4254cf977be2000b41f7a6b97eaada8e2a0
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0 ID:20160507030302