Closed Bug 245660 Opened 21 years ago Closed 20 years ago

use nsIImageLoadingContent for copying image location

Categories

(Firefox :: General, defect, P3)

Product:
Firefox
Component:
General
Type:
defect

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: Biesinger, Assigned: Gavin)

References

Details

Attachments

(1 file)

Patch v1
4.52 KB, patch
mconnor
: review+
asa
: approval-aviary1.1a1+
Details | Diff | Splinter Review
Once bug 196380's patch is checked in (later today), firefox can use nsIImageLoadingContent::currentURI to find the URI to copy for the image, instead of trying all kinds of elements that are known to be images. it needs to make sure though that it really is an image that's being displayed (<object> also implements this interface); easiest way is to test currentURI != null.
Assignee: firefox → gavin.sharp
OS: Linux → All
Hardware: PC → All
Version: unspecified → Trunk
Attached patch Patch v1 — — Splinter Review
Port of the patch from bug 245661.
Attachment #174114 - Flags: review?(mconnor)
Whiteboard: [patch-r?]
Status: NEW → ASSIGNED
Priority: -- → P3
*** Bug 242303 has been marked as a duplicate of this bug. ***
Attachment #174114 - Flags: review?(mconnor) → review+
Attachment #174114 - Flags: approval-aviary1.1a?
Whiteboard: [patch-r?] → [patch-r+] [checkin needed]
Comment on attachment 174114 [details] [diff] [review] Patch v1 a=asa
Attachment #174114 - Flags: approval-aviary1.1a? → approval-aviary1.1a+
Whiteboard: [patch-r+] [checkin needed] → [patch-r+][checkin needed][a+]
Checked in by timeless.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Whiteboard: [patch-r+][checkin needed][a+]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: