fixed #37

nitin42 · nitin42 · commit cc3c003067ef · 2018-05-18T09:33:39.000+05:30
diff --git a/src/extension/components/ReactPerfDevtool.js b/src/extension/components/ReactPerfDevtool.js
@@ -76,7 +76,7 @@ export class ReactPerfDevtool extends React.Component {
   }
 
   componentWillUnmount() {
-    clearInterval(this.timer)
+    this.timer && clearInterval(this.timer)
   }
 
   reloadInspectedWindow = () => chrome.devtools.inspectedWindow.reload()
@@ -252,6 +252,9 @@ export class ReactPerfDevtool extends React.Component {
     })
 
     this.clearMeasures()
+
+    // Makes sure that we are not batching calls for example - setting chart state here false and batching drawChart may result in an error because the canvas may not render.
+    this.timer && clearInterval(this.timer)
   }
 
   browserReload = () =>

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ export class ReactPerfDevtool extends React.Component {`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`componentWillUnmount() {`
`79`		`- clearInterval(this.timer)`
	`79`	`+ this.timer && clearInterval(this.timer)`
`80`	`80`	`}`
`81`	`81`
`82`	`82`	`reloadInspectedWindow = () => chrome.devtools.inspectedWindow.reload()`
`@@ -252,6 +252,9 @@ export class ReactPerfDevtool extends React.Component {`
`252`	`252`	`})`
`253`	`253`
`254`	`254`	`this.clearMeasures()`
	`255`	`+`
	`256`	`+ // Makes sure that we are not batching calls for example - setting chart state here false and batching drawChart may result in an error because the canvas may not render.`
	`257`	`+ this.timer && clearInterval(this.timer)`
`255`	`258`	`}`
`256`	`259`
`257`	`260`	`browserReload = () =>`