Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the spurious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631, 240300, 352106, 386896, 569998, 575208
Review-Url: https://codereview.chromium.org/2618063002
Cr-Commit-Position: refs/heads/master@{#442076}
Committed: https://chromium.googlesource.com/chromium/src/+/3963e231e4c43187a05f3c6420f5df997d734ea0

[dtapuska, 2017-01-06 18:43:40]

Description was changed from

==========
Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the suprious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631
==========

to

==========
Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the suprious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631
==========

[dtapuska, 2017-01-06 18:43:40]

dtapuska@chromium.org changed reviewers:
+ rbyers@chromium.org

[dtapuska, 2017-01-06 18:43:48]

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

[commit-bot: I haz the power, 2017-01-06 18:44:07]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Rick Byers, 2017-01-06 18:58:48]

sweet, hope it works out well.  LGTM

[dtapuska, 2017-01-06 19:01:44]

Description was changed from

==========
Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the suprious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631
==========

to

==========
Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the suprious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631,240300,352106,386896,569998,575208
==========

[dtapuska, 2017-01-06 20:20:39]

The CQ bit was unchecked by dtapuska@chromium.org

[dtapuska, 2017-01-06 20:20:45]

The CQ bit was checked by dtapuska@chromium.org

[commit-bot: I haz the power, 2017-01-06 20:21:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dtapuska, 2017-01-06 20:55:42]

dtapuska@chromium.org changed reviewers:
+ tdresser@chromium.org

[dtapuska, 2017-01-06 20:55:42]

Tim, can you look at this I had to modify the check on linux as it seemed to be
generating a mouse leave for context menus.

[tdresser, 2017-01-06 21:39:15]

Nit: suprious -> spurious in the CL description.

LGTM

https://codereview.chromium.org/2618063002/diff/20001/ui/events/blink/blink_f...
File ui/events/blink/blink_features.h (right):

https://codereview.chromium.org/2618063002/diff/20001/ui/events/blink/blink_f...
ui/events/blink/blink_features.h:16: // converted to mouse move events due to a
number of incosistencies on
Might as well fix "inconsistencies" while we're here.

[dtapuska, 2017-01-06 21:47:06]

Description was changed from

==========
Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the suprious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631,240300,352106,386896,569998,575208
==========

to

==========
Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the spurious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631,240300,352106,386896,569998,575208
==========

[dtapuska, 2017-01-06 21:47:29]

https://codereview.chromium.org/2618063002/diff/20001/ui/events/blink/blink_f...
File ui/events/blink/blink_features.h (right):

https://codereview.chromium.org/2618063002/diff/20001/ui/events/blink/blink_f...
ui/events/blink/blink_features.h:16: // converted to mouse move events due to a
number of incosistencies on
On 2017/01/06 21:39:15, tdresser wrote:
> Might as well fix "inconsistencies" while we're here.

Done.

[dtapuska, 2017-01-06 21:47:31]

The CQ bit was checked by dtapuska@chromium.org

[dtapuska, 2017-01-06 21:47:32]

The patchset sent to the CQ was uploaded after l-g-t-m from rbyers@chromium.org,
tdresser@chromium.org
Link to the patchset: https://codereview.chromium.org/2618063002/#ps40001
(title: "Fix typo")

[commit-bot: I haz the power, 2017-01-06 21:47:55]

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, 2017-01-06 22:38:51]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483739251968010,
"parent_rev": "7223079dda8d234c24aed43d249bf27ece1e30ba", "commit_rev":
"3963e231e4c43187a05f3c6420f5df997d734ea0"}

[commit-bot: I haz the power, 2017-01-06 22:39:22]

Description was changed from

==========
Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the spurious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631,240300,352106,386896,569998,575208
==========

to

==========
Send Mouse Leave events all the time when we get a ET_MOUSE_EXITED

Once we have shipped this for a release this feature flag can be removed.
I've tested Windows and the issues causing the spurious ET_MOUSE_EXITED
don't exist anymore so I believe this is safe to ship in this fashion.

BUG=450631,240300,352106,386896,569998,575208

Review-Url: https://codereview.chromium.org/2618063002
Cr-Commit-Position: refs/heads/master@{#442076}
Committed:
https://chromium.googlesource.com/chromium/src/+/3963e231e4c43187a05f3c6420f5...
==========

[commit-bot: I haz the power, 2017-01-06 22:39:23]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/3963e231e4c43187a05f3c6420f5...