The default bug view has changed. See this FAQ.

Url bar at bottom left does not disappear if link is removed from dom

RESOLVED FIXED in mozilla14

Status

()

Core
DOM
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Jan Gerber, Assigned: mounir)

Tracking

Trunk
mozilla14
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/5.0 (X11; Linux i686; rv:2.0) Gecko/20100101 Firefox/4.0
Build Identifier: Mozilla/5.0 (X11; Linux i686; rv:2.0) Gecko/20100101 Firefox/4.0

If you hover over a link, the url shows up at the bottom right,
if now the link element is removed from the dom(or the div around it), the url preview is not removed and sticks around forever.


Reproducible: Always

Steps to Reproduce:
1. open http://firefogg.org/j/bugs/linkpreview.html
2. move mouse over 'link'
Actual Results:  
url preview stays around at bottom

Expected Results:  
url preview should be removed if the link that is previewed is removed from the dom

Updated

6 years ago
Blocks: 541656

Comment 1

6 years ago
I have reproduced the alleged bug in:
Mozilla/5.0 (Windows NT 6.0; rv:2.2a1pre) Gecko/20110403 Firefox/4.2a1pre
Mozilla/5.0 (X11; Linux x86_64; rv:2.2a1pre) Gecko/20110403 Firefox/4.2a1pre

Comment 2

6 years ago
Mozilla/5.0 (X11; Linux i686; rv:2.2a1pre) Gecko/20110407 Firefox/4.2a1pre

Confirming issue on latest trunk.

Present ever since Firefox 3.6.16
Status: UNCONFIRMED → NEW
Ever confirmed: true
Same Issue/Dupe of 629024?
Like that one it's repro'able against 3.5. and thus not related to Bug 541656.
It's just more obvious now ;-).
No longer blocks: 541656
OS: Linux → All
Version: unspecified → Trunk
Blocks: 629024
Duplicate of this bug: 669686
The front-end actually just reacts to setOverLink() calls from the core. That ends up being triggered by nsGenericElement::LeaveLink in the link-hover case - seems like we should make it fire when links are removed from the DOM, somehow.
Component: General → DOM
Product: Firefox → Core
QA Contact: general → general
(Assignee)

Comment 6

5 years ago
Created attachment 615276 [details] [diff] [review]
Patch v1
Assignee: nobody → mounir
Status: NEW → ASSIGNED
Attachment #615276 - Flags: review?(bzbarsky)
Comment on attachment 615276 [details] [diff] [review]
Patch v1

r=me
Attachment #615276 - Flags: review?(bzbarsky) → review+
(Assignee)

Updated

5 years ago
Attachment #615276 - Flags: approval-mozilla-central?
Attachment #615276 - Flags: approval-mozilla-central? → approval-mozilla-central+
(Assignee)

Comment 8

5 years ago
https://hg.mozilla.org/mozilla-central/rev/23a0762a0dd6
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
(Assignee)

Comment 9

5 years ago
Backed out because of excess of stupidity. I put some stuff that shouldn't be in the patch or even pushed the wrong patch (didn't check in details).

Backout rev:
https://hg.mozilla.org/mozilla-central/rev/f9fbee492a5c
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 10

5 years ago
Comment on attachment 615276 [details] [diff] [review]
Patch v1

Pushed to inbound.
Attachment #615276 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/01cf4fb5a8c3
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Duplicate of this bug: 629024
You need to log in before you can comment on or make changes to this bug.