Bug 1768475 Comment 12 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Beta/Release Uplift Approval Request
* **User impact if declined**: Reloading a "slow" page while the inspector is open leads to a blank inspector.

You can easily test this by opening youtube, opening the inspector and reloading the page. Depending on the load it might not always be blank, but on my machine it's pretty frequent.

(the patch is not on central yet, but the beta uplift window is almost over so I prefer to make this visible asap)
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: The change only impacts devtools users, is a small javascript change.

In Bug 1765760 we stopped waiting for documents to be fully initialized to start the DevTools Inspector. But there was one codepath left to update, which was not involved in the scenario we focused on for Bug 1765760. So we are just making the logic more consistent with Bug 1765760 here, and we are not introducing any new logic.
* **String changes made/needed**: 
* **Is Android affected?**: No
### Beta/Release Uplift Approval Request
* **User impact if declined**: Reloading a "slow" page while the inspector is open leads to a blank inspector.

You can easily test this by opening youtube, opening the inspector and reloading the page. Depending on the load it might not always be blank, but on my machine it's pretty frequent.

(the patch is not on central yet, but the beta uplift window is almost over so I prefer to make this visible asap)
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: Bug 1769701
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: The change only impacts devtools users, is a small javascript change.

In Bug 1765760 we stopped waiting for documents to be fully initialized to start the DevTools Inspector. But there was one codepath left to update, which was not involved in the scenario we focused on for Bug 1765760. So we are just making the logic more consistent with Bug 1765760 here, and we are not introducing any new logic.
* **String changes made/needed**: 
* **Is Android affected?**: No

Back to Bug 1768475 Comment 12