Move Node::enclosingBlockFlowElement() to htmlediting.h

Move Node::enclosingBlockFlowElement() to htmlediting.h

Move Node::enclosingBlockFlowElement() / inSameContainingBlockFlowElement() to
htmlediting.h as this is only needed for editing.

R=leviw@chromium.org, yosin@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179081

[leviw_travelin_and_unemployed, 2014-07-28 21:39:56]

lgtm

https://codereview.chromium.org/421293004/diff/1/Source/core/dom/Position.cpp
File Source/core/dom/Position.cpp (right):

https://codereview.chromium.org/421293004/diff/1/Source/core/dom/Position.cpp...
Source/core/dom/Position.cpp:951: if
(enclosingBlockFlowElement(*deprecatedNode()) !=
enclosingBlockFlowElement(*pos.deprecatedNode()))
!inSameContainingBlockFlowElement(*deprecatedNode(), *pos.deprecatedNode())?

https://codereview.chromium.org/421293004/diff/1/Source/core/dom/Position.cpp...
Source/core/dom/Position.cpp:951: if
(enclosingBlockFlowElement(*deprecatedNode()) !=
enclosingBlockFlowElement(*pos.deprecatedNode()))
!inSameContainingBlockFlowElement(*deprecatedNode(), *pos.deprecatedNode())?

https://codereview.chromium.org/421293004/diff/1/Source/core/editing/htmledit...
File Source/core/editing/htmlediting.cpp (right):

https://codereview.chromium.org/421293004/diff/1/Source/core/editing/htmledit...
Source/core/editing/htmlediting.cpp:351: return a && b ?
enclosingBlockFlowElement(*a) == enclosingBlockFlowElement(*b) : false;
a && b && enclosingBlockFlowElement(*a) == enclosingBlockFlowElement(*b)?

[Inactive, 2014-07-28 21:47:11]

https://codereview.chromium.org/421293004/diff/1/Source/core/dom/Position.cpp
File Source/core/dom/Position.cpp (right):

https://codereview.chromium.org/421293004/diff/1/Source/core/dom/Position.cpp...
Source/core/dom/Position.cpp:951: if
(enclosingBlockFlowElement(*deprecatedNode()) !=
enclosingBlockFlowElement(*pos.deprecatedNode()))
On 2014/07/28 21:39:56, leviw wrote:
> !inSameContainingBlockFlowElement(*deprecatedNode(), *pos.deprecatedNode())?

Done.

https://codereview.chromium.org/421293004/diff/1/Source/core/editing/htmledit...
File Source/core/editing/htmlediting.cpp (right):

https://codereview.chromium.org/421293004/diff/1/Source/core/editing/htmledit...
Source/core/editing/htmlediting.cpp:351: return a && b ?
enclosingBlockFlowElement(*a) == enclosingBlockFlowElement(*b) : false;
On 2014/07/28 21:39:56, leviw wrote:
> a && b && enclosingBlockFlowElement(*a) == enclosingBlockFlowElement(*b)?

Done.

[Inactive, 2014-07-28 22:16:45]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-07-28 22:17:30]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/421293004/20001

[commit-bot: I haz the power, 2014-07-29 00:08:17]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/19329)

[commit-bot: I haz the power, 2014-07-29 00:20:47]

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

[commit-bot: I haz the power, 2014-07-29 00:20:48]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/19343)

[Inactive, 2014-07-29 00:36:34]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-07-29 00:37:31]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/421293004/20001

[commit-bot: I haz the power, 2014-07-29 01:19:00]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/19349)

[commit-bot: I haz the power, 2014-07-29 01:52:40]

Change committed as 179081