Bug 1273624 - Really check for existence of parsed declarations in StyleRuleActor's form for compatibility; r=jryans
authorPatrick Brosset <pbrosset@mozilla.com>
Thu, 02 Jun 2016 18:02:11 +0200
changeset 339330 952534db8d857c2eaec8d1eea95b9fcafc9dec16
parent 339329 46d41d717dde8e69ff42e9bddcc6ad819dab95d9
child 339331 6cb4ec731f499d21cd1409d421f29aa824ecafab
push id6249
push userjlund@mozilla.com
push dateMon, 01 Aug 2016 13:59:36 +0000
treeherdermozilla-beta@bad9d4f5bf7e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjryans
bugs1273624
milestone49.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1273624 - Really check for existence of parsed declarations in StyleRuleActor's form for compatibility; r=jryans MozReview-Commit-ID: 3EBW8nTzWiH
devtools/client/inspector/rules/models/rule.js
--- a/devtools/client/inspector/rules/models/rule.js
+++ b/devtools/client/inspector/rules/models/rule.js
@@ -444,17 +444,17 @@ Rule.prototype = {
    * to parse the style's authoredText.
    */
   _getTextProperties: function () {
     let textProps = [];
     let store = this.elementStyle.store;
 
     // Starting with FF49, StyleRuleActors provide parsed declarations.
     let props = this.style.declarations;
-    if (!props) {
+    if (!props.length) {
       props = parseDeclarations(this.cssProperties.isKnown,
                                 this.style.authoredText, true);
     }
 
     for (let prop of props) {
       let name = prop.name;
       // If the authored text has an invalid property, it will show up
       // as nameless.  Skip these as we don't currently have a good