| commit | 3b770287cdb2c4058101476e2e9e0f4922afc51a | [log] [tgz] |
|---|---|---|
| author | Brett Chabot <brettchabot@google.com> | Tue Nov 12 17:39:15 2013 -0800 |
| committer | Brett Chabot <brettchabot@google.com> | Tue Nov 12 17:44:12 2013 -0800 |
| tree | 7f6f9841d6736948705f7b92bea9201a4cc7088e | |
| parent | 8515ffa5c6349c4514aff3fecf31b4eaba117bc5 [diff] |
Fix periodic 'aapt dump badging' failures. Root cause was a race condition in RunUtil. If process finished quickly, there was a chance that stdout wouldn't be read, since it is read in a background thread. This CL adds a join() to wait for the stdout/stderr threads to finish. Also clean up the unnecessary double-buffering of stdout and add tests Bug 11172308 Change-Id: Ia206a448bcb57440b244c58699d22e9d7b089aa4