1983546 - Traverse navigate events are fired incorrectly.

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect)

Description

[Andreas Farre [:farre]]

There are two instance when the navigate events are fired incorrectly for traversals.

1. when there are no event handlers on the navigation object but method trackers we skip firing the navigate event
2. when we're performing a same document history traversal we still fire a push/replace/reload navigate event

Comment 1

[Andreas Farre [:farre]]

Attachment: Bug 1983546 - Fire the navigate event correctly for history navigations. r=smaug!,jjaschke

In the same manner as with beforeunload we track when we need to fire a
traverse navigate event for the top level window context. This is
required when: any event handlers are registered on the window's
navigation object or when there are any ongoing or upcoming method
trackers associated with a window's navigation object.

Also make sure to not fire a push/replace/reload navigate event when
only a traverse navigate event should be fired.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/a9dce94db87a
https://hg.mozilla.org/integration/autoland/rev/df126e5491a1
Fire the navigate event correctly for history navigations. r=jjaschke

Comment 3

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/df126e5491a1