1651379 - If clicking a link causes it to disappear, the URL in the corner of the screen doesn't disappear

Status: NEW

(Core :: DOM: Core & HTML, defect, P3)

Description

[vaindil]

Attachment: clicking "load more comments" on old reddit causes the problem to occur as long as the mouse doesn't hover over a link after clicking

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:79.0) Gecko/20100101 Firefox/79.0

Steps to reproduce:

Find a URL that disappears when clicked and click it.

Actual results:

The text in the bottom corner of the browser window with the URL should disappear as well.

Expected results:

The URL text stays open until another link is hovered over. This happens across a few sites, most notably old reddit (see gif) and Discord, but I haven't been able to come up with a simple reproduction that always triggers it. (The issue on old reddit in the gif does reproduce it 100% of the time, but a link on its own on an otherwise blank page doesn't exhibit the problem.)

Comment 1

[Clara Guerrero ( Need Info Brindusa Tot Please)]

Hi,

I can also reproduce and see the void operator text at the bottom on my end as well, until hovering over another link.

I've tested this on windows 10 pro, 80.0a1 (2020-07-13) (64-bit), Release 78.0 (64-bit and Beta 79.0b5 (64-bit)

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/void

Here's the link I used to reproduce: https://old.reddit.com/r/AskReddit/comments/hq6dys/whats_a_dark_secretquestionable_practice_in_your/

I will move this over to a component so developers can take a look over it. If is not the correct component please feel free to change it to an appropriate one.

Thanks for the report.

Best regards, Clara.

Comment 2

[Clara Guerrero ( Need Info Brindusa Tot Please)]

Attachment: beta.webm

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:dao, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 4

[Dão Gottwald [:dao]]

The UI depends on nsIXULBrowserWindow::setOverLink being called (via nsDocShell::OnLeaveLink).

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

When EventStateManager::ContentRemoved() is never called with <a> element, but EventStateManager checks whether the removing content is an <a> element before checking Element::LeaveLink(). In the link of comment 1, <a> element is removed by removing its ancestor <div> element. So, perhaps, the anchor element notified DocShell should be stored anywhere, and it should be handled in EventStateManager::ContentRemoved() when its ancestor is removed. WDYT, smaug?

Comment 8

[Olli Pettay [:smaug][[e-mail]]]

Ah yes, bug 647665 didn't fix this for descendants of the removed element.
I guess EventStateManager::ContentRemoved should in general go through all the descendants.

It might be nice if UnbindFromTree could do this all, but that might be too late, since ContentRemoved is called just before UnbindFromTree.

Comment 9

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

I think that this is a bug of the looks, i.e., I mean that this does not cause any security issue sot that I give normal priority to this bug.