fix(client): do not reset karmaNavigating in unload handler (#3591)

johnjbarton · web-flow · commit 4a8178f3a050 · 2020-12-17T14:08:55.000-08:00
The unload handler itself does not know about navigation. May fix #3482
diff --git a/client/karma.js b/client/karma.js
@@ -135,7 +135,6 @@ function Karma (socket, iframe, opener, navigator, location, document) {
       // TODO(vojta): show what test (with explanation about jasmine.UPDATE_INTERVAL)
       self.error('Some of your tests did a full page reload!')
     }
-    karmaNavigating = false
   }
 
   function clearContext () {
diff --git a/static/karma.js b/static/karma.js
@@ -145,7 +145,6 @@ function Karma (socket, iframe, opener, navigator, location, document) {
       // TODO(vojta): show what test (with explanation about jasmine.UPDATE_INTERVAL)
       self.error('Some of your tests did a full page reload!')
     }
-    karmaNavigating = false
   }
 
   function clearContext () {

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,6 @@ function Karma (socket, iframe, opener, navigator, location, document) {`
`135`	`135`	`// TODO(vojta): show what test (with explanation about jasmine.UPDATE_INTERVAL)`
`136`	`136`	`self.error('Some of your tests did a full page reload!')`
`137`	`137`	`}`
`138`		`- karmaNavigating = false`
`139`	`138`	`}`
`140`	`139`
`141`	`140`	`function clearContext () {`
Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,6 @@ function Karma (socket, iframe, opener, navigator, location, document) {`
`145`	`145`	`// TODO(vojta): show what test (with explanation about jasmine.UPDATE_INTERVAL)`
`146`	`146`	`self.error('Some of your tests did a full page reload!')`
`147`	`147`	`}`
`148`		`- karmaNavigating = false`
`149`	`148`	`}`
`150`	`149`
`151`	`150`	`function clearContext () {`