When I open the debugger panel I see a moment when the pause button turns into a blue play button (as if the debugger is paused). After a second it goes back to normal. See attached gif. I'm assuming this is due to Bug 1200798.

I was calling `onNavigate` on startup which is wrong, because all I wanted to do was load sources. In `onNavigate` the thread state is checked and the button is updated, and the JS engine is paused for a second on load, so it was reflecting that.

Comment on attachment 8695339 [details] [diff] [review] 1229923.patch Review of attachment 8695339 [details] [diff] [review]: ----------------------------------------------------------------- Fixes the problem for me. Is there a reasonable way to add a regression test for this?

(In reply to Brian Grinstead [:bgrins] from comment #4) > > Fixes the problem for me. Is there a reasonable way to add a regression > test for this? I don't think it needs a test. We would be testing that the ThreadState._update function is not called on startup which I think is too specific of a workflow to be in a test. Writing tests like that make it hard to refactor in the future (having just gone through updating a lot of tests...) I'd be down for a test if we were looking for a specific action to happen. I don't think every single change needs a test though, especially as I clean up code from the refactoring.