Commit 9bbc4e38 authored by ap's avatar ap

- Fix http://bugzilla.opendarwin.org/show_bug.cgi?id=6289

        REGRESSION: firstCharacterForRange(0, 0) returns a 0x0 rect

        * khtml/editing/visible_text.cpp:
        (khtml::TextIterator::rangeFromLocationAndLength):
        Only special-case (0, 0) ranges for empty documents.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@11845 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent d166e7a8
2005-12-30 Alexey Proskuryakov <ap@nypop.com>
- Fix http://bugzilla.opendarwin.org/show_bug.cgi?id=6289
REGRESSION: firstCharacterForRange(0, 0) returns a 0x0 rect
* khtml/editing/visible_text.cpp:
(khtml::TextIterator::rangeFromLocationAndLength):
Only special-case (0, 0) ranges for empty documents.
2005-12-29 Geoffrey Garen <ggaren@apple.com>
Added test case for <rdar://problem/4268278> Submitting a form in onUnload event
......
......@@ -997,7 +997,8 @@ RangeImpl *TextIterator::rangeFromLocationAndLength(DocumentImpl *doc, int range
TextIterator it(rangeOfContents(doc).get());
if (rangeLocation == 0 && rangeLength == 0) {
// FIXME: the atEnd() check shouldn't be necessary, workaround for <http://bugzilla.opendarwin.org/show_bug.cgi?id=6289>.
if (rangeLocation == 0 && rangeLength == 0 && it.atEnd()) {
int exception = 0;
textRunRange = it.range();
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment