Adding unit test to check for incorrect mouse enter/exit event handling

Adding unit test to check for incorrect mouse enter/exit event handling


BUG=142355
TEST=Unit test


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=151925

[Mr4D (OOO till 08-26), 2012-08-15 16:13:00]

As promised - the unit test for the missing events. This CL will go in after the
fix is checked in.
Note: I have tested that it does what it should in moth cases.

[sky, 2012-08-15 16:55:54]

The test should be as close to the fix as possible. You should add it some where
in views. Most likely widget_unittest.

[Mr4D (OOO till 08-26), 2012-08-15 23:26:20]

Rewritten the test to your comment.

Please look again.

[sky, 2012-08-16 00:38:15]

LGTM with the following change.

http://codereview.chromium.org/10824309/diff/1002/ui/views/widget/widget_unit...
File ui/views/widget/widget_unittest.cc (right):

http://codereview.chromium.org/10824309/diff/1002/ui/views/widget/widget_unit...
ui/views/widget/widget_unittest.cc:94: int CheckEnteredCalls() {
Can you remove Check from the name of these methods since they don't actually
check anything.

[Mr4D (OOO till 08-26), 2012-08-16 16:04:17]

Done and submitted. Thanks!

http://codereview.chromium.org/10824309/diff/1002/ui/views/widget/widget_unit...
File ui/views/widget/widget_unittest.cc (right):

http://codereview.chromium.org/10824309/diff/1002/ui/views/widget/widget_unit...
ui/views/widget/widget_unittest.cc:94: int CheckEnteredCalls() {
On 2012/08/16 00:38:15, sky wrote:
> Can you remove Check from the name of these methods since they don't actually
> check anything.

Done.

[commit-bot: I haz the power, 2012-08-16 16:04:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skuhne@chromium.org/10824309/6002

[commit-bot: I haz the power, 2012-08-16 18:32:35]

Change committed as 151925