Bug 1508293 - Update comment for TRACK_CHANGE event in Changes reducer; r=bradwerth
authorRazvan Caliman <rcaliman@mozilla.com>
Mon, 26 Nov 2018 15:55:36 +0000
changeset 507552 7ac08d9d34640922c7b9593eb9ca3d12f56432b7
parent 507551 7f46f3f93ec7cf9d3374178eb801e9804299ad3b
child 507553 33930dc85090600ba8c1ab269b9854ac4f1c0ebb
push id1905
push userffxbld-merge
push dateMon, 21 Jan 2019 12:33:13 +0000
treeherdermozilla-release@c2fca1944d8c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbradwerth
bugs1508293
milestone65.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 1508293 - Update comment for TRACK_CHANGE event in Changes reducer; r=bradwerth Differential Revision: https://phabricator.services.mozilla.com/D12768
devtools/client/inspector/changes/reducers/changes.js
--- a/devtools/client/inspector/changes/reducers/changes.js
+++ b/devtools/client/inspector/changes/reducers/changes.js
@@ -124,32 +124,33 @@ function removeRule(ruleId, rules) {
 }
 
 /**
  * Aggregated changes grouped by sources (stylesheet/element), which contain rules,
  * which contain collections of added and removed CSS declarations.
  *
  * Structure:
  *    <sourceId>: {
- *      type: // "stylesheet" or "element"
- *      href: // Stylesheet or document URL
+ *      type: // {String} One of: "stylesheet", "inline" or "element"
+ *      href: // {String|null} Stylesheet or document URL; null for inline stylesheets
  *      rules: {
  *        <ruleId>: {
- *          selector: "" // String CSS selector or CSS at-rule text
- *          changeType:  // Optional string: "rule-add" or "rule-remove"
- *          children: [] // Array of <ruleId> for child rules of this rule.
- *          parent:      // <ruleId> of the parent rule
- *          add: {
- *            <property>: <value> // CSS declaration
- *            ...
- *          },
- *          remove: {
- *            <property>: <value> // CSS declaration
- *           ...
- *          }
+ *          selector:    // {String} CSS selector or CSS at-rule text
+ *          changeType:  // {String} Optional; one of: "rule-add" or "rule-remove"
+ *          children: [] // {Array} of <ruleId> for child rules of this rule
+ *          parent:      // {String} <ruleId> of the parent rule
+ *          add: [       // {Array} of objects with CSS declarations
+ *            {
+ *              property:    // {String} CSS property name
+ *              value:       // {String} CSS property value
+ *              index:       // {Number} Position of the declaration within its CSS rule
+ *            }
+ *            ... // more declarations
+ *          ],
+ *          remove: []   // {Array} of objects with CSS declarations
  *        }
  *        ... // more rules
  *      }
  *    }
  *    ... // more sources
  */
 const INITIAL_STATE = {};