Make more things private to the style crate.

1 files changed, 4 insertions, 15 deletions

diff --git a/src/components/layout/layout_task.rs b/src/components/layout/layout_task.rs
index a0e1dd45d89..e4cba397379 100644
--- a/src/components/layout/layout_task.rs
+++ b/src/components/layout/layout_task.rs
@@ -56,7 +56,7 @@ use std::comm::{channel, Sender, Receiver, Select};
 use std::mem;
 use std::ptr;
 use style::{AuthorOrigin, Stylesheet, Stylist};
-use style::CSSFontFaceRule;
+use style::iter_font_face_rules;
 use sync::{Arc, Mutex};
 use url::Url;
 
@@ -490,20 +490,9 @@ impl LayoutTask {
     fn handle_add_stylesheet(&mut self, sheet: Stylesheet) {
         // Find all font-face rules and notify the font cache of them.
         // GWTODO: Need to handle unloading web fonts (when we handle unloading stylesheets!)
-        // GWTODO: Need to handle font-face nested within media rules.
-        for rule in sheet.rules.iter() {
-            match rule {
-                &CSSFontFaceRule(ref font_face_rule) => {
-                    let mut font_urls = vec!();
-                    for source in font_face_rule.sources.iter() {
-                        font_urls.push(source.url.clone());
-                    }
-                    self.font_cache_task.add_web_font(font_urls, font_face_rule.family.as_slice());
-                },
-                _ => {}
-            }
-        }
-
+        iter_font_face_rules(&sheet, |family, url| {
+            self.font_cache_task.add_web_font(family.to_string(), url.clone());
+        });
         self.stylist.add_stylesheet(sheet, AuthorOrigin);
     }