Closed Bug 162461 Opened 22 years ago Closed 22 years ago

inLayoutUtils::GetContainerFor is suboptimal

Categories

(Other Applications :: DOM Inspector, defect)

Product:
Other Applications
Component:
DOM Inspector
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: sicking, Assigned: sicking)

Details

Attachments

(1 file)

patch to fix
1.74 KB, patch
bzbarsky
: review+
hewitt
: superreview+
Details | Diff | Splinter Review 
inLayoutUtils::GetContainerFor could be much neater implemented using
nsIDOMWindowInternal::GetFrameElement.

Patch coming up
bz, hewitt: reviews?
Assignee: hewitt → sicking
Comment on attachment 95085 [details] [diff] [review]
patch to fix

r=bzbarsky
Attachment #95085 - Flags: review+
Comment on attachment 95085 [details] [diff] [review]
patch to fix

how convenient

sr=hewitt
Attachment #95085 - Flags: superreview+
checked in. Thanks for reviews
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Product: Core → Other Applications
QA Contact: timeless → dom-inspector
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: