Clean up DragController

• Component: New Bugs
• Status: RESOLVED FIXED

[Eric Seidel (no email)]

Clean up DragController

Looking at DragController this evening to fix a few bugs I was confused by m_document.  I'm no longer confused by it now that I've renamed it to m_documentUnderMouse.

I also saw copy/paste code involving clearing the selection when moving out of one document and into another.  I moved that code into a function called mouseMovedIntoDocument.

I also got rid of a couple needless null-checks after confirming via source inspection they were not ever possible.

[Eric Seidel (no email)]

Created attachment 30542 [details]
Clean up DragController

 3 files changed, 82 insertions(+), 48 deletions(-)

[Eric Seidel (no email)]

Created attachment 30543 [details]
Clean up DragController

 4 files changed, 86 insertions(+), 51 deletions(-)

[Alexey Proskuryakov]

Comment on attachment 30543 [details]
Clean up DragController

r=me with some trivial comments over IRC.

[Eric Seidel (no email)]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/page/DragController.cpp
	M	WebCore/page/DragController.h
	M	WebCore/page/mac/DragControllerMac.mm
Committed r43968

Hopefully future readers of DragController.cpp will find it easier to understand now. :)