Use ContainsValue() instead of std::find() in chromecast/

Use ContainsValue() instead of std::find() in chromecast/

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

[Tripta, 2017-06-01 07:56:36]

Description was changed from

==========
Use ContainsValue() instead of std::find() in chromecast/

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

to

==========
Use ContainsValue() instead of std::find() in chromecast/

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

[Tripta, 2017-06-01 07:56:36]

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

[Tripta, 2017-06-01 07:56:55]

Please review the code changes.

[Tripta, 2017-06-01 07:57:02]

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

[commit-bot: I haz the power, 2017-06-01 07:57:15]

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-06-01 07:57:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-01 07:57:16]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[a.suchit2, 2017-06-01 09:10:54]

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

[commit-bot: I haz the power, 2017-06-01 09:11:18]

Dry run: 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-06-01 09:27:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-01 09:27:19]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Tripta, 2017-06-01 10:43:33]

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

[commit-bot: I haz the power, 2017-06-01 10:43:44]

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-06-01 10:43:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-01 10:43:46]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[Tripta, 2017-06-01 11:54:28]

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

[commit-bot: I haz the power, 2017-06-01 11:54:40]

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-06-01 11:54:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-01 11:54:41]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[Tripta, 2017-06-01 12:03:30]

tripta.g@samsung.com changed reviewers:
+ alokp@chromium.org, dalecurtis@chromium.org, sky@chromium.org

[Tripta, 2017-06-01 12:03:58]

Please review code changes.

[sky, 2017-06-01 18:55:22]

sky@chromium.org changed reviewers:
- sky@chromium.org

[sky, 2017-06-01 18:55:22]

When adding more than one reviewer please indicate who you expect to review
what. Also, it looks like you have local OWNERs here and don't need me, so I'm
removing myself.

[alokp, 2017-06-01 19:01:21]

https://codereview.chromium.org/2920723002/diff/20001/chromecast/graphics/cas...
File chromecast/graphics/cast_focus_client_aura.cc (right):

https://codereview.chromium.org/2920723002/diff/20001/chromecast/graphics/cas...
chromecast/graphics/cast_focus_client_aura.cc:5: #include "base/stl_util.h"
base/stl_util.h should be included after cast_focus_client_aura.h

[Tripta, 2017-06-02 04:40:22]

Have added base/stl_util.h after cast_focus_client_aura.h

https://codereview.chromium.org/2920723002/diff/20001/chromecast/graphics/cas...
File chromecast/graphics/cast_focus_client_aura.cc (right):

https://codereview.chromium.org/2920723002/diff/20001/chromecast/graphics/cas...
chromecast/graphics/cast_focus_client_aura.cc:5: #include "base/stl_util.h"
On 2017/06/01 19:01:21, alokp wrote:
> base/stl_util.h should be included after cast_focus_client_aura.h

Have made the changes.

[alokp, 2017-06-02 04:59:35]

please run git-cl format before uploading the patch.

https://codereview.chromium.org/2920723002/diff/40001/chromecast/graphics/cas...
File chromecast/graphics/cast_focus_client_aura.cc (right):

https://codereview.chromium.org/2920723002/diff/40001/chromecast/graphics/cas...
chromecast/graphics/cast_focus_client_aura.cc:6: #include "base/stl_util.h"
add a line here.

https://codereview.chromium.org/2920723002/diff/40001/chromecast/graphics/cas...
chromecast/graphics/cast_focus_client_aura.cc:7: 
remove this line

[Tripta, 2017-06-02 05:33:31]

Patchset #4 (id:60001) has been deleted

[Tripta, 2017-06-02 08:08:02]

Have made the changes and have run "git cl format" this time.

https://codereview.chromium.org/2920723002/diff/40001/chromecast/graphics/cas...
File chromecast/graphics/cast_focus_client_aura.cc (right):

https://codereview.chromium.org/2920723002/diff/40001/chromecast/graphics/cas...
chromecast/graphics/cast_focus_client_aura.cc:6: #include "base/stl_util.h"
On 2017/06/02 04:59:35, alokp wrote:
> add a line here.

Have made the changes. Please check.

https://codereview.chromium.org/2920723002/diff/40001/chromecast/graphics/cas...
chromecast/graphics/cast_focus_client_aura.cc:7: 
On 2017/06/02 04:59:35, alokp wrote:
> remove this line

Have made the changes. Please check.

[Tripta, 2017-06-02 08:34:26]

Patchset #4 (id:80001) has been deleted

[Tripta, 2017-06-02 09:48:54]

Some issue with my git. The patch is uploaded now.

[DaleCurtis, 2017-06-02 18:59:37]

dalecurtis@chromium.org changed reviewers:
- dalecurtis@chromium.org

[alokp, 2017-06-03 04:03:48]

lgtm

[a.suchit2, 2017-06-05 02:55:12]

The CQ bit was checked by a.suchit@chromium.org

[commit-bot: I haz the power, 2017-06-05 02:55:21]

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-06-05 03:10:02]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1496631312411920,
"parent_rev": "3300fca96e914bae525ab150e35eb946a5292821", "commit_rev":
"a7ee6cb0800d2e560eeb4b912c6d8f21ef1987ae"}

[commit-bot: I haz the power, 2017-06-05 03:10:11]

Description was changed from

==========
Use ContainsValue() instead of std::find() in chromecast/

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

to

==========
Use ContainsValue() instead of std::find() in chromecast/

BUG=561800

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

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

Committed patchset #4 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/a7ee6cb0800d2e560eeb4b912c6d...