Failing to parse a URL in the UWP app causes a panic

[jdm]

Try to load an invalid URL in the UWP app and watch it panic. We should pop up an alert dialog or something, but it should really be ignored rather than bring down the browser.

[paulrouget]

I can't reproduce. An invalid URL should load duckduckgo or show an error message.
Can you share an example of an invalid URL?
Do you just type it in the url bar?

[jdm]

Hmm. I know that a panic was caught in

servo/ports/libsimpleservo/capi/src/lib.rs

Lines 434 to 439 in d671010

	catch_any_panic(|| {
	debug!("load_url");
	let url = unsafe { CStr::from_ptr(url) };
	let url = url.to_str().expect("Can't read string");
	call(|s| s.load_uri(url));
	});

when I tried to paste a URL. I assumed what had happened was that I ended up with something like https://mrdoob.neocities.org/018/https://mrdoob.neocities.org/019/, but I didn't see the actual URL that I tried to load.

[paulrouget]

Right. It's better if this would not panic. But I don't see how we could end up with a non-valid URL there.

I'll investigate, and at least handle invalid URL better.