From cb5ceec15060d597470f56ce1eb2e68153583fb9 Mon Sep 17 00:00:00 2001
From: Roman Zaynetdinov <roman.zaynetdinov@lekane.com>
Date: Sun, 25 Dec 2016 11:54:26 +0200
Subject: [PATCH] Remove unused FrameChange's document_ready field

---
 components/constellation/constellation.rs | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/components/constellation/constellation.rs b/components/constellation/constellation.rs
index fa500a40bfd7..d2e0dc421cd0 100644
--- a/components/constellation/constellation.rs
+++ b/components/constellation/constellation.rs
@@ -412,10 +412,6 @@ struct FrameChange {
     /// The pipeline for the document being loaded.
     new_pipeline_id: PipelineId,
 
-    /// Is this document ready to be activated?
-    /// TODO: this flag is never set, it can be removed.
-    document_ready: bool,
-
     /// Is the new document replacing the current document (e.g. a reload)
     /// or pushing it into the session history (e.g. a navigation)?
     replace: bool,
@@ -1446,7 +1442,6 @@ impl<Message, LTF, STF> Constellation<Message, LTF, STF>
             frame_id: top_level_frame_id,
             old_pipeline_id: pipeline_id,
             new_pipeline_id: new_pipeline_id,
-            document_ready: false,
             replace: false,
         });
     }
@@ -1480,7 +1475,6 @@ impl<Message, LTF, STF> Constellation<Message, LTF, STF>
             frame_id: self.root_frame_id,
             old_pipeline_id: None,
             new_pipeline_id: root_pipeline_id,
-            document_ready: false,
             replace: false,
         });
         self.compositor_proxy.send(ToCompositorMsg::ChangePageUrl(root_pipeline_id, url));
@@ -1585,7 +1579,6 @@ impl<Message, LTF, STF> Constellation<Message, LTF, STF>
             frame_id: load_info.info.frame_id,
             old_pipeline_id: load_info.old_pipeline_id,
             new_pipeline_id: load_info.info.new_pipeline_id,
-            document_ready: false,
             replace: load_info.info.replace,
         });
     }
@@ -1630,7 +1623,6 @@ impl<Message, LTF, STF> Constellation<Message, LTF, STF>
             frame_id: frame_id,
             old_pipeline_id: None,
             new_pipeline_id: new_pipeline_id,
-            document_ready: false,
             replace: replace,
         });
     }
@@ -1766,7 +1758,6 @@ impl<Message, LTF, STF> Constellation<Message, LTF, STF>
                     frame_id: root_frame_id,
                     old_pipeline_id: Some(source_id),
                     new_pipeline_id: new_pipeline_id,
-                    document_ready: false,
                     replace: replace,
                 });
 
@@ -1937,7 +1928,7 @@ impl<Message, LTF, STF> Constellation<Message, LTF, STF>
             .map(|id| (id, true));
         let pipeline_id_loaded = self.pending_frames.iter().rev()
             .find(|x| x.old_pipeline_id == current_pipeline_id)
-            .map(|x| (x.new_pipeline_id, x.document_ready))
+            .map(|x| (x.new_pipeline_id, false))
             .or(current_pipeline_id_loaded);
         if let Err(e) = resp_chan.send(pipeline_id_loaded) {
             warn!("Failed get_pipeline response ({}).", e);