Changeset 57037 in webkit

Timestamp:

Apr 2, 2010 5:38:27 PM (14 years ago)

Author:

eric@webkit.org

Message:

2010-04-02 MORITA Hajime <​morrita@google.com>

Reviewed by Alexey Proskuryakov.

​https://bugs.webkit.org/show_bug.cgi?id=37011
Position::primaryDirection() should not use its own accessor.

No new tests. This is small clenaup with no behaviour change.

• dom/Position.cpp: (WebCore::Position::primaryDirection):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/Position.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-04-02  MORITA Hajime  <morrita@google.com>
+
+        Reviewed by Alexey Proskuryakov.
+
+        https://bugs.webkit.org/show_bug.cgi?id=37011
+        Position::primaryDirection() should not use its own accessor.
+
+        No new tests. This is small clenaup with no behaviour change.
+
+        * dom/Position.cpp:
+        (WebCore::Position::primaryDirection):
+
 2010-04-02  Eric Uhrhane  <ericu@chromium.org>
 

trunk/WebCore/dom/Position.cpp

@@ -1113 +1113 @@
 {
     TextDirection primaryDirection = LTR;
-    for (const RenderObject* r = node()->renderer(); r; r = r->parent()) {
+    for (const RenderObject* r = m_anchorNode->renderer(); r; r = r->parent()) {
         if (r->isBlockFlow()) {
             primaryDirection = r->style()->direction();