commit | a504ba7dfbcbc6b1d726cead0020aec7d526ba99 | [log] [tgz] |
---|---|---|
author | [email protected] <[email protected]@0039d316-1c4b-4281-b951-d872f2087c98> | Wed Jan 25 01:42:57 2012 |
committer | [email protected] <[email protected]@0039d316-1c4b-4281-b951-d872f2087c98> | Wed Jan 25 01:42:57 2012 |
tree | 4c739837311972aa4162898bf4c0d12baef618a2 | |
parent | e28d1f29805fcc1df5113c02068fea705e171c30 [diff] |
[ChromeOS-HTerm] Use pseudo-terminal instead of pipes to communicate with crosh process. Some applications we run in crosh expect their stdin/stdout to be terminal, and malfunction if we use pipes. One of results of this change is we stopped distinguishing between stdout and stderr (both streams are detected through same file descriptor). I still have to update extension API to reflect this. Also, added support for detecting when crosh process exits. This depends on https://chromiumcodereview.appspot.com/9150051/ and https://chromiumcodereview.appspot.com/9176013/ (and I tested it with those two in place). TEST=ExtensionTerminalPrivateApiTest.*, ProcessProxy.*, git try BUG=chromium-os:23273 Review URL: https://chromiumcodereview.appspot.com/9195039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118975 0039d316-1c4b-4281-b951-d872f2087c98