320020 - my.yahoo.com - use of getElementsByClass causes temporary hang and/or "Unresponsive script" warning

Status: RESOLVED WORKSFORME

(Tech Evangelism Graveyard :: English US, defect)

Description

[baconburg]

User-Agent:       Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

WARNING: Unresponsive Script

Reproducible: Always

Steps to Reproduce:
1.Signing onto my yahoo
2.
3.

Actual Results:  
error message

Expected Results:  
no error message

Comment 1

[Ria Klaassen (not reading all bugmail)]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051212 Firefox/1.5

Works for me. Do you also see this problem in Firefox's safe-mode?
http://kb.mozillazine.org/Safe_Mode_(Firefox)

Comment 2

[Jesse Ruderman]

Bug 320409 is another report of this problem on my.yahoo.com.  I think I saw another one a few weeks ago, but I can't find it now.

It would help if you could:

* Make a reduced testcase and attach it, or

* Try temporarily removing parts of your customized My Yahoo page in order to figure out which part makes Firefox unhappy.  Use Camino to do the removal if it takes too long to load the page in Firefox.  (IIRC, the report I read a few weeks ago implicated a mail-related feature, so you might try that first.)  Then I can try adding the same feature to my My Yahoo page and see if I hit the same problem.

Comment 3

[Jesse Ruderman]

Does Firefox appear to hang for a few seconds before the dialog appears?  If not, the dialog is appearing incorrectly, which means it's definitely a bug in Firefox.  If it does, it could either be a bug in Firefox (something is taking longer than it should) or a bug in the site (it's using an unnecessarily slow algorithm implemented in JavaScript).

Comment 4

[Jesse Ruderman]

It turns out that my father experiences this problem on his blazing-fast 500MHz Celeron computer.  When told me about the problem last month, I told him to change dom.max_script_run_time from 5 to 12, but I didn't remember that incident until several days after commenting on this bug report.

The "My Yahoo!" page is a bit slow to load, but the browser becomes unresponsive for about 6 seconds right around when it finishes loading (probably due to an onload script).

I'll try to reduce the bug on my father's computer, and then I'll try to figure out whether it's a bug in Firefox or a bug in My Yahoo.

Comment 5

[Jesse Ruderman]

At least for my dad's My Yahoo page, most of the hang is spent in the page's getElementsByClass function.  getElementsByClass is called twice.

function getElementsByClass(theNode, theClass) {
  var cArray = [];
  var z = new RegExp('\\b' + theClass + '\\b');
  function doTree(theNode) {
    if (theNode.className && z.test(theNode.className)){
      cArray.push(theNode);
    }
    for (var i=0, len=theNode.childNodes.length; i<len; i++) {
      doTree(theNode.childNodes[i]);
    }
  }
  doTree(theNode);
  return cArray;
}

I'm leaning toward saying this is Yahoo's bug.  They should find a way to make the page work without having a script recurse through the document.

Comment 6

[Jesse Ruderman]

My partially-reduced testcase causes IE to hang much longer than it causes Firefox to hang.  Hmm.

Comment 7

[Jesse Ruderman]

Attachment: partially simplified testcase

Ripped out a bunch of scripts, ripped out the list of stocks my dad follows and replaced it with a bunch of repeated links, brought a few scripts inline.

Comment 8

[Jesse Ruderman]

my.yahoo.com is #26 on the http://reporter.mozilla.org/ list.  About 30% of the reports are about this issue.

Comment 9

[Jesse Ruderman]

Most people who reported that the dialog appeared said it was annoying, and none of them mentioned the 5-second hang before the appearance of the dialog.  Maybe they're reading headlines and not trying to scroll.  I wonder if we should increase the default dom.max_script_run_time from 5 to 10.

Comment 10

[Jesse Ruderman]

*** Bug 325227 has been marked as a duplicate of this bug. ***

Comment 11

[Ria Klaassen (not reading all bugmail)]

*** Bug 325600 has been marked as a duplicate of this bug. ***

Comment 12

[OstGote!]

check if bug 335058 helps

Comment 13

[Black_Ps`]

Is this like  Bug 506442 ?

Comment 14

[Wayne Mery (:wsmwk)]

(In reply to comment #13)
> Is this like  Bug 506442 ?

baconburg, Black_Ps`
do you still see this issue?

Comment 15

[Black_Ps`]

No, the problem has been solved!

Comment 16

[Wayne Mery (:wsmwk)]

WFM based on comment 15