Bug 1352777 - Fix dropmarkers in JSON viewer not collapsing long string r?rickychien draft
authorMatt R <matthieu.rigolot@gmail.com>
Mon, 24 Apr 2017 08:35:09 +0100
changeset 566989 0f8520b43baa4b703547dedcd0e35875decee8e4
parent 503306 7513b3f42058e9bcf9950d4acf4647d4ad2240f0
child 625483 48109fd81cb8bb551e4dadbb20c4ba8020149eab
push id55400
push userbmo:matthieu.rigolot@gmail.com
push dateMon, 24 Apr 2017 09:04:39 +0000
reviewersrickychien
bugs1352777
milestone55.0a1
Bug 1352777 - Fix dropmarkers in JSON viewer not collapsing long string r?rickychien MozReview-Commit-ID: BdixqPyi8x4
devtools/client/shared/components/tree/tree-cell.js
--- a/devtools/client/shared/components/tree/tree-cell.js
+++ b/devtools/client/shared/components/tree/tree-cell.js
@@ -37,17 +37,18 @@ define(function (require, exports, modul
     },
 
     /**
      * Optimize cell rendering. Rerender cell content only if
      * the value or expanded state changes.
      */
     shouldComponentUpdate: function (nextProps, nextState) {
       return (this.props.value != nextProps.value) ||
-        (this.state !== nextState);
+        (this.state !== nextState) ||
+        (this.props.member.open != nextProps.member.open);
     },
 
     getCellClass: function (object, id) {
       let decorator = this.props.decorator;
       if (!decorator || !decorator.getCellClass) {
         return [];
       }