Bug 11718 – REGRESSION: When I mouse up after dragging a selection outside of a iframe, the iframe continues to scroll automatically.

• Status: RESOLVED
• Resolution: FIXED
• Component: Frames

[Chris Petersen]

11/29/06 8:30 AM Chris Petersen:
* SUMMARY
After I mouse up after dragging a selection outside of the iframe, the iframe continues to scroll. This is the wrong behavior since scrolling should stop in iframe after I mouse up (outside of the iframe).

This regression first started with Webkit Nightly r17582 since this works with the previous nightly r17566. 

* STEPS TO REPRODUCE
1. With webkit nightly r17582 or newer, open test case "iframe_test.html"
2. Place in focus in the iframe and mouse down. Start to drag a selection over text in the iframe and continue to mouse down. Move the cursor so that it just outside the bottom of iframe. This should cause the iframe to scroll.
3. Now, mouse up while the iframe is scrolling. Notice the iframe DOESN'T stop scrolling when you mouse up.

* RESULTS
iFrame scrolling should stop when mousing up outside of iframe but doesn't.

* REGRESSION
Yes, this doesn't happen in stock Safari under Tiger.

[Chris Petersen]

I believe this is the likely candidate that caused this regression:

http://trac.webkit.org/projects/webkit/changeset/17574

[Chris Petersen]

This is filed as <rdar://problem/4854259>

[Chris Petersen]

Created attachment 11670 [details]
iFrame test case

[Alexey Proskuryakov]

Created attachment 11714 [details]
proposed fix

This fix makes an EventHandler hold a reference to a subframe's node in m_mousePressNode - previously, this data member wasn't reset for subframe events, continuing to reference nodes from earlier actions.

There seems to be a lot of duplication in EventHandler data members at the moment (m_mousePressNode vs. m_clickNode; m_bMousePressed vs. m_mousePressed). I chose not to attempt merging any of these in this patch, and plan to file a follow-up bug.

[Geoffrey Garen]

Comment on attachment 11714 [details]
proposed fix

r=me

Please do file that follow-up bug.

[Alexey Proskuryakov]

Committed revision 18079. Filed follow-up bug 11784.