auto merge of #4070 : kparaju/servo/master-mime-sniffer-failing-4046, r=jdm

Fixes #4046

I tested it by:

    ./mach test-wpt --include=XMLHttpRequest > old.log
    # make code changes
    ./mach test-wpt --include=XMLHttpRequest > new.log
    grep panic old.log # SnifferManager panics
    grep panic new.log # No SnifferManager panics
This commit is contained in:
bors-servo 2014-11-23 22:24:34 -07:00
commit 65350bb225

View file

@ -31,10 +31,15 @@ impl SnifferManager {
}
impl SnifferManager {
fn start(&self, next_rx: Sender<LoadResponse>) {
fn start(self, next_rx: Sender<LoadResponse>) {
loop {
match self.data_receiver.try_recv() {
Ok(snif_data) => next_rx.send(snif_data),
Ok(snif_data) => {
let result = next_rx.send_opt(snif_data);
if result.is_err() {
break;
}
}
Err(Disconnected) => break,
Err(_) => (),
}