Do not show a tooltip same as the previous one.

Do not show a tooltip same as the previous one.

If a tooltip is closed by a keyboard event and the mouse cursor was
not moved, the same tooltip was shown again after some delay only on
Windows.  Blink receives native mousemove events repeatedly on Windows
even if the mouse cursor is not moved.

ChromeClient saves the last tooltip string and the last mouse position, and
do not call virtual setToolTip() if they are not changed. This CL not only
fixes the bug, but also reduces unnecessary IPC messages for tooltip.

BUG=557660
TEST=automated

Committed: https://crrev.com/91db68e8cb2080d1e9e07858efea3f7e198112a9
Cr-Commit-Position: refs/heads/master@{#363174}

[tkent, 2015-12-04 08:16:41]

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

[tkent, 2015-12-04 08:16:41]

yosin@, would you review this please?

[tkent, 2015-12-04 08:17:36]

Description was changed from

==========
Do not show a tooltip same as the previous one.

If a tooltip is closed by a keyboard event and the mouse cursor was
not moved, the same tooltip was shown again after some delay only on
Windows.  Blink receives native mousemove events repeatedly on Windows
even if the mouse cursor is not moved.

ChromeClient saves the last tooltip string and the mouse position, and
do not call virtual setToolTip(). This CL not only fixes the bug, but
also reduce unnecessary IPC messages for tooltip.

BUG=557660
TEST=automated
==========

to

==========
Do not show a tooltip same as the previous one.

If a tooltip is closed by a keyboard event and the mouse cursor was
not moved, the same tooltip was shown again after some delay only on
Windows.  Blink receives native mousemove events repeatedly on Windows
even if the mouse cursor is not moved.

ChromeClient saves the last tooltip string and the last mouse position, and
do not call virtual setToolTip() if they are not changed. This CL not only
fixes the bug, but also reduce unnecessary IPC messages for tooltip.

BUG=557660
TEST=automated
==========

[tkent, 2015-12-04 08:17:54]

Description was changed from

==========
Do not show a tooltip same as the previous one.

If a tooltip is closed by a keyboard event and the mouse cursor was
not moved, the same tooltip was shown again after some delay only on
Windows.  Blink receives native mousemove events repeatedly on Windows
even if the mouse cursor is not moved.

ChromeClient saves the last tooltip string and the last mouse position, and
do not call virtual setToolTip() if they are not changed. This CL not only
fixes the bug, but also reduce unnecessary IPC messages for tooltip.

BUG=557660
TEST=automated
==========

to

==========
Do not show a tooltip same as the previous one.

If a tooltip is closed by a keyboard event and the mouse cursor was
not moved, the same tooltip was shown again after some delay only on
Windows.  Blink receives native mousemove events repeatedly on Windows
even if the mouse cursor is not moved.

ChromeClient saves the last tooltip string and the last mouse position, and
do not call virtual setToolTip() if they are not changed. This CL not only
fixes the bug, but also reduces unnecessary IPC messages for tooltip.

BUG=557660
TEST=automated
==========

[yosin_UTC9, 2015-12-04 08:44:53]

lgtm

It is good change, I love it.

[yosin_UTC9, 2015-12-04 08:44:55]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2015-12-04 08:45:21]

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

[commit-bot: I haz the power, 2015-12-04 09:26:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-04 09:26:44]

Description was changed from

==========
Do not show a tooltip same as the previous one.

If a tooltip is closed by a keyboard event and the mouse cursor was
not moved, the same tooltip was shown again after some delay only on
Windows.  Blink receives native mousemove events repeatedly on Windows
even if the mouse cursor is not moved.

ChromeClient saves the last tooltip string and the last mouse position, and
do not call virtual setToolTip() if they are not changed. This CL not only
fixes the bug, but also reduces unnecessary IPC messages for tooltip.

BUG=557660
TEST=automated
==========

to

==========
Do not show a tooltip same as the previous one.

If a tooltip is closed by a keyboard event and the mouse cursor was
not moved, the same tooltip was shown again after some delay only on
Windows.  Blink receives native mousemove events repeatedly on Windows
even if the mouse cursor is not moved.

ChromeClient saves the last tooltip string and the last mouse position, and
do not call virtual setToolTip() if they are not changed. This CL not only
fixes the bug, but also reduces unnecessary IPC messages for tooltip.

BUG=557660
TEST=automated

Committed: https://crrev.com/91db68e8cb2080d1e9e07858efea3f7e198112a9
Cr-Commit-Position: refs/heads/master@{#363174}
==========

[commit-bot: I haz the power, 2015-12-04 09:26:44]

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