11/29/06 8:30 AM Chris Petersen:
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.
iFrame scrolling should stop when mousing up outside of iframe but doesn't.
Yes, this doesn't happen in stock Safari under Tiger.
I believe this is the likely candidate that caused this regression:
This is filed as <rdar://problem/4854259>
Created attachment 11670 [details]
iFrame test case
Created attachment 11714 [details]
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.
Comment on attachment 11714 [details]
Please do file that follow-up bug.
Committed revision 18079. Filed follow-up bug 11784.