Ozone documentation: Take into account recent changes in Linux Desktop

Ozone documentation: Take into account recent changes in Linux Desktop

BUG=295089
Committed: https://crrev.com/15744c760b64d09988003601596ed16bad40df6f
Cr-Commit-Position: refs/heads/master@{#430599}

[fwang, 2016-11-08 10:13:20]

fwang@igalia.com changed reviewers:
+ tonikitoo@igalia.com

[fwang, 2016-11-08 10:13:20]

PTAL

[fwang, 2016-11-08 10:16:02]

Description was changed from

==========
Ozone documentation: Take into account recent changes in Linux Desktop

BUG=295089
==========

to

==========
Ozone documentation: Take into account recent changes in Linux Desktop

BUG=295089
==========

[tonikitoo, 2016-11-08 13:09:22]

lgtm with minor changes.

https://codereview.chromium.org/2490533002/diff/1/docs/ozone_overview.md
File docs/ozone_overview.md (right):

https://codereview.chromium.org/2490533002/diff/1/docs/ozone_overview.md#newc...
docs/ozone_overview.md:169: * `content_shell`
lets omit content_shell for now, since it is build-able but non-functional at
this point.

https://codereview.chromium.org/2490533002/diff/1/docs/ozone_overview.md#newc...
docs/ozone_overview.md:275: --disable-setuid-sandbox
can we launch with it, if we build the 'chrome_sandbox' target?

[fwang, 2016-11-08 13:12:18]

https://codereview.chromium.org/2490533002/diff/1/docs/ozone_overview.md
File docs/ozone_overview.md (right):

https://codereview.chromium.org/2490533002/diff/1/docs/ozone_overview.md#newc...
docs/ozone_overview.md:169: * `content_shell`
On 2016/11/08 13:09:21, tonikitoo wrote:
> lets omit content_shell for now, since it is build-able but non-functional at
> this point.

OK

https://codereview.chromium.org/2490533002/diff/1/docs/ozone_overview.md#newc...
docs/ozone_overview.md:275: --disable-setuid-sandbox
On 2016/11/08 13:09:21, tonikitoo wrote:
> can we launch with it, if we build the 'chrome_sandbox' target?

I guess so. But I think the sandbox and all these --disable-setuid-sandbox
option changes should be handled in a separate CL.

[tonikitoo, 2016-11-08 13:41:52]

The CQ bit was checked by tonikitoo@igalia.com

[tonikitoo, 2016-11-08 13:41:52]

lgtm

[commit-bot: I haz the power, 2016-11-08 13:42:03]

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, 2016-11-08 13:59:01]

Description was changed from

==========
Ozone documentation: Take into account recent changes in Linux Desktop

BUG=295089
==========

to

==========
Ozone documentation: Take into account recent changes in Linux Desktop

BUG=295089
==========

[commit-bot: I haz the power, 2016-11-08 13:59:01]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-08 14:02:47]

Description was changed from

==========
Ozone documentation: Take into account recent changes in Linux Desktop

BUG=295089
==========

to

==========
Ozone documentation: Take into account recent changes in Linux Desktop

BUG=295089

Committed: https://crrev.com/15744c760b64d09988003601596ed16bad40df6f
Cr-Commit-Position: refs/heads/master@{#430599}
==========

[commit-bot: I haz the power, 2016-11-08 14:02:48]

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