Bug 1333383 - fix netmonitor response tab for json responses with null values;r=jdescottes a=gchang
authorCosm <kausam2015@gmail.com>
Fri, 27 Jan 2017 16:36:25 +0530
changeset 376019 a6c09cee1a8c2ede72b14bd426ef9f85528f0c8c
parent 376018 8eb9b71cc50a07d8412d083da69ee733c6d8a80f
child 376020 acd362e1cb115a7204e97f74a338d664353d4dc9
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjdescottes, gchang
bugs1333383
milestone53.0a2
Bug 1333383 - fix netmonitor response tab for json responses with null values;r=jdescottes a=gchang MozReview-Commit-ID: 2udR3B7uA4h
devtools/client/netmonitor/shared/components/properties-view.js
--- a/devtools/client/netmonitor/shared/components/properties-view.js
+++ b/devtools/client/netmonitor/shared/components/properties-view.js
@@ -107,17 +107,17 @@ const PropertiesView = createClass({
   renderValueWithRep(props) {
     const { member } = props;
 
     // Hide strings with following conditions
     // 1. this row is a togglable section
     // 2. the `value` object has a `value` property, only happend in Cookies panel
     // Put 2 here to not dup this method
     if (member.level === 0 ||
-      (typeof member.value === "object" && member.value.value)) {
+      (typeof member.value === "object" && member.value && member.value.value)) {
       return null;
     }
 
     return Rep(Object.assign(props, {
       // FIXME: A workaround for the issue in StringRep
       // Force StringRep to crop the text everytime
       member: Object.assign({}, member, { open: false }),
       mode: MODE.TINY,