Bug 1563587 Comment 27 Edit History

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

(In reply to Rob Wu [:robwu] from comment #26)
> Shouldn't this change be listed on MDN (dev-doc-needed) or be added to https://www.fxsitecompat.dev ?
yes


> I didn't look at the full patch in detail, but at the top of [the patch](https://hg.mozilla.org/mozilla-central/rev/214fac6eb1c0) `setTimeout` was added after `history.back()`. Shouldn't the `popstate` event be used instead, to make sure that the history navigation has been processed?
I don't know which setTimeout you're talking about. In some cases popstate would have worked, in some case not.
(In reply to Rob Wu [:robwu] from comment #26)
> Shouldn't this change be listed on MDN (dev-doc-needed) or be added to https://www.fxsitecompat.dev ?

yes


> I didn't look at the full patch in detail, but at the top of [the patch](https://hg.mozilla.org/mozilla-central/rev/214fac6eb1c0) `setTimeout` was added after `history.back()`. Shouldn't the `popstate` event be used instead, to make sure that the history navigation has been processed?

I don't know which setTimeout you're talking about. In some cases popstate would have worked, in some case not.
And wpt tests got modified some more on github.

Back to Bug 1563587 Comment 27