OK, so after a bit of digging into this: 1) Comment 1 seems to be about the change in what url shows up in the url bar when the tab is (incorrectly) not closed. It's entirely possible that Honza's changes had to do with that, yes, since they would affect the "final channel URI". It's not at all clear to me what the right behavior there is. It should be spun off to a new bug, because it's not what this original bug is about. I filed bug 1532752 on this. 2) In Firefox 54 (so _before_ Honza's changes) the noreferrer case is broken while the "normal" case works, as I expected bu code inspection. That's also the case in Firefox 65, and is what this bug is about. This part is definitely Document Navigation, but is NOT a regression from Honza's patches, obviously. I'll look into this. 3) In current nightly, the case without noreferrer is _also_ broken. That's because target="_blank" now implies noopener (as of bug 1503681) and it's the noopener bits that are really relevant here (noreferrer also implies noopener). If I use a nonexistent named target and leave off the rel="noreferrer" things work as they should.
Bug 1531289 Comment 6 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
OK, so after a bit of digging into this: 1) Comment 1 seems to be about the change in what url shows up in the url bar when the tab is (incorrectly) not closed. It's entirely possible that Honza's changes had to do with that, yes, since they would affect the "final channel URI". It's not at all clear to me what the right behavior there is. It should be spun off to a new bug, because it's not what this original bug is about. I filed bug 1532752 on this. 2) In Firefox 54 (so _before_ Honza's changes) the noreferrer case is broken while the "normal" case works, as I expected based on code inspection. That's also the case in Firefox 65, and is what this bug is about. This part is definitely Document Navigation, but is NOT a regression from Honza's patches, obviously. I'll look into this. 3) In current nightly, the case without noreferrer is _also_ broken. That's because target="_blank" now implies noopener (as of bug 1503681) and it's the noopener bits that are really relevant here (noreferrer also implies noopener). If I use a nonexistent named target and leave off the rel="noreferrer" things work as they should.