[Editing][DOM][CodeHealth] Make Node::hasEditableStyle global functions.

[Editing][DOM][CodeHealth] Make Node::hasEditableStyle global functions.

Make Node::hasEditableStyle and Node::layoutObjectIsRichlyEditable global functions.

This is a preparation to move these functions from core/dom to core/editing.

TEST=No change in behavior

Committed: https://crrev.com/d65d16a817f1f0522c93bc2471d1a62e37285f3e
Cr-Commit-Position: refs/heads/master@{#407036}

[yoichio, 2016-07-21 08:40:51]

The CQ bit was checked by yoichio@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-21 08:41:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yoichio, 2016-07-21 09:15:22]

Description was changed from

==========
[Editing][DOM][CodeHealth] Make Node::hasEditableStyle and Node::isRichEditable
global functions.

BUG=
==========

to

==========
[Editing][DOM][CodeHealth] Make Node::hasEditableStyle global functions.

Make Node::hasEditableStyle and Node::layoutObjectIsRichlyEditable global
functions.

This is a preparation to move these functions from core/dom to core/editing.

TEST=No change in behavior
==========

[yoichio, 2016-07-21 09:15:22]

yoichio@chromium.org changed reviewers:
+ tkent@chromium.org, yosin@chromium.org

[tkent, 2016-07-21 09:26:40]

lgtm

https://codereview.chromium.org/2171493003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/dom/Node.cpp (right):

https://codereview.chromium.org/2171493003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/Node.cpp:613: bool
layoutObjectIsRichlyEditable(const Node& node, EditableType editableType)
Off topic: The names 'hasEditableStyle' and 'layoutObjectIsRichlyEditable' are
inconsistent.  Let's rename them later.
Off topic: It seems we can remove EditableType argument.  Let's remove it later.

[commit-bot: I haz the power, 2016-07-21 09:51:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-21 09:51:51]

Dry run: This issue passed the CQ dry run.

[yoichio, 2016-07-22 02:11:59]

On 2016/07/21 09:26:40, tkent wrote:
> lgtm
> 
>
https://codereview.chromium.org/2171493003/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/dom/Node.cpp (right):
> 
>
https://codereview.chromium.org/2171493003/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/dom/Node.cpp:613: bool
> layoutObjectIsRichlyEditable(const Node& node, EditableType editableType)
> Off topic: The names 'hasEditableStyle' and 'layoutObjectIsRichlyEditable' are
> inconsistent.  Let's rename them later.
> Off topic: It seems we can remove EditableType argument.  Let's remove it
later.

Yup, code is still very dirty so we should clean up each of them.

[yoichio, 2016-07-22 02:12:08]

The CQ bit was checked by yoichio@chromium.org

[commit-bot: I haz the power, 2016-07-22 02:13:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-22 03:41:25]

Description was changed from

==========
[Editing][DOM][CodeHealth] Make Node::hasEditableStyle global functions.

Make Node::hasEditableStyle and Node::layoutObjectIsRichlyEditable global
functions.

This is a preparation to move these functions from core/dom to core/editing.

TEST=No change in behavior
==========

to

==========
[Editing][DOM][CodeHealth] Make Node::hasEditableStyle global functions.

Make Node::hasEditableStyle and Node::layoutObjectIsRichlyEditable global
functions.

This is a preparation to move these functions from core/dom to core/editing.

TEST=No change in behavior
==========

[commit-bot: I haz the power, 2016-07-22 03:41:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-22 03:44:11]

Description was changed from

==========
[Editing][DOM][CodeHealth] Make Node::hasEditableStyle global functions.

Make Node::hasEditableStyle and Node::layoutObjectIsRichlyEditable global
functions.

This is a preparation to move these functions from core/dom to core/editing.

TEST=No change in behavior
==========

to

==========
[Editing][DOM][CodeHealth] Make Node::hasEditableStyle global functions.

Make Node::hasEditableStyle and Node::layoutObjectIsRichlyEditable global
functions.

This is a preparation to move these functions from core/dom to core/editing.

TEST=No change in behavior

Committed: https://crrev.com/d65d16a817f1f0522c93bc2471d1a62e37285f3e
Cr-Commit-Position: refs/heads/master@{#407036}
==========

[commit-bot: I haz the power, 2016-07-22 03:44:13]

Patchset 1 (id:??) landed as
https://crrev.com/d65d16a817f1f0522c93bc2471d1a62e37285f3e
Cr-Commit-Position: refs/heads/master@{#407036}