Chromecast: disables wifi-based features on ATV.

Chromecast: disables wifi-based features on ATV.

R=lcwu@chromium.org,byungchul@chromium.org
BUG=internal b/18934906

Committed: https://crrev.com/dc4666755b0408c2ec6eba26211a77ca502ff9e6
Cr-Commit-Position: refs/heads/master@{#317683}

[lcwu1, 2015-02-23 21:48:50]

lgtm % a suggestion in additional comment

https://codereview.chromium.org/947993003/diff/1/chromecast/browser/cast_brow...
File chromecast/browser/cast_browser_main_parts.cc (right):

https://codereview.chromium.org/947993003/diff/1/chromecast/browser/cast_brow...
chromecast/browser/cast_browser_main_parts.cc:104: { switches::kNoWifi, "" },
It might make it clear to add a comment saying the NoWifi switch here is not to
diable WiFi on Android, but to disable Chromecast-specific Wifi support.

[gunsch, 2015-02-23 22:00:33]

https://codereview.chromium.org/947993003/diff/1/chromecast/browser/cast_brow...
File chromecast/browser/cast_browser_main_parts.cc (right):

https://codereview.chromium.org/947993003/diff/1/chromecast/browser/cast_brow...
chromecast/browser/cast_browser_main_parts.cc:104: { switches::kNoWifi, "" },
On 2015/02/23 21:48:50, lcwu1 wrote:
> It might make it clear to add a comment saying the NoWifi switch here is not
to
> diable WiFi on Android, but to disable Chromecast-specific Wifi support.

Done.

https://codereview.chromium.org/947993003/diff/1/chromecast/browser/cast_brow...
chromecast/browser/cast_browser_main_parts.cc:104: { switches::kNoWifi, "" },
On 2015/02/23 21:48:50, lcwu1 wrote:
> It might make it clear to add a comment saying the NoWifi switch here is not
to
> diable WiFi on Android, but to disable Chromecast-specific Wifi support.

Done.

[gunsch, 2015-02-23 22:00:39]

The CQ bit was checked by gunsch@chromium.org

[gunsch, 2015-02-23 22:00:41]

The CQ bit was unchecked by gunsch@chromium.org

[gunsch, 2015-02-23 22:00:48]

The CQ bit was checked by gunsch@chromium.org

[gunsch, 2015-02-23 22:00:48]

The patchset sent to the CQ was uploaded after l-g-t-m from lcwu@chromium.org
Link to the patchset: https://codereview.chromium.org/947993003/#ps20001 (title:
"adds comment")

[commit-bot: I haz the power, 2015-02-23 22:01:47]

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

[commit-bot: I haz the power, 2015-02-24 00:08:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-24 00:08:49]

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