Commit 852d80b2 authored by rjw's avatar rjw

Fixed 3099505. With the recent white space changes, multiple slaves may

        be created for contiguous text on the same line.  The fix changes the
        iteration order of over the slave array.  Now, when searching for the end
        point of a line we'll always find the last slave on a line first.

        * khtml/khtml_part.cpp:
        (lastSlaveAt):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2679 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 5e10a7c6
2002-11-14 Richard Williamson <rjw@apple.com>
Fixed 3099505. With the recent white space changes, multiple slaves may
be created for contiguous text on the same line. The fix changes the
iteration order of over the slave array. Now, when searching for the end
point of a line we'll always find the last slave on a line first.
* khtml/khtml_part.cpp:
(lastSlaveAt):
2002-11-14 Darin Adler <darin@apple.com>
- fixed 3095854 -- small select boxes get no scrollbar
......
2002-11-14 Richard Williamson <rjw@apple.com>
Fixed 3099505. With the recent white space changes, multiple slaves may
be created for contiguous text on the same line. The fix changes the
iteration order of over the slave array. Now, when searching for the end
point of a line we'll always find the last slave on a line first.
* khtml/khtml_part.cpp:
(lastSlaveAt):
2002-11-14 Darin Adler <darin@apple.com>
- fixed 3095854 -- small select boxes get no scrollbar
......
2002-11-14 Richard Williamson <rjw@apple.com>
Fixed 3099505. With the recent white space changes, multiple slaves may
be created for contiguous text on the same line. The fix changes the
iteration order of over the slave array. Now, when searching for the end
point of a line we'll always find the last slave on a line first.
* khtml/khtml_part.cpp:
(lastSlaveAt):
2002-11-14 Darin Adler <darin@apple.com>
- fixed 3095854 -- small select boxes get no scrollbar
......
......@@ -4015,7 +4015,7 @@ static bool lastSlaveAt (khtml::RenderObject *renderNode, int y, DOM::NodeImpl*&
if (renderNode->isText()){
khtml::RenderText *textRenderer = static_cast<khtml::RenderText *>(renderNode);
khtml::TextSlaveArray slaves = textRenderer->textSlaves();
for (int i = 0; i < (int)slaves.count(); i++){
for (int i = (int)slaves.count()-1; i >= 0; i--){
if (slaves[i]->m_y == y){
endNode = textRenderer->element();
endOffset = slaves[i]->m_start + slaves[i]->m_len;
......
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