Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(560)

Issue 8361008: Workaround 75604 for m15 (Closed)

Created:
9 years, 2 months ago by gavinp
Modified:
9 years ago
Reviewers:
Nate Chapin
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/trunk
Visibility:
Public.

Description

Workaround 75604 for m15 The double notification is what causes the crash here: eliminating it should prevent the crash; and in all examples of this I've seen to date, it should be good enough. By the time this crash happens, the ScriptRunner has already set a timer that will remove this element as a listener to the cachedscript. However, I don't have a full fix yet (I'm working on a really reliably repo, even), but I do have a reproduction which works locally on a linux machine, and this fix both eliminates the crash in that case, and seems to still keep script heavy sites working. I propose this as a workaround, for now, and to re-evaluate it once we have a better reproduction, and a proper fix that just stops the double notification. BUG=75604 TEST=tested manually

Patch Set 1 : speeling fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M Source/WebCore/dom/ScriptElement.cpp View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Nate Chapin
9 years, 2 months ago (2011-10-20 19:57:24 UTC) #1
LGTM.

Powered by Google App Engine
This is Rietveld 408576698