59021 2011-04-20 13:36:31 -0700 [chromium] firstRectForCharacterRange returns wrong result in DRT 2013-04-11 13:40:24 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) PC OS X 10.5 RESOLVED WONTFIX P2 Normal --- 1 xji webkit-unassigned schenney oldest_to_newest 389483 0 90400 xji 2011-04-20 13:36:31 -0700 Created attachment 90400 test case For the following test case, <div id="div" dir=ltr class="test_move_by_word" title="0 4 11 15|18 15 11 4 0" contenteditable>abc def hij opq</div> When caret is at the right of "opq", the caretRect returned from firstRectForCharacterRange() is NULL. Following is the gdb result. Breakpoint 1, WebKit::WebFrameImpl::firstRectForCharacterRange ( this=0x7fffe8097df0, location=18, length=0, rect=...) at third_party/WebKit/Source/WebKit/chromium/src/WebFrameImpl.cpp:1163 1163 if (!range) (gdb) p range $4 = {m_ptr = 0x0} (gdb) p scope->nodeType() $5 = WebCore::Node::ELEMENT_NODE (gdb) where 3 #0 WebKit::WebFrameImpl::firstRectForCharacterRange (this=0x7fffe8097df0, location=18, length=0, rect=...) at third_party/WebKit/Source/WebKit/chromium/src/WebFrameImpl.cpp:1163 #1 0x000000000045b658 in TextInputController::firstRectForCharacterRange ( this=0x7fffe8097590, arguments=..., result=0x7fffffffb5c0) at third_party/WebKit/Tools/DumpRenderTree/chromium/TextInputController.cpp:192 #2 0x000000000045c42a in CppBoundClass::MemberCallback<TextInputController>::run (this=0x7fffe8089b00, arguments=..., result=0x7fffffffb5c0) at third_party/WebKit/Tools/DumpRenderTree/chromium/CppBoundClass.h:125 (More stack frames follow...) scope is the <div> node. TextIterator::rangeFromLocationAndLength(<div> node, 18, 0) returns NULL range. 389549 1 xji 2011-04-20 14:57:12 -0700 related chromium's bug: http://code.google.com/p/chromium/issues/detail?id=3273 90400 2011-04-20 13:36:31 -0700 2011-04-20 13:36:31 -0700 test case FirstRectForCharacterRange.html text/html 1229 xji PCFET0NUWVBFIEhUTUw+CjxodG1sPgo8aGVhZD4KPG1ldGEgaHR0cC1lcXVpdj0iQ29udGVudC1U eXBlIiBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9dXRmLTgiPgo8c2NyaXB0Pgp2YXIgbWVz c2FnZXMgPSBbXTsKCmZ1bmN0aW9uIGxvZyhtZXNzYWdlKQp7CiAgICBtZXNzYWdlcy5wdXNoKG1l c3NhZ2UpOwp9CgpmdW5jdGlvbiBmbHVzaExvZygpCnsKICAgIGRvY3VtZW50LmdldEVsZW1lbnRC eUlkKCJjb25zb2xlIikuYXBwZW5kQ2hpbGQoZG9jdW1lbnQuY3JlYXRlVGV4dE5vZGUobWVzc2Fn ZXMuam9pbigiIikpKTsKfQoKZnVuY3Rpb24gY2FyZXRDb29yZGluYXRlcygpCnsKICAgIGlmICgh d2luZG93LnRleHRJbnB1dENvbnRyb2xsZXIpIHsKICAgICAgICByZXR1cm4geyB4OiAwLCB5IDow IH07CiAgICB9CiAgICB2YXIgc2VsZWN0ZWRSYW5nZSA9IHRleHRJbnB1dENvbnRyb2xsZXIuc2Vs ZWN0ZWRSYW5nZSgpOwogICAgbG9nKCJzZWxlY3RlZFJhbmdlOiAiICsgc2VsZWN0ZWRSYW5nZVsw XSArICJcbiIpOwoKICAgIHZhciBjYXJldFJlY3QgPSB0ZXh0SW5wdXRDb250cm9sbGVyLmZpcnN0 UmVjdEZvckNoYXJhY3RlclJhbmdlKHRleHRJbnB1dENvbnRyb2xsZXIuc2VsZWN0ZWRSYW5nZSgp WzBdLCAwKTsKICAgIGxvZygiY2FyZXRSZWN0OiAiICsgY2FyZXRSZWN0ICsgIlxuIik7CiAgICAv L3JldHVybiB7IHg6IGNhcmV0UmVjdFswXSwgeTogY2FyZXRSZWN0WzFdIH07Cn0KCgpvbmxvYWQg PSBmdW5jdGlvbigpIHsKICAgIHZhciB0ZXN0ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImRp diIpOwogICAgdmFyIHNlbCA9IGdldFNlbGVjdGlvbigpOwogICAgc2VsLnNldFBvc2l0aW9uKHRl c3QsIDApOwogICAgc2VsLm1vZGlmeSgibW92ZSIsICJmb3J3YXJkIiwgImxpbmVCb3VuZGFyeSIp OwogICAgdmFyIHBvaW50ID0gY2FyZXRDb29yZGluYXRlcygpOwogICAgZmx1c2hMb2coKTsKfTsK CmlmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCiAgICBsYXlvdXRUZXN0Q29udHJvbGxl ci5kdW1wQXNUZXh0KCk7Cjwvc2NyaXB0Pgo8L2hlYWQ+Cjxib2R5Pgo8ZGl2IGlkPSJkaXYiIGRp cj1sdHIgY2xhc3M9InRlc3RfbW92ZV9ieV93b3JkIiB0aXRsZT0iMCA0IDExIDE1fDE4IDE1IDEx IDQgMCIgY29udGVudGVkaXRhYmxlPmFiYyBkZWYgICAgaGlqIG9wcTwvZGl2PgoKPHByZSBpZD0i Y29uc29sZSI+PC9wcmU+CjwvYm9keT4KPC9odG1sPgo=