Bug 14329 – REGRESSION: TEXTAREA - cannot drag-and-drop text at en.wikipedia.org/

• Status: RESOLVED
• Resolution: FIXED
• Component: HTML Editing

[Charles Gaudette]

My initial try to reduce this failed, so I'm submitting in this slightly unorthodox way because I may not have the time to reduce (successfully).

STEPS:
(1) http://en.wikipedia.org/wiki/User:Charles_Gaudette/Tiffany
(2) open an "edit this page" view
(3) scroll the textarea to the comment, "TRY DRAGGING "1919" ANYWHERE"
(4) select "1919", try drag and drop inside the textarea

ALSO NOTE:
- above the comment point in Step #3 the drag-and-drop feature works
- the point at which drag-and-drop text stops working has not yet been tracked down
- the point at which drag-and-drop text stops working may not be static!

[Alexey Proskuryakov]

I'm getting an assertion failure:

ASSERTION FAILED: Uncaught exception - Can't cache image
0
(/Users/ap/WebKit/WebCore/platform/mac/BlockExceptions.mm:36 ReportBlockedObjCException)

If you scroll only by a few pixels you see that the drag image is present by clipped by the number of pixels you scrolled.

Created attachment 15196 [details]
Patch, including pixel test and change log

It is somewhat unfortunate that the column adjustment needs to be done in selectionRect() for the non-clipped case, but the way absolutePosition() works I couldn't think of a way to avoid that.

[Oliver Hunt]

Comment on attachment 15196 [details]
Patch, including pixel test and change log

Patch looks good, only thing i'll say is that you should make the layout test be text only if possible.  I can't immediately see why a pixel dump would be necessary in this case.

(In reply to comment #4)
> (From update of attachment 15196 [details] [edit])
> Patch looks good, only thing i'll say is that you should make the layout test
> be text only if possible.  I can't immediately see why a pixel dump would be
> necessary in this case.
> 

I don't know of any way that selection rects come into play other than in rendering. What am I missing?

[Oliver Hunt]

Indeed you're right -- no way to verify it without the damned pixel test :-/

[Mark Rowe (bdash)]

Landed in r23809.

*** Bug 14330 has been marked as a duplicate of this bug. ***

[Stephanie Lewis]

From the duplicate:

<rdar://problem/5290118>