diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2015-11-07 22:34:45 +0530 |
---|---|---|
committer | bors-servo <lbergstrom+bors@mozilla.com> | 2015-11-07 22:34:45 +0530 |
commit | 9a465c58429547b59b8a6e1258eaaea46bf7e5a2 (patch) | |
tree | 698f938b9cbbc80be946853d4704357c84e8192a /components/compositing/pipeline.rs | |
parent | 7c4fd5e77078b523ae368f42c7d43c9d76acaf50 (diff) | |
parent | 28f5d54334b17064b0eaf02614b8256f27f4bfb2 (diff) | |
download | servo-9a465c58429547b59b8a6e1258eaaea46bf7e5a2.tar.gz servo-9a465c58429547b59b8a6e1258eaaea46bf7e5a2.zip |
Auto merge of #8355 - Ms2ger:Exit, r=nox
Cleanup exit messages and related code.
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8355)
<!-- Reviewable:end -->
Diffstat (limited to 'components/compositing/pipeline.rs')
-rw-r--r-- | components/compositing/pipeline.rs | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/components/compositing/pipeline.rs b/components/compositing/pipeline.rs index ad425224d26..4cc8b87f14a 100644 --- a/components/compositing/pipeline.rs +++ b/components/compositing/pipeline.rs @@ -15,7 +15,7 @@ use ipc_channel::router::ROUTER; use layers::geometry::DevicePixel; use layout_traits::{LayoutControlChan, LayoutTaskFactory}; use msg::constellation_msg::{ConstellationChan, Failure, FrameId, PipelineId, SubpageId}; -use msg::constellation_msg::{LoadData, MozBrowserEvent, PipelineExitType, WindowSizeData}; +use msg::constellation_msg::{LoadData, MozBrowserEvent, WindowSizeData}; use msg::constellation_msg::{PipelineNamespaceId}; use net_traits::ResourceTask; use net_traits::image_cache_task::ImageCacheTask; @@ -248,13 +248,13 @@ impl Pipeline { let _ = self.chrome_to_paint_chan.send(ChromeToPaintMsg::PaintPermissionRevoked); } - pub fn exit(&self, exit_type: PipelineExitType) { + pub fn exit(&self) { debug!("pipeline {:?} exiting", self.id); // Script task handles shutting down layout, and layout handles shutting down the painter. // For now, if the script task has failed, we give up on clean shutdown. if self.script_chan - .send(ConstellationControlMsg::ExitPipeline(self.id, exit_type)) + .send(ConstellationControlMsg::ExitPipeline(self.id)) .is_ok() { // Wait until all slave tasks have terminated and run destructors // NOTE: We don't wait for script task as we don't always own it @@ -275,15 +275,10 @@ impl Pipeline { } pub fn force_exit(&self) { - let _ = self.script_chan.send( - ConstellationControlMsg::ExitPipeline(self.id, - PipelineExitType::PipelineOnly)).unwrap(); - let _ = self.chrome_to_paint_chan.send(ChromeToPaintMsg::Exit( - None, - PipelineExitType::PipelineOnly)); + let _ = self.script_chan.send(ConstellationControlMsg::ExitPipeline(self.id)).unwrap(); + let _ = self.chrome_to_paint_chan.send(ChromeToPaintMsg::Exit); let LayoutControlChan(ref layout_channel) = self.layout_chan; - let _ = layout_channel.send( - LayoutControlMsg::ExitNow(PipelineExitType::PipelineOnly)).unwrap(); + let _ = layout_channel.send(LayoutControlMsg::ExitNow).unwrap(); } pub fn to_sendable(&self) -> CompositionPipeline { |