[Chromecast] ATV: don't include logs from renderer process in logfile.

[Chromecast] ATV: don't include logs from renderer process in logfile.

Higher level of restrictions in M turn these into SELinux denials.

R=halliwell@chromium.org,slan@chromium.org
BUG=internal b/22888548

Committed: https://crrev.com/3f39ec09100c898b5357b7a6230542229c8f5deb
Cr-Commit-Position: refs/heads/master@{#344848}

[halliwell, 2015-08-21 20:10:33]

On 2015/08/21 19:36:16, gunsch wrote:

lgtm

[gunsch, 2015-08-21 20:11:18]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2015-08-21 20:11:45]

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

[slan, 2015-08-21 20:12:14]

lgtm

https://codereview.chromium.org/1311493002/diff/1/chromecast/app/cast_main_de...
File chromecast/app/cast_main_delegate.cc (right):

https://codereview.chromium.org/1311493002/diff/1/chromecast/app/cast_main_de...
chromecast/app/cast_main_delegate.cc:70: if (process_type.empty()) {
Could we add a note about permissions explaining why rendering logs are not
captured? (or perhaps the bug explaining the logcat strategy to capture these
logs eventually?)

[gunsch, 2015-08-21 20:19:26]

On 2015/08/21 20:12:14, slan wrote:
> lgtm
> 
>
https://codereview.chromium.org/1311493002/diff/1/chromecast/app/cast_main_de...
> File chromecast/app/cast_main_delegate.cc (right):
> 
>
https://codereview.chromium.org/1311493002/diff/1/chromecast/app/cast_main_de...
> chromecast/app/cast_main_delegate.cc:70: if (process_type.empty()) {
> Could we add a note about permissions explaining why rendering logs are not
> captured? (or perhaps the bug explaining the logcat strategy to capture these
> logs eventually?)

I filed b/23424042 to track the future log/crash plans.

[commit-bot: I haz the power, 2015-08-21 20:20:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-08-21 20:21:10]

Patchset 1 (id:??) landed as
https://crrev.com/3f39ec09100c898b5357b7a6230542229c8f5deb
Cr-Commit-Position: refs/heads/master@{#344848}