Auto merge of #5527 - jagtalon:jag/multiple-urls, r=mbrubeck

Switched from opts.urls from being of type Vec to type String and changing the name to `url` as well. Changed the other files that are using opts.urls accordingly.
This commit is contained in:
bors-servo 2015-04-07 22:08:33 -05:00
commit df57af1fc4
5 changed files with 28 additions and 38 deletions

View file

@ -119,23 +119,20 @@ impl Browser {
// Send the URL command to the constellation.
let cwd = env::current_dir().unwrap();
for url in opts.urls.iter() {
let url = match url::Url::parse(&*url) {
Ok(url) => url,
Err(url::ParseError::RelativeUrlWithoutBase)
=> url::Url::from_file_path(&*cwd.join(&*url)).unwrap(),
Err(_) => panic!("URL parsing failed"),
};
let url = match url::Url::parse(&opts.url) {
Ok(url) => url,
Err(url::ParseError::RelativeUrlWithoutBase)
=> url::Url::from_file_path(&*cwd.join(&opts.url)).unwrap(),
Err(_) => panic!("URL parsing failed"),
};
let ConstellationChan(ref chan) = constellation_chan;
chan.send(ConstellationMsg::InitLoadUrl(url)).unwrap();
}
let ConstellationChan(ref chan) = constellation_chan;
chan.send(ConstellationMsg::InitLoadUrl(url)).unwrap();
debug!("preparing to enter main loop");
let compositor = CompositorTask::create(window,
compositor_proxy,
compositor_receiver,
constellation_chan,
constellation_chan.clone(),
time_profiler_chan,
mem_profiler_chan);
@ -164,4 +161,3 @@ impl Browser {
self.compositor.shutdown();
}
}

View file

@ -22,8 +22,8 @@ use std::rt;
/// Global flags for Servo, currently set on the command line.
#[derive(Clone)]
pub struct Opts {
/// The initial URLs to load.
pub urls: Vec<String>,
/// The initial URL to load.
pub url: String,
/// How many threads to use for CPU painting (`-t`).
///
@ -183,7 +183,7 @@ static FORCE_CPU_PAINTING: bool = false;
pub fn default_opts() -> Opts {
Opts {
urls: vec!(),
url: String::new(),
paint_threads: 1,
gpu_painting: false,
tile_size: 512,
@ -277,12 +277,12 @@ pub fn from_cmdline_args(args: &[String]) -> bool {
return false;
}
let urls = if opt_match.free.is_empty() {
let url = if opt_match.free.is_empty() {
print_usage(app_name.as_slice(), opts.as_slice());
args_fail("servo asks that you provide 1 or more URLs");
args_fail("servo asks that you provide a URL");
return false;
} else {
opt_match.free.clone()
opt_match.free[0].clone()
};
let tile_size: usize = match opt_match.opt_str("s") {
@ -344,7 +344,7 @@ pub fn from_cmdline_args(args: &[String]) -> bool {
};
let opts = Opts {
urls: urls,
url: url,
paint_threads: paint_threads,
gpu_painting: gpu_painting,
tile_size: tile_size,