Use ContainsValue() instead of std::find() in ui/android, ui/base and ui/views

Use ContainsValue() instead of std::find() in ui/android, ui/base and ui/views

BUG=561800
Review-Url: https://codereview.chromium.org/2930573005
Cr-Commit-Position: refs/heads/master@{#478558}
Committed: https://chromium.googlesource.com/chromium/src/+/e43c9f5c05c4d6a929907441388882a8f1795b46

[Tripta, 2017-06-08 11:40:29]

Description was changed from

==========
Use ContainsValue() instead of std::find() in ui/android, ui/base and ui/views

BUG=561800
==========

to

==========
Use ContainsValue() instead of std::find() in ui/android, ui/base and ui/views

BUG=561800
==========

[Tripta, 2017-06-08 11:40:29]

tripta.g@samsung.com changed reviewers:
+ a.suchit@samsung.com

[Tripta, 2017-06-08 11:40:46]

Please review the code changes.

[Tripta, 2017-06-09 10:16:30]

tripta.g@samsung.com changed reviewers:
+ sadrul@chromium.org, sadrul@google.com

[Tripta, 2017-06-09 10:16:52]

Please review the code changes.

Regards
Tripta

[sadrul, 2017-06-12 02:05:25]

lgtm

https://codereview.chromium.org/2930573005/diff/20001/ui/base/ime/chromeos/ch...
File ui/base/ime/chromeos/character_composer_unittest.cc (right):

https://codereview.chromium.org/2930573005/diff/20001/ui/base/ime/chromeos/ch...
ui/base/ime/chromeos/character_composer_unittest.cc:253:
EXPECT_FALSE(!base::ContainsValue(subtrees, value)) << index;
EXPECT_TRUE(base::ContainsValue()) instead?

[Tripta, 2017-06-12 05:04:58]

The CQ bit was checked by tripta.g@samsung.com

[Tripta, 2017-06-12 05:04:58]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2930573005/#ps40001
(title: "Changed EXPECT_FALSE to EXPECT_TRUE")

[commit-bot: I haz the power, 2017-06-12 05:05:14]

CQ is trying da patch.

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

[Tripta, 2017-06-12 05:07:51]

Have changed EXPECT_FALSE to EXPECT_TRUE.

Thanks 
Tripta

https://codereview.chromium.org/2930573005/diff/20001/ui/base/ime/chromeos/ch...
File ui/base/ime/chromeos/character_composer_unittest.cc (right):

https://codereview.chromium.org/2930573005/diff/20001/ui/base/ime/chromeos/ch...
ui/base/ime/chromeos/character_composer_unittest.cc:253:
EXPECT_FALSE(!base::ContainsValue(subtrees, value)) << index;
On 2017/06/12 02:05:25, sadrul wrote:
> EXPECT_TRUE(base::ContainsValue()) instead?

Done.

[commit-bot: I haz the power, 2017-06-12 06:17:52]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1497243898400340,
"parent_rev": "a40d8abdbd69477dde5efab20084b34e3c6f9e64", "commit_rev":
"e43c9f5c05c4d6a929907441388882a8f1795b46"}

[commit-bot: I haz the power, 2017-06-12 06:18:00]

Description was changed from

==========
Use ContainsValue() instead of std::find() in ui/android, ui/base and ui/views

BUG=561800
==========

to

==========
Use ContainsValue() instead of std::find() in ui/android, ui/base and ui/views

BUG=561800

Review-Url: https://codereview.chromium.org/2930573005
Cr-Commit-Position: refs/heads/master@{#478558}
Committed:
https://chromium.googlesource.com/chromium/src/+/e43c9f5c05c4d6a9299074413888...
==========

[commit-bot: I haz the power, 2017-06-12 06:18:00]

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