[Webkit-unassigned] [Bug 81141] REGRESSION(r103452): 100% CPU usage and 5s pause after clicking on a link in Yahoo Mail

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=81141


Adam Klein <adamk at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |darin at apple.com,
                   |                            |mjs at apple.com,
                   |                            |ojan at chromium.org,
                   |                            |sam at webkit.org




--- Comment #5 from Adam Klein <adamk at chromium.org>  2012-03-14 15:14:12 PST ---
Thanks for the spin dump.

Looks like there's a DOMSubtreeModified listener modifying the ID each time it's called. Before r103452, only adding or removing an attribute would cause this event to fire, not modifying an existing one. Now it's possible to spin with a simple listener like:

function() { someDiv.id = 'foo'; }

This new behavior was an inadvertent side effect of a refactor that made the code more consistent. Based on the (rather unfortunately broad) spec, http://www.w3.org/TR/DOM-Level-3-Events/#event-type-DOMSubtreeModified, it seems like the behavior after r103452 is closer to the spec, and this sort of bug is easy to fix for a web app. But we could still consider suppressing the event in this case, given that mutation events are deprecated and to avoid making them any more powerful (useful to webapps) than they already were.

Adding some folks who may have input.

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