Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(286)

Issue 8899011: Virtual Me2Me: Redirect stdin of Xsession to /dev/null. (Closed)

Created:
9 years ago by Lambros
Modified:
9 years ago
Reviewers:
Jamie
CC:
chromium-reviews, jamiewalch+watch_chromium.org, hclam+watch_chromium.org, simonmorris+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, pam+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Virtual Me2Me: Redirect stdin of Xsession to /dev/null. This fixes the problem of the Me2Me process being stopped when running as a shell background job. BUG=106935 TEST=Follow repro steps Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114238

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M remoting/tools/me2me_virtual_host.py View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Lambros
9 years ago (2011-12-09 22:31:21 UTC) #1
Jamie
LGTM, but do we know what's reading from stdin and why? How is it going ...
9 years ago (2011-12-09 22:33:39 UTC) #2
Lambros
9 years ago (2011-12-13 18:37:24 UTC) #3
On 2011/12/09 22:33:39, Jamie wrote:
> LGTM, but do we know what's reading from stdin and why? How is it going to
> behave if we redirect it?
Can't tell for sure, but looks like it might be /usr/bin/x-session-manager
(which symlinks to gnome-session).  And the system's gnome-session has its stdin
pointing at /dev/null :)

Powered by Google App Engine
This is Rietveld 408576698