[Webkit-unassigned] [Bug 42534] Crash in Notification::disconnectFrame() triggered by Frame::lifeSupportTimerFired()
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Jul 21 08:49:10 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=42534
--- Comment #8 from Yael <yael.aharon at nokia.com> 2010-07-21 08:49:10 PST ---
(In reply to comment #7)
> + if (!presenter())
> + return;
> m_notificationPresenter->cancelRequestsForPermission(m_scriptExecutionContext);
>
> Is this null check still needed with other changes?
>
I think it is safe to add this check, especially since I don't have exact steps to reproduce the error.
> It would probably be better to use m_notificationPresenter consistently - it's not immediately obvious if presenter() is the same thing.
I agree, I'll update the patch.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list