[Webkit-unassigned] [Bug 19465] Cursor sometimes gets 'stuck' in textareas when trying to navigate with arrow keys

[bugzilla-daemon at webkit.org]

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


darin at apple.com changed:

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




------- Comment #8 from darin at apple.com  2008-06-11 14:57 PDT -------
(From update of attachment 21642)
Thanks!

This looks good.

Normally, we require that the patch includes the output of the test was well as
the input. So there should be files with "expected" in their names in the
patch. You generate those by running run-webkit-tests.

Also, we try to design the tests so they are self explanatory. Someone should
be able to tell what successful results look like.

Further, we often make the tests "text only" so they can be used
cross-platform. That's done by making sure the actual text of the result
reflects success (so a dump of the render tree is not necessary) and adding a
call to layoutTestController.dumpAsText. This tells the test engine to dump
only the text rather than the render tree.

There are many examples to show how this is done. One is
editing/selection/anchor-focus1.html -- just to pick out a specific example.

By the way, I'm not absolutely sure you need the runEditingTest() machinery. I
think that sets up some things you really don't need if you're testing input in
a textarea. You'd have to find a different way to do
moveSelectionForwardBySentenceCommand() and moveSelectionForwardByLineCommand()
if you didn't use editing.js -- I think you could just make keyboard events for
arrow keys for those.

It's not good to use the "by sentence" in the test, since that's very platform
dependent. It would be better to move to the end of the line than to move
forward by a sentence. Maybe the easiest thing would be to move forward by a
line and then backward by a character?

review- because this doesn't include expected test results, but please consider
my other comments too.


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