Reviewed by Anders.
- http://bugzilla.opendarwin.org/show_bug.cgi?id=10135
Cleanup: remove useless parameters from RenderObject::position()
No test possible (no functionality change)
* rendering/RenderBox.cpp:
(WebCore::RenderBox::position):
* rendering/RenderBox.h:
* rendering/RenderObject.h:
(WebCore::RenderObject::position):
* rendering/RenderTableRow.h:
(WebCore::RenderTableRow::position):
* rendering/RenderTableSection.h:
(WebCore::RenderTableSection::position):
* rendering/RenderText.cpp:
(WebCore::RenderText::position):
* rendering/RenderText.h:
* rendering/bidi.cpp:
(WebCore::RenderBlock::constructLine):
(WebCore::RenderBlock::computeVerticalPositionsForLine):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@15666 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/rendering/bidi.cpp b/WebCore/rendering/bidi.cpp
index d23490e..90121fb 100644
--- a/WebCore/rendering/bidi.cpp
+++ b/WebCore/rendering/bidi.cpp
@@ -813,7 +813,10 @@
if (r->box->isInlineTextBox()) {
InlineTextBox *text = static_cast<InlineTextBox*>(r->box);
text->setStart(r->start);
- text->setLen(r->stop-r->start);
+ text->setLen(r->stop - r->start);
+ bool visuallyOrdered = r->obj->style()->visuallyOrdered();
+ text->m_reversed = r->level % 2 && !visuallyOrdered;
+ text->m_dirOverride = r->override || visuallyOrdered;
}
}
}
@@ -995,7 +998,7 @@
// Position is used to properly position both replaced elements and
// to update the static normal flow x/y of positioned elements.
- r->obj->position(r->box, r->start, r->stop - r->start, r->level%2, r->override);
+ r->obj->position(r->box);
}
}