[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

Move Node::*editable* functions to core/editing/EditingUtilities

TEST=No change in behavior

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

[yoichio, 2016-07-25 09:03:39]

Description was changed from

==========
[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

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

to

==========
[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

Move Node::*editable* functions to core/editing/EditingUtilities

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

[yoichio, 2016-07-25 09:03:39]

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

[yoichio, 2016-07-25 09:03:54]

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

[commit-bot: I haz the power, 2016-07-25 09:04:07]

Dry run: 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-25 10:26:58]

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

[commit-bot: I haz the power, 2016-07-25 10:26:59]

Dry run: This issue passed the CQ dry run.

[yoichio, 2016-07-26 02:00:18]

Description was changed from

==========
[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

Move Node::*editable* functions to core/editing/EditingUtilities

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

to

==========
[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

Move Node::*editable* functions to core/editing/EditingUtilities

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

[tkent, 2016-07-26 02:05:18]

lgtm

[yoichio, 2016-07-26 05:04:25]

The CQ bit was checked by yoichio@chromium.org

[commit-bot: I haz the power, 2016-07-26 05:04:37]

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-26 05:08:31]

Description was changed from

==========
[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

Move Node::*editable* functions to core/editing/EditingUtilities

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

to

==========
[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

Move Node::*editable* functions to core/editing/EditingUtilities

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

[commit-bot: I haz the power, 2016-07-26 05:08:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-26 05:09:58]

Description was changed from

==========
[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

Move Node::*editable* functions to core/editing/EditingUtilities

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

to

==========
[Editing][DOM][CodeHealth] Move Node::*editable* functions to core/editing

Move Node::*editable* functions to core/editing/EditingUtilities

TEST=No change in behavior

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

[commit-bot: I haz the power, 2016-07-26 05:10:00]

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

[tkent, 2016-07-27 06:26:41]

https://codereview.chromium.org/2177783003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/editing/EditingUtilities.h (right):

https://codereview.chromium.org/2177783003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/editing/EditingUtilities.h:75: CORE_EXPORT bool
isContentEditable(const Node&);
Can you clarify differences between
 - isContentEditable and hasEditableStyle
 - isContentRichlyEditable and layoutObjectIsRichlyEditable
 - rootEditableElement and highestEditableRoot
?

[yoichio, 2016-07-27 07:35:26]

On 2016/07/27 06:26:41, tkent wrote:
>
https://codereview.chromium.org/2177783003/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/editing/EditingUtilities.h (right):
> 
>
https://codereview.chromium.org/2177783003/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/editing/EditingUtilities.h:75: CORE_EXPORT bool
> isContentEditable(const Node&);
> Can you clarify differences between
>  - isContentEditable and hasEditableStyle
>  - isContentRichlyEditable and layoutObjectIsRichlyEditable
>  - rootEditableElement and highestEditableRoot
> ?

I have found some function does logically same in
 these refactoring.
I will 'shrink' them:)