Get rid of EUpdateStyle parameter from isEditablePosition()

Get rid of unused parameter EUpdateStyle parameter from isEditablePosition()

This patch gets rid of unused parameter |EUpdateStyle| from
|isEditablePosition()| to reduce code size for improving code health.

This patch also introduces |needsLayoutTreeUpdate()| for checking
|isEditablePosition()| called with clean layout tree.

BUG=623005
TEST=n/a; no behavior changes

Committed: https://crrev.com/019e4a359a1f7db417cf1132e5fd982cfacec21c
Cr-Commit-Position: refs/heads/master@{#402379}

[yosin_UTC9, 2016-06-22 07:55:38]

Description was changed from

==========
2016-06-22T16:53:43

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

to

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=n/a
TEST=n/a; no behavior changes
==========

[yosin_UTC9, 2016-06-23 04:28:06]

Description was changed from

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=n/a
TEST=n/a; no behavior changes
==========

to

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=n/a
TEST=n/a; no behavior changes

fast/css-generated-content/drag-state.html
fast/forms/number/number-wheel-event.html
fast/forms/number/number-keyoperation.html
fast/forms/search/search-disabled-readonly.html
fast/forms/search/input-search-press-escape-key.html
fast/forms/autofilled.html
=> adjustStyleDifference



paint/invalidation/caret-color.html
FrameSelection::hasCursor() from BlockPainter
LocalFrame::nodeImage() => DragImageBuilder => LayoutObject::updateDragState()
==========

[yosin_UTC9, 2016-06-23 09:56:50]

Description was changed from

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=n/a
TEST=n/a; no behavior changes

fast/css-generated-content/drag-state.html
fast/forms/number/number-wheel-event.html
fast/forms/number/number-keyoperation.html
fast/forms/search/search-disabled-readonly.html
fast/forms/search/input-search-press-escape-key.html
fast/forms/autofilled.html
=> adjustStyleDifference



paint/invalidation/caret-color.html
FrameSelection::hasCursor() from BlockPainter
LocalFrame::nodeImage() => DragImageBuilder => LayoutObject::updateDragState()
==========

to

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=n/a
TEST=n/a; no behavior changes

fast/repaint/inline-outline-repaint.html
=> repaint diff


fast/css-generated-content/drag-state.html
fast/forms/number/number-wheel-event.html
fast/forms/number/number-keyoperation.html
fast/forms/search/search-disabled-readonly.html
fast/forms/search/input-search-press-escape-key.html
fast/forms/autofilled.html
=> adjustStyleDifference

paint/invalidation/caret-color.html
FrameSelection::hasCursor() from BlockPainter
LocalFrame::nodeImage() => DragImageBuilder => LayoutObject::updateDragState()
==========

[yosin_UTC9, 2016-06-24 06:43:08]

Description was changed from

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=n/a
TEST=n/a; no behavior changes

fast/repaint/inline-outline-repaint.html
=> repaint diff


fast/css-generated-content/drag-state.html
fast/forms/number/number-wheel-event.html
fast/forms/number/number-keyoperation.html
fast/forms/search/search-disabled-readonly.html
fast/forms/search/input-search-press-escape-key.html
fast/forms/autofilled.html
=> adjustStyleDifference

paint/invalidation/caret-color.html
FrameSelection::hasCursor() from BlockPainter
LocalFrame::nodeImage() => DragImageBuilder => LayoutObject::updateDragState()
==========

to

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=n/a
TEST=n/a; no behavior changes

fast/repaint/inline-outline-repaint.html
=> repaint diff


fast/css-generated-content/drag-state.html
fast/forms/number/number-wheel-event.html
fast/forms/number/number-keyoperation.html
fast/forms/search/search-disabled-readonly.html
fast/forms/search/input-search-press-escape-key.html
fast/forms/autofilled.html
=> adjustStyleDifference

paint/invalidation/caret-color.html

fast/repaint/inline-outline-repaint.html

FrameSelection::hasCursor() from BlockPainter
LocalFrame::nodeImage() => DragImageBuilder => LayoutObject::updateDragState()
==========

[yosin_UTC9, 2016-06-24 07:50:20]

Description was changed from

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=n/a
TEST=n/a; no behavior changes

fast/repaint/inline-outline-repaint.html
=> repaint diff


fast/css-generated-content/drag-state.html
fast/forms/number/number-wheel-event.html
fast/forms/number/number-keyoperation.html
fast/forms/search/search-disabled-readonly.html
fast/forms/search/input-search-press-escape-key.html
fast/forms/autofilled.html
=> adjustStyleDifference

paint/invalidation/caret-color.html

fast/repaint/inline-outline-repaint.html

FrameSelection::hasCursor() from BlockPainter
LocalFrame::nodeImage() => DragImageBuilder => LayoutObject::updateDragState()
==========

to

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=623005
TEST=n/a; no behavior changes

fast/repaint/inline-outline-repaint.html
=> repaint diff


fast/css-generated-content/drag-state.html
fast/forms/number/number-wheel-event.html
fast/forms/number/number-keyoperation.html
fast/forms/search/search-disabled-readonly.html
fast/forms/search/input-search-press-escape-key.html
fast/forms/autofilled.html
=> adjustStyleDifference

paint/invalidation/caret-color.html

fast/repaint/inline-outline-repaint.html

FrameSelection::hasCursor() from BlockPainter
LocalFrame::nodeImage() => DragImageBuilder => LayoutObject::updateDragState()
==========

[yosin_UTC9, 2016-06-27 08:42:25]

Description was changed from

==========
Get rid of EUpdateStyle parameter from isEditablePosition()

BUG=623005
TEST=n/a; no behavior changes

fast/repaint/inline-outline-repaint.html
=> repaint diff


fast/css-generated-content/drag-state.html
fast/forms/number/number-wheel-event.html
fast/forms/number/number-keyoperation.html
fast/forms/search/search-disabled-readonly.html
fast/forms/search/input-search-press-escape-key.html
fast/forms/autofilled.html
=> adjustStyleDifference

paint/invalidation/caret-color.html

fast/repaint/inline-outline-repaint.html

FrameSelection::hasCursor() from BlockPainter
LocalFrame::nodeImage() => DragImageBuilder => LayoutObject::updateDragState()
==========

to

==========
Get rid of unused parameter EUpdateStyle parameter from isEditablePosition()

This patch gets rid of unused parameter |EUpdateStyle| from
|isEditablePosition()| to reduce code size for improving code health.

BUG=623005
TEST=n/a; no behavior changes
==========

[yosin_UTC9, 2016-06-27 08:43:27]

Description was changed from

==========
Get rid of unused parameter EUpdateStyle parameter from isEditablePosition()

This patch gets rid of unused parameter |EUpdateStyle| from
|isEditablePosition()| to reduce code size for improving code health.

BUG=623005
TEST=n/a; no behavior changes
==========

to

==========
Get rid of unused parameter EUpdateStyle parameter from isEditablePosition()

This patch gets rid of unused parameter |EUpdateStyle| from
|isEditablePosition()| to reduce code size for improving code health.

This patch also introduces |needsLayoutTreeUpdate()| for checking
|isEditablePosition()| called with clean layout tree.

BUG=623005
TEST=n/a; no behavior changes
==========

[yosin_UTC9, 2016-06-27 08:43:36]

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

[yosin_UTC9, 2016-06-27 08:43:37]

PTAL

[yoichio, 2016-06-28 02:04:51]

lgtm

[yosin_UTC9, 2016-06-28 02:09:55]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-06-28 02:10:20]

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-06-28 02:14:55]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-06-28 02:16:06]

Description was changed from

==========
Get rid of unused parameter EUpdateStyle parameter from isEditablePosition()

This patch gets rid of unused parameter |EUpdateStyle| from
|isEditablePosition()| to reduce code size for improving code health.

This patch also introduces |needsLayoutTreeUpdate()| for checking
|isEditablePosition()| called with clean layout tree.

BUG=623005
TEST=n/a; no behavior changes
==========

to

==========
Get rid of unused parameter EUpdateStyle parameter from isEditablePosition()

This patch gets rid of unused parameter |EUpdateStyle| from
|isEditablePosition()| to reduce code size for improving code health.

This patch also introduces |needsLayoutTreeUpdate()| for checking
|isEditablePosition()| called with clean layout tree.

BUG=623005
TEST=n/a; no behavior changes

Committed: https://crrev.com/019e4a359a1f7db417cf1132e5fd982cfacec21c
Cr-Commit-Position: refs/heads/master@{#402379}
==========

[commit-bot: I haz the power, 2016-06-28 02:16:07]

Patchset 4 (id:??) landed as
https://crrev.com/019e4a359a1f7db417cf1132e5fd982cfacec21c
Cr-Commit-Position: refs/heads/master@{#402379}