Fix flag comparison in event_param_traits_unittest

Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense. The warning was:
  event_param_traits_unittest.cc(90): warning C4806: '&': unsafe operation:
  no value of type 'bool' promoted to type 'ui::MouseEventFlags' can equal
  the given constant

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses. This fix removes the !!.

Bug was introduced in crrev.com/1695783002, but only triggers on VS 2015
gn builds.

BUG=440500, 582658
Committed: https://crrev.com/48843ccd23afd0147f1484c57bd3fab73e482427
Cr-Commit-Position: refs/heads/master@{#380008}

[brucedawson, 2016-03-08 21:42:36]

Description was changed from

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense.

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses.

BUG=440500
==========

to

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense.

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses.

Bug was introduced in crrev.com/1695783002

BUG=440500
==========

[brucedawson, 2016-03-08 21:44:18]

Description was changed from

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense.

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses.

Bug was introduced in crrev.com/1695783002

BUG=440500
==========

to

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense. The warning was:
  event_param_traits_unittest.cc(90): warning C4806: '&': unsafe operation:
  no value of type 'bool' promoted to type 'ui::MouseEventFlags' can equal
  the given constant

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses.

Bug was introduced in crrev.com/1695783002

BUG=440500
==========

[brucedawson, 2016-03-08 21:44:35]

brucedawson@chromium.org changed reviewers:
+ sky@chromium.org

[brucedawson, 2016-03-08 21:44:36]

PTAL

[brucedawson, 2016-03-08 22:08:13]

Description was changed from

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense. The warning was:
  event_param_traits_unittest.cc(90): warning C4806: '&': unsafe operation:
  no value of type 'bool' promoted to type 'ui::MouseEventFlags' can equal
  the given constant

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses.

Bug was introduced in crrev.com/1695783002

BUG=440500
==========

to

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense. The warning was:
  event_param_traits_unittest.cc(90): warning C4806: '&': unsafe operation:
  no value of type 'bool' promoted to type 'ui::MouseEventFlags' can equal
  the given constant

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses. This fix removes the !!.

Bug was introduced in crrev.com/1695783002, but only triggers on VS 2015
gn builds.

BUG=440500,582658
==========

[sky, 2016-03-08 22:56:17]

LGTM

[brucedawson, 2016-03-09 00:05:05]

The CQ bit was checked by brucedawson@chromium.org

[commit-bot: I haz the power, 2016-03-09 00:05:31]

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

[commit-bot: I haz the power, 2016-03-09 01:09:10]

Description was changed from

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense. The warning was:
  event_param_traits_unittest.cc(90): warning C4806: '&': unsafe operation:
  no value of type 'bool' promoted to type 'ui::MouseEventFlags' can equal
  the given constant

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses. This fix removes the !!.

Bug was introduced in crrev.com/1695783002, but only triggers on VS 2015
gn builds.

BUG=440500,582658
==========

to

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense. The warning was:
  event_param_traits_unittest.cc(90): warning C4806: '&': unsafe operation:
  no value of type 'bool' promoted to type 'ui::MouseEventFlags' can equal
  the given constant

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses. This fix removes the !!.

Bug was introduced in crrev.com/1695783002, but only triggers on VS 2015
gn builds.

BUG=440500,582658
==========

[commit-bot: I haz the power, 2016-03-09 01:09:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-09 01:10:37]

Description was changed from

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense. The warning was:
  event_param_traits_unittest.cc(90): warning C4806: '&': unsafe operation:
  no value of type 'bool' promoted to type 'ui::MouseEventFlags' can equal
  the given constant

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses. This fix removes the !!.

Bug was introduced in crrev.com/1695783002, but only triggers on VS 2015
gn builds.

BUG=440500,582658
==========

to

==========
Fix flag comparison in event_param_traits_unittest

The result of flags() was converted to a boolean and then masked against
an enum. VS 2015 complained that this makes no sense. The warning was:
  event_param_traits_unittest.cc(90): warning C4806: '&': unsafe operation:
  no value of type 'bool' promoted to type 'ui::MouseEventFlags' can equal
  the given constant

Presumably the intent is to make sure the flags match. This can either
be done by removing the !! or by wrapping the '&' expression in
parentheses. This fix removes the !!.

Bug was introduced in crrev.com/1695783002, but only triggers on VS 2015
gn builds.

BUG=440500,582658

Committed: https://crrev.com/48843ccd23afd0147f1484c57bd3fab73e482427
Cr-Commit-Position: refs/heads/master@{#380008}
==========

[commit-bot: I haz the power, 2016-03-09 01:10:38]

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