[Webkit-unassigned] [Bug 52211] FrameLoader::isProcessingUserGesture is wrong in dispatchWillPerformClientRedirect

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 12 18:55:44 PST 2011


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





--- Comment #6 from Johnny(Jianning) Ding <jnd at chromium.org>  2011-01-12 18:55:44 PST ---
(In reply to comment #4)

> Actually I meant to take that out.  In fact, didCancelClientRedirect gets called from several completely different code paths, and in at least one other - namely when the redirect actually succeeds and the new page finishes loading - the gesture state isn't set, and it's unclear how it could be set.
> 
> So if we set it here, the gesture state in didCancelClientRedirect will be inconsistent - sometimes it will be the same as when the redirect started, and sometimes not.  This isn't useful to anybody.
> 
> The correct thing for the client to do if they want to know the gesture state in didCancelClientRedirect is to save it in willPerformClientRedirect and check (and clear) the saved state in didCancelClientRedirect.

That was my concern. Thanks to clearly comment and fix it.

-- 
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