Mixing content editable and non-editable in direction RTL

Source/core/editing/htmlediting.cpp

Index: Source/core/editing/htmlediting.cpp
diff --git a/Source/core/editing/htmlediting.cpp b/Source/core/editing/htmlediting.cpp
index 3813ae4cacc9522a4702d2e33e1404b4538b4432..4336f4f1abab4cf060b8f10734d473344a029926 100644
--- a/Source/core/editing/htmlediting.cpp
+++ b/Source/core/editing/htmlediting.cpp
@@ -351,13 +351,13 @@ bool inSameContainingBlockFlowElement(Node* a, Node* b)
     return a && b && enclosingBlockFlowElement(*a) == enclosingBlockFlowElement(*b);
 }
 
-TextDirection directionOfEnclosingBlock(const Position& position)
+bool isEnclosingBoxHasLeftToRightDirection(const Position& position)
 {
     Element* enclosingBlockElement = enclosingBlock(position.containerNode());
     if (!enclosingBlockElement)
-        return LTR;
+        return true;
     RenderObject* renderer = enclosingBlockElement->renderer();
-    return renderer ? renderer->style()->direction() : LTR;
+    return renderer ? (renderer->style()->direction() == LTR) : true;

[yosin, 2014/10/01 08:00:11]

nit: We don't need to use "?" operator here. We could write:
return !renderer || render->style()->direction == LTR

[Habib Virji, 2014/10/01 15:56:16]

Done.

 }
 
 // This method is used to create positions in the DOM. It returns the maximum valid offset