diff options
author | Eric Guzman | 2012-06-29 14:31:19 -0700 |
---|---|---|
committer | Eric Guzman | 2012-06-29 14:31:19 -0700 |
commit | 25890906f10e4530f0245632d028a646ece563c1 (patch) | |
tree | 28badd817a961fb61f4e9a567b28c8fd030122ba | |
parent | 06ac6fcdde5b1f9a00e4c966ce165af0a4c3c1da (diff) | |
download | ninja-25890906f10e4530f0245632d028a646ece563c1.tar.gz |
Another null check on results from getMatchedCSSRules
-rwxr-xr-x | js/controllers/styles-controller.js | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/js/controllers/styles-controller.js b/js/controllers/styles-controller.js index 39deeaa8..b50f43cc 100755 --- a/js/controllers/styles-controller.js +++ b/js/controllers/styles-controller.js | |||
@@ -629,7 +629,7 @@ var stylesController = exports.StylesController = Montage.create(Component, { | |||
629 | if(!rule) { | 629 | if(!rule) { |
630 | ///// This should never be hit if providing cssText from existing rule (like those | 630 | ///// This should never be hit if providing cssText from existing rule (like those |
631 | ///// returned from getMatchedCSSRules() | 631 | ///// returned from getMatchedCSSRules() |
632 | console.warn('StylesController::_getRuleWithCSSText - No rule found with given cssText.'); | 632 | //console.warn('StylesController::_getRuleWithCSSText - No rule found with given cssText.'); |
633 | } | 633 | } |
634 | 634 | ||
635 | return rule; | 635 | return rule; |
@@ -649,7 +649,7 @@ var stylesController = exports.StylesController = Montage.create(Component, { | |||
649 | win = doc.defaultView; | 649 | win = doc.defaultView; |
650 | 650 | ||
651 | if(!element.parentNode) { | 651 | if(!element.parentNode) { |
652 | console.warn('StylesController::getMatchingRules - Un-attached element queried'); | 652 | //console.warn('StylesController::getMatchingRules - Un-attached element queried'); |
653 | return []; | 653 | return []; |
654 | } | 654 | } |
655 | 655 | ||
@@ -657,7 +657,7 @@ var stylesController = exports.StylesController = Montage.create(Component, { | |||
657 | matchedRules = win.getMatchedCSSRules(element); | 657 | matchedRules = win.getMatchedCSSRules(element); |
658 | 658 | ||
659 | if(!matchedRules) { | 659 | if(!matchedRules) { |
660 | console.warn('StylesController::getMatchingRules - matched rules are null'); | 660 | //console.warn('StylesController::getMatchingRules - matched rules are null'); |
661 | return []; | 661 | return []; |
662 | } | 662 | } |
663 | 663 | ||
@@ -688,7 +688,8 @@ var stylesController = exports.StylesController = Montage.create(Component, { | |||
688 | }, this); | 688 | }, this); |
689 | 689 | ||
690 | } catch(ERROR) { | 690 | } catch(ERROR) { |
691 | console.warn('StylesController::getMatchingRules - getMatchedCSSRules Exception.'); | 691 | //console.warn('StylesController::getMatchingRules - getMatchedCSSRules Exception.'); |
692 | return []; | ||
692 | } | 693 | } |
693 | ///// Function for sorting by specificity values | 694 | ///// Function for sorting by specificity values |
694 | function sorter(ruleA, ruleB) { | 695 | function sorter(ruleA, ruleB) { |