From 8891587901f41c850a29f3bfa3453bc80880b694 Mon Sep 17 00:00:00 2001
From: Keegan McAllister <kmcallister@mozilla.com>
Date: Wed, 11 Dec 2013 15:16:01 -0800
Subject: [PATCH] contenttest: Fail if Servo exits with non-zero status

Remove "let stdout" to make the borrow checker happy.
---
 src/test/harness/contenttest/contenttest.rs | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/test/harness/contenttest/contenttest.rs b/src/test/harness/contenttest/contenttest.rs
index bee7e175f4f..e7548b24610 100644
--- a/src/test/harness/contenttest/contenttest.rs
+++ b/src/test/harness/contenttest/contenttest.rs
@@ -100,10 +100,9 @@ fn run_test(file: ~str) {
     };
 
     let mut prc = Process::new(config).unwrap();
-    let stdout = prc.io[1].get_mut_ref();
     let mut output = ~[];
     loop {
-        let byte = stdout.read_byte();
+        let byte = prc.io[1].get_mut_ref().read_byte();
         match byte {
             Some(byte) => {
                 print!("{}", byte as char);
@@ -120,4 +119,9 @@ fn run_test(file: ~str) {
             fail!(line);
         }
     }
+
+    let retval = prc.wait();
+    if retval != 0 {
+        fail!("Servo exited with non-zero status {}", retval);
+    }
 }