From ae9a947b7019a57d399823142775c6b61ee69546 Mon Sep 17 00:00:00 2001 From: Eric Guzman Date: Fri, 29 Jun 2012 11:49:11 -0700 Subject: Styles controller - adding check for attached element in getMatchedCSSRules --- js/controllers/styles-controller.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/js/controllers/styles-controller.js b/js/controllers/styles-controller.js index 2282794c..64efdb19 100755 --- a/js/controllers/styles-controller.js +++ b/js/controllers/styles-controller.js @@ -629,7 +629,7 @@ var stylesController = exports.StylesController = Montage.create(Component, { if(!rule) { ///// This should never be hit if providing cssText from existing rule (like those ///// returned from getMatchedCSSRules() - //console.warn('StylesController::_getRuleWithCSSText - No rule found with given cssText.'); + console.warn('StylesController::_getRuleWithCSSText - No rule found with given cssText.'); } return rule; @@ -647,6 +647,10 @@ var stylesController = exports.StylesController = Montage.create(Component, { doc = element.ownerDocument, win = doc.defaultView; + if(!element.parentNode) { + console.warn('StylesController::getMatchingRules - Un-attached element queried'); + } + try { mappedRules = nj.toArray(win.getMatchedCSSRules(element)).map(function(rule) { return this._getRuleWithCSSText(rule.cssText, doc); @@ -675,7 +679,7 @@ var stylesController = exports.StylesController = Montage.create(Component, { }, this); } catch(ERROR) { - console.warn('StylesController::getMatchingRules - Un-attached element queried.'); + console.warn('StylesController::getMatchingRules - getMatchedCSSRules Exception.'); } ///// Function for sorting by specificity values function sorter(ruleA, ruleB) { -- cgit v1.2.3