Should WindowClient.navigate(url) navigate with "replacement enabled"?

[mkruisselbrink]

Currently the navigate(url) method is specified as navigating with replacement enabled. Is that correct? I wonder if that was just a mistaken copy of the text from openWindow (where it probably is correct to navigate with replacement enabled).

[annevk]

That does not seem correct indeed. For openWindow() it is correct since you want to replace the initial about:blank.

[NekR]

Does anyone vendor already implements it with replacement enabled?

[wanderview]

We're still working on it for gecko here:

https://bugzilla.mozilla.org/show_bug.cgi?id=1218148

Looking at the current patches we are not using replacement.

[aliams]

I agree: it makes more sense to not use replacement for navigate().

[jakearchibald]

Pre F2F notes: Absolutely seems like a copy-paste error on my part.

[jakearchibald]

F2F: Jake's an idiot and made a copy-paste error. This shouldn't be a replace.