Bug 1357461: Add some logging. draft
authorEmilio Cobos Álvarez <emilio@crisal.io>
Fri, 02 Jun 2017 19:51:28 +0200
changeset 588587 cb72032d157611eb095875920d254d539edd1d7e
parent 588586 57113ef7f65801812a28108817bdb80723170671
child 588588 ca0a309b52ee9508e9685482a3da93d8995fa3d5
push id62096
push userbmo:emilio+bugs@crisal.io
push dateSat, 03 Jun 2017 10:43:57 +0000
bugs1357461
milestone55.0a1
Bug 1357461: Add some logging. MozReview-Commit-ID: 41fT906fHvv
servo/components/style/stylist.rs
--- a/servo/components/style/stylist.rs
+++ b/servo/components/style/stylist.rs
@@ -793,25 +793,29 @@ impl Stylist {
         &self,
         stylesheets: I,
         guard: &SharedRwLockReadGuard
     ) -> bool
         where I: Iterator<Item = &'a Arc<Stylesheet>>
     {
         use invalidation::media_queries::PotentiallyEffectiveMediaRules;
 
+        debug!("Stylist::media_features_change_changed_style");
+
         for stylesheet in stylesheets {
             let effective_now =
                 stylesheet.media.read_with(guard)
                     .evaluate(&self.device, self.quirks_mode);
 
             let effective_then =
                 self.effective_media_query_results.was_effective(&**stylesheet);
 
             if effective_now != effective_then {
+                debug!(" > Stylesheet changed -> {}, {}",
+                       effective_then, effective_now);
                 return true
             }
 
             if !effective_now {
                 continue;
             }
 
             let mut iter =
@@ -837,31 +841,35 @@ impl Stylist {
                     CssRule::Import(ref lock) => {
                         let import_rule = lock.read_with(guard);
                         let mq = import_rule.stylesheet.media.read_with(guard);
                         let effective_now =
                             mq.evaluate(&self.device, self.quirks_mode);
                         let effective_then =
                             self.effective_media_query_results.was_effective(import_rule);
                         if effective_now != effective_then {
+                            debug!(" > @import rule changed {} -> {}",
+                                   effective_then, effective_now);
                             return true;
                         }
 
                         if !effective_now {
                             iter.skip_children();
                         }
                     }
                     CssRule::Media(ref lock) => {
                         let media_rule = lock.read_with(guard);
                         let mq = media_rule.media_queries.read_with(guard);
                         let effective_now =
                             mq.evaluate(&self.device, self.quirks_mode);
                         let effective_then =
                             self.effective_media_query_results.was_effective(media_rule);
                         if effective_now != effective_then {
+                            debug!(" > @media rule changed {} -> {}",
+                                   effective_then, effective_now);
                             return true;
                         }
 
                         if !effective_now {
                             iter.skip_children();
                         }
                     }
                 }