Allow TV turn on without switching inputs

Allow TV turn on without switching inputs

BUG= internal b/35766299
TEST= unittests, on device

Review-Url: https://codereview.chromium.org/2859733002
Cr-Commit-Position: refs/heads/master@{#468878}
Committed: https://chromium.googlesource.com/chromium/src/+/a1bd664cf7bd902f632c6359e63cb06131f78bad

[almasrymina, 2017-05-03 00:44:16]

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

[commit-bot: I haz the power, 2017-05-03 00:45:03]

Dry run: CQ is trying da patch.

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

[almasrymina, 2017-05-03 01:07:08]

Description was changed from

==========
Allow TV turn on without switching inputs

BUG= internal b/35766299
TEST= unittests, on device
==========

to

==========
Allow TV turn on without switching inputs

BUG= internal b/35766299
TEST= unittests, on device
==========

[almasrymina, 2017-05-03 01:07:08]

almasrymina@chromium.org changed reviewers:
+ gfhuang@chromium.org, halliwell@chromium.org, jasonroberts@google.com,
wzhong@chromium.org

[gfhuang, 2017-05-03 01:11:03]

nit

https://codereview.chromium.org/2859733002/diff/40001/chromecast/public/avset...
File chromecast/public/avsettings.h (right):

https://codereview.chromium.org/2859733002/diff/40001/chromecast/public/avset...
chromecast/public/avsettings.h:171: virtual bool TurnActive(bool
also_switch_inputs) = 0;
"switch_to_cast" is better name.

[almasrymina, 2017-05-03 01:14:54]

Done. PTAL.

[gfhuang, 2017-05-03 01:15:35]

lgtm

[halliwell, 2017-05-03 01:16:40]

On 2017/05/03 01:15:35, gfhuang wrote:
> lgtm

lgtm

[almasrymina, 2017-05-03 01:17:46]

The CQ bit was checked by almasrymina@chromium.org

[commit-bot: I haz the power, 2017-05-03 01:18:39]

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-05-03 01:18:40]

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

[commit-bot: I haz the power, 2017-05-03 01:18:41]

CLs for remote refs other than refs/heads/master must contain NOTRY=true and
NOPRESUBMIT=true in order for the CQ to process them

[almasrymina, 2017-05-03 02:44:00]

The CQ bit was checked by almasrymina@chromium.org

[almasrymina, 2017-05-03 02:44:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
gfhuang@chromium.org, halliwell@chromium.org
Link to the patchset: https://codereview.chromium.org/2859733002/#ps120001
(title: "Fix target ref")

[commit-bot: I haz the power, 2017-05-03 02:44:20]

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-05-03 02:59:24]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1493779440824960,
"parent_rev": "c9c0d96118b8381d85b0f311c4d2497d5b5373dd", "commit_rev":
"a1bd664cf7bd902f632c6359e63cb06131f78bad"}

[commit-bot: I haz the power, 2017-05-03 02:59:30]

Description was changed from

==========
Allow TV turn on without switching inputs

BUG= internal b/35766299
TEST= unittests, on device
==========

to

==========
Allow TV turn on without switching inputs

BUG= internal b/35766299
TEST= unittests, on device

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

[commit-bot: I haz the power, 2017-05-03 02:59:33]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/chromium/src/+/a1bd664cf7bd902f632c6359e63c...