[Editing][Regression] Contenteditable w/ "-webkit-user-select:all" should be editable.

third_party/WebKit/Source/core/dom/Node.cpp

Index: third_party/WebKit/Source/core/dom/Node.cpp
diff --git a/third_party/WebKit/Source/core/dom/Node.cpp b/third_party/WebKit/Source/core/dom/Node.cpp
index 84a8cea65e1140ef19992d5e3c884b1fd0ce5c1f..c4cf180839ef4ae124141a4e3fbc4879e58bd6b2 100644
--- a/third_party/WebKit/Source/core/dom/Node.cpp
+++ b/third_party/WebKit/Source/core/dom/Node.cpp
@@ -557,9 +557,10 @@ bool Node::hasEditableStyle(EditableLevel editableLevel, UserSelectAllTreatment
 
     for (const Node& node : NodeTraversal::inclusiveAncestorsOf(*this)) {
         if ((node.isHTMLElement() || node.isDocumentNode()) && node.layoutObject()) {
+            const bool isContentEditable = node.isHTMLElement() && toHTMLElement(node).contentEditable() == "true";

[yosin_UTC9, 2016/07/06 07:56:48]

We should not check "contentEditable" attribute. editability is defined by
-webkit-user-modify.

We need to support following case:

<div contentEditable>
  ...
  <span style="user-select: all">this content is editable</span>
  ...
</div>

[yoichio, 2016/07/07 07:57:02]

Done.

             // Elements with user-select: all style are considered atomic
             // therefore non editable.
-            if (nodeIsUserSelectAll(&node) && treatment == UserSelectAllIsAlwaysNonEditable)
+            if (!isContentEditable && nodeIsUserSelectAll(&node) && treatment == UserSelectAllIsAlwaysNonEditable)
                 return false;
             switch (node.layoutObject()->style()->userModify()) {
             case READ_ONLY: