[Webkit-unassigned] [Bug 33188] ER: Need a way to know when FrameLoader is loading a URL based on user action

[bugzilla-daemon at webkit.org]

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


Adam Barth <abarth at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #45848|review?                     |review-
               Flag|                            |




--- Comment #6 from Adam Barth <abarth at webkit.org>  2010-01-04 20:30:24 PST ---
(From update of attachment 45848)
We aren't accepting FrameLoader patches without tests.

That said, this code is wrong, mostly because all our user gesture code is
wrong.  Why is frame->script()->processingUserGesture() insufficient?  I think
the correct fix is to teach that function to return the correct answer.

Basically, the right way to do this is when an event enters WebKit (or maybe
WebCore), we know whether it's a real user input event or a fake event.  At
that time, we should create a scoped object that stores the user gesture state
in a static.  script()->processingUserGesture should then read the static.  The
current approach of trying to recover the information from Event is wrong and
doesn't work in edge cases.

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