Get rid of a redundant parameter ResetCaretBlinkOption from FrameSelection::updateAppearance()

Get rid of a redundant parameter ResetCaretBlinkOption from FrameSelection::updateAppearance()

This patch gets rid of rid of a redundant parameter |ResetCaretBlinkOption| from
|FrameSelection::updateAppearance()| with introducing |stopCaretBlinkTimer()|
by calling |stopCaretBlinkTimer()| at call site which calls |updateAppearance()|
with |ResetCaretBlink| to simplify source code for improving readability.

This patch is a follow-up patch of http://crrev.com/1629213002, which simplifies
code path in |updateAppearance()|.

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

Committed: https://crrev.com/013af3f06c0c8d9d7b5bb52421ddff41a697f0bb
Cr-Commit-Position: refs/heads/master@{#371472}

[yosin_UTC9, 2016-01-26 05:41:06]

Description was changed from

==========
Introduce |FrameSelection::stopCaretBlinkTimer()|

BUG=n/a
==========

to

==========
Introduce FrameSelection::stopCaretBlinkTimer()

BUG=n/a
==========

[yosin_UTC9, 2016-01-26 05:41:14]

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

[commit-bot: I haz the power, 2016-01-26 05:41:23]

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

[yosin_UTC9, 2016-01-26 07:10:45]

Description was changed from

==========
Introduce FrameSelection::stopCaretBlinkTimer()

BUG=n/a
==========

to

==========
Get rid of a redundant parameter ResetCaretBlinkOption from
FrameSelection::updateAppearance()

This patch gets rid of rid of a redundant parameter |ResetCaretBlinkOption| from
|FrameSelection::updateAppearance()| with introducing |stopCaretBlinkTimer()|
by calling |stopCaretBlinkTimer()| at call site which calls |updateAppearance()|
with |ResetCaretBlink| to simplify source code for improving readability.

This patch is a follow-up patch of http://crrev.com/1629213002, which simplifies
code path in |updateAppearance()|.

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

[yosin_UTC9, 2016-01-26 07:10:45]

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

[yosin_UTC9, 2016-01-26 07:10:54]

PTAL

[commit-bot: I haz the power, 2016-01-26 07:11:44]

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

[commit-bot: I haz the power, 2016-01-26 07:11:44]

Dry run: This issue passed the CQ dry run.

[yoichio, 2016-01-26 07:22:20]

lgtm

[yosin_UTC9, 2016-01-26 07:29:09]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-01-26 07:29:20]

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

[commit-bot: I haz the power, 2016-01-26 07:37:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-26 07:38:39]

Description was changed from

==========
Get rid of a redundant parameter ResetCaretBlinkOption from
FrameSelection::updateAppearance()

This patch gets rid of rid of a redundant parameter |ResetCaretBlinkOption| from
|FrameSelection::updateAppearance()| with introducing |stopCaretBlinkTimer()|
by calling |stopCaretBlinkTimer()| at call site which calls |updateAppearance()|
with |ResetCaretBlink| to simplify source code for improving readability.

This patch is a follow-up patch of http://crrev.com/1629213002, which simplifies
code path in |updateAppearance()|.

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

to

==========
Get rid of a redundant parameter ResetCaretBlinkOption from
FrameSelection::updateAppearance()

This patch gets rid of rid of a redundant parameter |ResetCaretBlinkOption| from
|FrameSelection::updateAppearance()| with introducing |stopCaretBlinkTimer()|
by calling |stopCaretBlinkTimer()| at call site which calls |updateAppearance()|
with |ResetCaretBlink| to simplify source code for improving readability.

This patch is a follow-up patch of http://crrev.com/1629213002, which simplifies
code path in |updateAppearance()|.

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

Committed: https://crrev.com/013af3f06c0c8d9d7b5bb52421ddff41a697f0bb
Cr-Commit-Position: refs/heads/master@{#371472}
==========

[commit-bot: I haz the power, 2016-01-26 07:38:40]

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