Use FocusParams in FocusController::setFocusedElement and Document::setFocusedElement arguments.

Use FocusParams in FocusController::setFocusedElement and Document::setFocusedElement arguments.

Adds a FocusController::setFocusedElement overload and Document::clearFocusedElement
to simplify the code.

These functions receive unused SelectionBehaviorOnFocus as a member of FocusParams.
We'll start to use SelectionBehaviorOnFocus in these functions later.

This CL introduces SelectionBehaviorOnFocus::None for setFocusedElement
callsites other than Element::focus.

This CL has no behavior changes.

BUG=526516
Committed: https://crrev.com/99a0f4166c156b1bcd64ffd8b04a087a311175d3
Cr-Commit-Position: refs/heads/master@{#359073}

[tkent, 2015-11-11 01:24:30]

Description was changed from

==========
Use FocusParams in FocusController::setFocusedElement and
Document::setFocusedElement.

FocusController::setFocusedElement and Documenet::setFocusedElement
receive unused SelectionBehaviorOnFocus in FocusParams.  We'll start to use
SelectionBehaviorOnFocus in these functions later.
This CL introduces SelectionBehaviorOnFocus::None for setFocusedElement
callsites other than Element::focus.

This CL has no behavior changes.

BUG=526516
==========

to

==========
Use FocusParams in FocusController::setFocusedElement and
Document::setFocusedElement arguments.

Adds a FocusController::setFocusedElement overload and
Document::clearFocusedElement
to simplify the code.

These functions receive unused SelectionBehaviorOnFocus as a member of
FocusParams.
We'll start to use SelectionBehaviorOnFocus in these functions later.

This CL introduces SelectionBehaviorOnFocus::None for setFocusedElement
callsites other than Element::focus.

This CL has no behavior changes.

BUG=526516
==========

[tkent, 2015-11-11 01:54:27]

tkent@chromium.org changed reviewers:
+ kochi@chromium.org

[tkent, 2015-11-11 01:54:28]

kochi@, would you review this please?

My plan is to move updateFocusAppearance(SelectionBehaviorOnFocus) from
Element::focus to Document::setFocusedElement.

[kochi, 2015-11-11 08:07:54]

lgtm

[tkent, 2015-11-11 08:11:07]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2015-11-11 08:11:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1433103002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1433103002/1

[commit-bot: I haz the power, 2015-11-11 09:21:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-11 09:22:20]

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