Closed Bug 647665 Opened 13 years ago Closed 12 years ago

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

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla14

People

(Reporter: j, Assigned: mounir)

References

()

Details

Attachments

(1 file)

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
Blocks: 541656
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
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
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
Attached patch Patch v1Splinter Review
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+
Attachment #615276 - Flags: approval-mozilla-central?
Attachment #615276 - Flags: approval-mozilla-central? → approval-mozilla-central+
https://hg.mozilla.org/mozilla-central/rev/23a0762a0dd6
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
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 → ---
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
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: