Chromecast: device identification for whitelisted apps and servers.

Chromecast: device identification for whitelisted apps and servers.

R=dougsteed@chromium.org,lcwu@chromium.org
BUG=336640
Committed: https://crrev.com/b098afcdc9323e47baa8ad1f268f945a0b7120c3
Cr-Commit-Position: refs/heads/master@{#297898}

[gunsch, 2014-10-02 16:17:26]

ping?

[dougsteed, 2014-10-02 17:27:41]

lgtm

[gunsch, 2014-10-02 19:07:52]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2014-10-02 19:09:17]

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

[commit-bot: I haz the power, 2014-10-02 19:09:18]

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

[commit-bot: I haz the power, 2014-10-02 19:09:20]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[lcwu1, 2014-10-02 19:56:34]

LGTM % a nit in comment.

https://codereview.chromium.org/618863002/diff/1/chromecast/shell/browser/cas...
File chromecast/shell/browser/cast_content_browser_client.cc (right):

https://codereview.chromium.org/618863002/diff/1/chromecast/shell/browser/cas...
chromecast/shell/browser/cast_content_browser_client.cc:158: // it, because
CastNetworkDelegate is bound to the latter thread.
s/latter/IO (as there is no mention of a 'former' thread in the previous
sentences :-))

[gunsch, 2014-10-02 20:29:15]

The CQ bit was checked by gunsch@chromium.org

[gunsch, 2014-10-02 20:29:26]

https://codereview.chromium.org/618863002/diff/1/chromecast/shell/browser/cas...
File chromecast/shell/browser/cast_content_browser_client.cc (right):

https://codereview.chromium.org/618863002/diff/1/chromecast/shell/browser/cas...
chromecast/shell/browser/cast_content_browser_client.cc:158: // it, because
CastNetworkDelegate is bound to the latter thread.
On 2014/10/02 19:56:33, lcwu1 wrote:
> s/latter/IO (as there is no mention of a 'former' thread in the previous
> sentences :-))

Done.

[commit-bot: I haz the power, 2014-10-02 20:30:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/618863002/20001

[commit-bot: I haz the power, 2014-10-02 20:54:26]

Committed patchset #2 (id:20001) as 4bc25ae983d4808edcfcb06d5369db28a3ab253e

[commit-bot: I haz the power, 2014-10-02 20:54:51]

Patchset 2 (id:??) landed as
https://crrev.com/b098afcdc9323e47baa8ad1f268f945a0b7120c3
Cr-Commit-Position: refs/heads/master@{#297898}