Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-05 13:40:08 +01:00
Code Issues Projects Releases Packages Wiki Activity

Don't unnecessarily clone entire opts::Opt structure.

Browse source 
`opts::Opt` is a pretty big structure, so cloning everything is
excessive when we only need a few items.
This commit is contained in:
Corey Farwell 2016-11-11 21:31:44 -05:00
parent 3b2e3dcfb9
commit f0020297f5
1 changed files with 11 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

16
components/servo/lib.rs
View file

@ -87,9 +87,11 @@ use profile::time as profile_time;
use profile_traits::mem; use profile_traits::mem;
use profile_traits::time; use profile_traits::time;
use script_traits::{ConstellationMsg, SWManagerSenders, ScriptMsg}; use script_traits::{ConstellationMsg, SWManagerSenders, ScriptMsg};
use std::borrow::Cow;
use std::cmp::max; use std::cmp::max;
use std::rc::Rc; use std::rc::Rc;
use std::sync::mpsc::Sender; use std::sync::mpsc::Sender;
use url::Url;
use util::opts; use util::opts;
use util::prefs::PREFS; use util::prefs::PREFS;
use util::resource_files::resources_dir_path; use util::resource_files::resources_dir_path;
@ -173,7 +175,9 @@ impl<Window> Browser<Window> where Window: WindowMethods + 'static {
// Create the constellation, which maintains the engine // Create the constellation, which maintains the engine
// pipelines, including the script and layout threads, as well // pipelines, including the script and layout threads, as well
// as the navigation context. // as the navigation context.
let (constellation_chan, sw_senders) = create_constellation(opts.clone(), let (constellation_chan, sw_senders) = create_constellation(opts.user_agent.clone(),
opts.config_dir.clone(),
opts.url.clone(),
compositor_proxy.clone_compositor_proxy(), compositor_proxy.clone_compositor_proxy(),
time_profiler_chan.clone(), time_profiler_chan.clone(),
mem_profiler_chan.clone(), mem_profiler_chan.clone(),
@ -238,7 +242,9 @@ impl<Window> Browser<Window> where Window: WindowMethods + 'static {
} }
} }
fn create_constellation(opts: opts::Opts, fn create_constellation(user_agent: Cow<'static, str>,
config_dir: Option<String>,
url: Option<Url>,
compositor_proxy: Box<CompositorProxy + Send>, compositor_proxy: Box<CompositorProxy + Send>,
time_profiler_chan: time::ProfilerChan, time_profiler_chan: time::ProfilerChan,
mem_profiler_chan: mem::ProfilerChan, mem_profiler_chan: mem::ProfilerChan,
@ -250,10 +256,10 @@ fn create_constellation(opts: opts::Opts,
let bluetooth_thread: IpcSender<BluetoothRequest> = BluetoothThreadFactory::new(); let bluetooth_thread: IpcSender<BluetoothRequest> = BluetoothThreadFactory::new();
let (public_resource_threads, private_resource_threads) = let (public_resource_threads, private_resource_threads) =
new_resource_threads(opts.user_agent, new_resource_threads(user_agent,
devtools_chan.clone(), devtools_chan.clone(),
time_profiler_chan.clone(), time_profiler_chan.clone(),
opts.config_dir.map(Into::into)); config_dir.map(Into::into));
let image_cache_thread = new_image_cache_thread(public_resource_threads.sender(), let image_cache_thread = new_image_cache_thread(public_resource_threads.sender(),
webrender_api_sender.create_api()); webrender_api_sender.create_api());
let font_cache_thread = FontCacheThread::new(public_resource_threads.sender(), let font_cache_thread = FontCacheThread::new(public_resource_threads.sender(),
@ -280,7 +286,7 @@ fn create_constellation(opts: opts::Opts,
layout_thread::LayoutThread, layout_thread::LayoutThread,
script::script_thread::ScriptThread>::start(initial_state); script::script_thread::ScriptThread>::start(initial_state);
if let Some(url) = opts.url { if let Some(url) = url {
constellation_chan.send(ConstellationMsg::InitLoadUrl(url)).unwrap(); constellation_chan.send(ConstellationMsg::InitLoadUrl(url)).unwrap();
}; };