format script_layout_interface

3 files changed, 123 insertions, 82 deletions

diff --git a/components/script_layout_interface/lib.rs b/components/script_layout_interface/lib.rs
index 36c2c2a57ca..53af4c302fd 100644
--- a/components/script_layout_interface/lib.rs
+++ b/components/script_layout_interface/lib.rs
@@ -15,13 +15,15 @@ extern crate canvas_traits;
 extern crate cssparser;
 extern crate euclid;
 extern crate gfx_traits;
-#[macro_use] extern crate html5ever;
+#[macro_use]
+extern crate html5ever;
 extern crate ipc_channel;
 extern crate libc;
 #[macro_use]
 extern crate log;
 extern crate malloc_size_of;
-#[macro_use] extern crate malloc_size_of_derive;
+#[macro_use]
+extern crate malloc_size_of_derive;
 extern crate metrics;
 extern crate msg;
 extern crate net_traits;
@@ -98,7 +100,6 @@ impl DomParallelInfo {
     }
 }
 
-
 #[derive(Clone, Copy, Debug, Eq, PartialEq)]
 pub enum LayoutNodeType {
     Element(LayoutElementType),
@@ -126,7 +127,7 @@ pub enum LayoutElementType {
 
 pub enum HTMLCanvasDataSource {
     WebGL(webrender_api::ImageKey),
-    Image(Option<IpcSender<CanvasMsg>>)
+    Image(Option<IpcSender<CanvasMsg>>),
 }
 
 pub struct HTMLCanvasData {
@@ -149,7 +150,8 @@ pub struct TrustedNodeAddress(pub *const c_void);
 unsafe impl Send for TrustedNodeAddress {}
 
 pub fn is_image_data(uri: &str) -> bool {
-    static TYPES: &'static [&'static str] = &["data:image/png", "data:image/gif", "data:image/jpeg"];
+    static TYPES: &'static [&'static str] =
+        &["data:image/png", "data:image/gif", "data:image/jpeg"];
     TYPES.iter().any(|&type_| uri.starts_with(type_))
 }
 
diff --git a/components/script_layout_interface/reporter.rs b/components/script_layout_interface/reporter.rs
index c5f72460c4c..e96e5669b00 100644
--- a/components/script_layout_interface/reporter.rs
+++ b/components/script_layout_interface/reporter.rs
@@ -22,24 +22,28 @@ pub struct CSSErrorReporter {
 }
 
 impl ParseErrorReporter for CSSErrorReporter {
-    fn report_error(&self,
-                    url: &ServoUrl,
-                    location: SourceLocation,
-                    error: ContextualParseError) {
+    fn report_error(&self, url: &ServoUrl, location: SourceLocation, error: ContextualParseError) {
         if log_enabled!(log::Level::Info) {
-            info!("Url:\t{}\n{}:{} {}",
-                  url.as_str(),
-                  location.line,
-                  location.column,
-                  error)
+            info!(
+                "Url:\t{}\n{}:{} {}",
+                url.as_str(),
+                location.line,
+                location.column,
+                error
+            )
         }
 
         //TODO: report a real filename
-        let _ = self.script_chan.lock().unwrap().send(
-            ConstellationControlMsg::ReportCSSError(self.pipelineid,
-                                                    "".to_owned(),
-                                                    location.line,
-                                                    location.column,
-                                                    error.to_string()));
+        let _ = self
+            .script_chan
+            .lock()
+            .unwrap()
+            .send(ConstellationControlMsg::ReportCSSError(
+                self.pipelineid,
+                "".to_owned(),
+                location.line,
+                location.column,
+                error.to_string(),
+            ));
     }
 }
diff --git a/components/script_layout_interface/wrapper_traits.rs b/components/script_layout_interface/wrapper_traits.rs
index a23a417cb6d..189bd004b11 100644
--- a/components/script_layout_interface/wrapper_traits.rs
+++ b/components/script_layout_interface/wrapper_traits.rs
@@ -63,7 +63,9 @@ impl PseudoElementType {
 
     pub fn style_pseudo_element(&self) -> PseudoElement {
         match *self {
-            PseudoElementType::Normal => unreachable!("style_pseudo_element called with PseudoElementType::Normal"),
+            PseudoElementType::Normal => {
+                unreachable!("style_pseudo_element called with PseudoElementType::Normal")
+            },
             PseudoElementType::Before => PseudoElement::Before,
             PseudoElementType::After => PseudoElement::After,
             PseudoElementType::DetailsSummary => PseudoElement::DetailsSummary,
@@ -101,12 +103,17 @@ pub trait LayoutNode: Debug + GetLayoutData + TNode {
     }
 }
 
-pub struct ReverseChildrenIterator<ConcreteNode> where ConcreteNode: LayoutNode {
+pub struct ReverseChildrenIterator<ConcreteNode>
+where
+    ConcreteNode: LayoutNode,
+{
     current: Option<ConcreteNode>,
 }
 
 impl<ConcreteNode> Iterator for ReverseChildrenIterator<ConcreteNode>
-                            where ConcreteNode: LayoutNode {
+where
+    ConcreteNode: LayoutNode,
+{
     type Item = ConcreteNode;
     fn next(&mut self) -> Option<ConcreteNode> {
         let node = self.current;
@@ -115,17 +122,21 @@ impl<ConcreteNode> Iterator for ReverseChildrenIterator<ConcreteNode>
     }
 }
 
-pub struct TreeIterator<ConcreteNode> where ConcreteNode: LayoutNode {
+pub struct TreeIterator<ConcreteNode>
+where
+    ConcreteNode: LayoutNode,
+{
     stack: Vec<ConcreteNode>,
 }
 
-impl<ConcreteNode> TreeIterator<ConcreteNode> where ConcreteNode: LayoutNode {
+impl<ConcreteNode> TreeIterator<ConcreteNode>
+where
+    ConcreteNode: LayoutNode,
+{
     fn new(root: ConcreteNode) -> TreeIterator<ConcreteNode> {
         let mut stack = vec![];
         stack.push(root);
-        TreeIterator {
-            stack: stack,
-        }
+        TreeIterator { stack: stack }
     }
 
     pub fn next_skipping_children(&mut self) -> Option<ConcreteNode> {
@@ -134,7 +145,9 @@ impl<ConcreteNode> TreeIterator<ConcreteNode> where ConcreteNode: LayoutNode {
 }
 
 impl<ConcreteNode> Iterator for TreeIterator<ConcreteNode>
-                            where ConcreteNode: LayoutNode {
+where
+    ConcreteNode: LayoutNode,
+{
     type Item = ConcreteNode;
     fn next(&mut self) -> Option<ConcreteNode> {
         let ret = self.stack.pop();
@@ -143,16 +156,17 @@ impl<ConcreteNode> Iterator for TreeIterator<ConcreteNode>
     }
 }
 
-
 /// A thread-safe version of `LayoutNode`, used during flow construction. This type of layout
 /// node does not allow any parents or siblings of nodes to be accessed, to avoid races.
-pub trait ThreadSafeLayoutNode: Clone + Copy + Debug + GetLayoutData + NodeInfo + PartialEq + Sized {
+pub trait ThreadSafeLayoutNode:
+    Clone + Copy + Debug + GetLayoutData + NodeInfo + PartialEq + Sized
+{
     type ConcreteNode: LayoutNode<ConcreteThreadSafeLayoutNode = Self>;
     type ConcreteElement: TElement;
 
-    type ConcreteThreadSafeLayoutElement:
-        ThreadSafeLayoutElement<ConcreteThreadSafeLayoutNode = Self>
-        + ::selectors::Element<Impl=SelectorImpl>;
+    type ConcreteThreadSafeLayoutElement: ThreadSafeLayoutElement<
+            ConcreteThreadSafeLayoutNode = Self,
+        > + ::selectors::Element<Impl = SelectorImpl>;
     type ChildrenIterator: Iterator<Item = Self> + Sized;
 
     /// Converts self into an `OpaqueNode`.
@@ -173,19 +187,27 @@ pub trait ThreadSafeLayoutNode: Clone + Copy + Debug + GetLayoutData + NodeInfo
     fn parent_style(&self) -> Arc<ComputedValues>;
 
     fn get_before_pseudo(&self) -> Option<Self> {
-        self.as_element().and_then(|el| el.get_before_pseudo()).map(|el| el.as_node())
+        self.as_element()
+            .and_then(|el| el.get_before_pseudo())
+            .map(|el| el.as_node())
     }
 
     fn get_after_pseudo(&self) -> Option<Self> {
-        self.as_element().and_then(|el| el.get_after_pseudo()).map(|el| el.as_node())
+        self.as_element()
+            .and_then(|el| el.get_after_pseudo())
+            .map(|el| el.as_node())
     }
 
     fn get_details_summary_pseudo(&self) -> Option<Self> {
-        self.as_element().and_then(|el| el.get_details_summary_pseudo()).map(|el| el.as_node())
+        self.as_element()
+            .and_then(|el| el.get_details_summary_pseudo())
+            .map(|el| el.as_node())
     }
 
     fn get_details_content_pseudo(&self) -> Option<Self> {
-        self.as_element().and_then(|el| el.get_details_content_pseudo()).map(|el| el.as_node())
+        self.as_element()
+            .and_then(|el| el.get_details_content_pseudo())
+            .map(|el| el.as_node())
     }
 
     fn debug_id(self) -> usize;
@@ -199,7 +221,8 @@ pub trait ThreadSafeLayoutNode: Clone + Copy + Debug + GetLayoutData + NodeInfo
 
     #[inline]
     fn get_pseudo_element_type(&self) -> PseudoElementType {
-        self.as_element().map_or(PseudoElementType::Normal, |el| el.get_pseudo_element_type())
+        self.as_element()
+            .map_or(PseudoElementType::Normal, |el| el.get_pseudo_element_type())
     }
 
     fn get_style_and_layout_data(&self) -> Option<OpaqueStyleAndLayoutData>;
@@ -283,13 +306,8 @@ pub trait DangerousThreadSafeLayoutNode: ThreadSafeLayoutNode {
     unsafe fn dangerous_next_sibling(&self) -> Option<Self>;
 }
 
-pub trait ThreadSafeLayoutElement
-    : Clone
-    + Copy
-    + Sized
-    + Debug
-    + ::selectors::Element<Impl=SelectorImpl>
-    + GetLayoutData
+pub trait ThreadSafeLayoutElement:
+    Clone + Copy + Sized + Debug + ::selectors::Element<Impl = SelectorImpl> + GetLayoutData
 {
     type ConcreteThreadSafeLayoutNode: ThreadSafeLayoutNode<ConcreteThreadSafeLayoutElement = Self>;
 
@@ -329,7 +347,13 @@ pub trait ThreadSafeLayoutElement
 
     #[inline]
     fn get_before_pseudo(&self) -> Option<Self> {
-        if self.style_data().styles.pseudos.get(&PseudoElement::Before).is_some() {
+        if self
+            .style_data()
+            .styles
+            .pseudos
+            .get(&PseudoElement::Before)
+            .is_some()
+        {
             Some(self.with_pseudo(PseudoElementType::Before))
         } else {
             None
@@ -338,7 +362,13 @@ pub trait ThreadSafeLayoutElement
 
     #[inline]
     fn get_after_pseudo(&self) -> Option<Self> {
-        if self.style_data().styles.pseudos.get(&PseudoElement::After).is_some() {
+        if self
+            .style_data()
+            .styles
+            .pseudos
+            .get(&PseudoElement::After)
+            .is_some()
+        {
             Some(self.with_pseudo(PseudoElementType::After))
         } else {
             None
@@ -347,8 +377,7 @@ pub trait ThreadSafeLayoutElement
 
     #[inline]
     fn get_details_summary_pseudo(&self) -> Option<Self> {
-        if self.local_name() == &local_name!("details") &&
-           self.namespace() == &ns!(html) {
+        if self.local_name() == &local_name!("details") && self.namespace() == &ns!(html) {
             Some(self.with_pseudo(PseudoElementType::DetailsSummary))
         } else {
             None
@@ -358,8 +387,9 @@ pub trait ThreadSafeLayoutElement
     #[inline]
     fn get_details_content_pseudo(&self) -> Option<Self> {
         if self.local_name() == &local_name!("details") &&
-           self.namespace() == &ns!(html) &&
-           self.get_attr(&ns!(), &local_name!("open")).is_some() {
+            self.namespace() == &ns!(html) &&
+            self.get_attr(&ns!(), &local_name!("open")).is_some()
+        {
             Some(self.with_pseudo(PseudoElementType::DetailsContent))
         } else {
             None
@@ -374,49 +404,52 @@ pub trait ThreadSafeLayoutElement
     fn style(&self, context: &SharedStyleContext) -> Arc<ComputedValues> {
         let data = self.style_data();
         match self.get_pseudo_element_type() {
-            PseudoElementType::Normal => {
-                data.styles.primary().clone()
-            },
+            PseudoElementType::Normal => data.styles.primary().clone(),
             other => {
                 // Precompute non-eagerly-cascaded pseudo-element styles if not
                 // cached before.
                 let style_pseudo = other.style_pseudo_element();
                 match style_pseudo.cascade_type() {
                     // Already computed during the cascade.
-                    PseudoElementCascadeType::Eager => {
-                        self.style_data()
-                            .styles.pseudos.get(&style_pseudo)
-                            .unwrap().clone()
-                    },
-                    PseudoElementCascadeType::Precomputed => {
-                        context.stylist.precomputed_values_for_pseudo::<Self::ConcreteElement>(
+                    PseudoElementCascadeType::Eager => self
+                        .style_data()
+                        .styles
+                        .pseudos
+                        .get(&style_pseudo)
+                        .unwrap()
+                        .clone(),
+                    PseudoElementCascadeType::Precomputed => context
+                        .stylist
+                        .precomputed_values_for_pseudo::<Self::ConcreteElement>(
                             &context.guards,
                             &style_pseudo,
                             Some(data.styles.primary()),
                             &ServoMetricsProvider,
-                        )
-                    }
+                        ),
                     PseudoElementCascadeType::Lazy => {
-                        context.stylist.lazily_compute_pseudo_element_style(
-                           &context.guards,
-                           unsafe { self.unsafe_get() },
-                           &style_pseudo,
-                           RuleInclusion::All,
-                           data.styles.primary(),
-                           /* is_probe = */ false,
-                           &ServoMetricsProvider,
-                           /* matching_func = */ None,
-                        ).unwrap()
-                    }
+                        context
+                            .stylist
+                            .lazily_compute_pseudo_element_style(
+                                &context.guards,
+                                unsafe { self.unsafe_get() },
+                                &style_pseudo,
+                                RuleInclusion::All,
+                                data.styles.primary(),
+                                /* is_probe = */ false,
+                                &ServoMetricsProvider,
+                                /* matching_func = */ None,
+                            ).unwrap()
+                    },
                 }
-            }
+            },
         }
     }
 
     #[inline]
     fn selected_style(&self) -> Arc<ComputedValues> {
         let data = self.style_data();
-        data.styles.pseudos
+        data.styles
+            .pseudos
             .get(&PseudoElement::Selection)
             .unwrap_or(data.styles.primary())
             .clone()
@@ -433,11 +466,13 @@ pub trait ThreadSafeLayoutElement
     fn resolved_style(&self) -> Arc<ComputedValues> {
         let data = self.style_data();
         match self.get_pseudo_element_type() {
-            PseudoElementType::Normal
-                => data.styles.primary().clone(),
-            other
-                => data.styles.pseudos
-                       .get(&other.style_pseudo_element()).unwrap().clone(),
+            PseudoElementType::Normal => data.styles.primary().clone(),
+            other => data
+                .styles
+                .pseudos
+                .get(&other.style_pseudo_element())
+                .unwrap()
+                .clone(),
         }
     }
 }