addressed majority of review comments

components/script/dom/bindings/trace.rs

@@ -96,6 +96,9 @@ use url::Url;
 use util::str::{DOMString, LengthOrPercentageOrAuto};
 use uuid::Uuid;
 use webrender_traits::WebGLError;
+use style::selector_impl::ServoSelectorImpl;
+use style::stylesheets::CSSRule;
+
 
 /// A trait to allow tracing (only) DOM objects.
 pub trait JSTraceable {

components/script/dom/cssrule.rs

@@ -8,7 +8,6 @@ use dom::bindings::codegen::Bindings::CSSRuleBinding::CSSRuleMethods;
 use dom::bindings::global::GlobalRef;
 use dom::bindings::js::{Root};
 use dom::bindings::reflector::{Reflector, reflect_dom_object};
-use dom::cssstylerule::CSSStyleRule;
 use dom::window::Window;
 use style::selector_impl::ServoSelectorImpl;
 use style::stylesheets;

components/script/dom/cssrulelist.rs

@@ -10,7 +10,6 @@ use dom::bindings::reflector::{Reflector, reflect_dom_object};
 use dom::cssrule::CSSRule;
 use dom::cssstylesheet::CSSStyleSheet;
 use dom::window::Window;
-use heapsize::HeapSizeOf;
 
 #[dom_struct]
 pub struct CSSRuleList {
@@ -20,15 +19,15 @@ pub struct CSSRuleList {
 
 impl CSSRuleList {
     #[allow(unrooted_must_root)]
-    pub fn new_inherited(stylesheet: &CSSStyleSheet) -> CSSRuleList {
+    fn new_inherited(stylesheet: JS<CSSStyleSheet>) -> CSSRuleList {
         CSSRuleList {
             reflector_: Reflector::new(),
-            stylesheet: JS::from_ref(stylesheet),
+            stylesheet: stylesheet
         }
     }
 
     #[allow(unrooted_must_root)]
-    pub fn new(window: &Window, stylesheet: &CSSStyleSheet) -> Root<CSSRuleList> {
+    pub fn new(window: &Window, stylesheet: JS<CSSStyleSheet>) -> Root<CSSRuleList> {
         reflect_dom_object(box CSSRuleList::new_inherited(stylesheet),
                            GlobalRef::Window(window), CSSRuleListBinding::Wrap)
     }
@@ -37,7 +36,7 @@ impl CSSRuleList {
 impl CSSRuleListMethods for CSSRuleList {
     // https://drafts.csswg.org/cssom/#dom-stylesheetlist-length
     fn Length(&self) -> u32 {
-       self.stylesheet.get_cssstylesheet().heap_size_of_children() as u32
+       self.stylesheet.rules.len() as u32
     }
 
     // https://drafts.csswg.org/cssom/#dom-stylesheetlist-item

components/script/dom/cssstylesheet.rs

@@ -8,7 +8,6 @@ use dom::bindings::global::GlobalRef;
 use dom::bindings::js::Root;
 use dom::bindings::reflector::{Reflector, reflect_dom_object};
 use dom::cssrulelist::CSSRuleList;
-use dom::node::Node;
 use dom::window::Window;
 use std::sync::Arc;
 use style::servo::Stylesheet;
@@ -19,7 +18,6 @@ use util::str::DOMString;
 pub struct CSSStyleSheet {
     ss: StyleSheet,
     stylesheet: Arc<Stylesheet>,
-        //node: Node,
 }
 
 impl CSSStyleSheet {
@@ -28,7 +26,6 @@ impl CSSStyleSheet {
         CSSStyleSheet {
             ss: StyleSheet::new_inherited(DOMString::from_string(String::from("text/css")), None, None, None),
             stylesheet: stylesheet,
-            //node: *node,
         }
     }
 

components/script/dom/mod.rs

@@ -239,6 +239,7 @@ pub mod crypto;
 pub mod css;
 pub mod cssrule;
 pub mod cssrulelist;
+pub mod cssstylerule;
 pub mod cssstylesheet;
 pub mod cssstyledeclaration;
 pub mod customevent;