[Webkit-unassigned] [Bug 17083] REGRESSION (r24267): nested click() calls on the same element do not work (Acid3 bug)

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=17083


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |koivisto at iki.fi,
                   |                            |mjs at apple.com,
                   |                            |darin at apple.com,
                   |                            |adele at apple.com
         AssignedTo|darin at apple.com             |webkit-
                   |                            |unassigned at lists.webkit.org
             Status|ASSIGNED                    |NEW
            Summary|nested click() calls on the |REGRESSION (r24267): nested
                   |same element do not work    |click() calls on the same
                   |(Acid3 bug)                 |element do not work (Acid3
                   |                            |bug)




------- Comment #5 from darin at apple.com  2008-02-10 13:17 PDT -------
This prohibition on click() recursion is new to Safari 3, added to prevent a
button at <http://forums.whirlpool.net.au/> from causing the browser to hang:

    http://trac.webkit.org/projects/webkit/changeset/24267

We decided to change our behavior to match Firefox at that time.

I have a patch that removes the prohibition and reintroduces the bug. Now we'll
have to think of another way to prevent that site from hanging. The hang is due
to the massive amount of work that ends up getting done during the infinite
recursion. I suppose that eventually we'd hit the JavaScript recursive call
limit, but in practice it's too slow and we never get there.

I'll attach my patch with test case, but we can't land it because it
re-introduces the bug fixed in r24267.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.