Closed Bug 240892 Opened 20 years ago Closed 20 years ago

[FIX]select element by click doesn't work for content in an iframe nested in a frame

Categories

(Other Applications :: DOM Inspector, defect, P1)

x86
All
defect

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.8alpha1

People

(Reporter: timeless, Assigned: bzbarsky)

References

()

Details

(Keywords: fixed1.7.5)

Attachments

(1 file)

original testworld is gmail.

note that the simpler tests:
data:text/html,<frameset cols="*,*"><frame src="about:logo"><frame src="about:">
and
data:text/html,<iframe src='data:text/html,<p>Click me</p>'></iframe>

both work fine.

useragents:
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7b) Gecko/20040410
and
bz's linux cvs build MOZ_CO_DATE="Fri Apr 16 19:52:29 CDT 2004"

steps:
1. load url in navigator
2. ctrl-shift-i
3. search>select element by click
4. click 'click me'

expected results:
item is found in domi's document - dom nodes view and stuff

you can replaced steps 1/2 w/
1. load domi
2. enter url in the field and press enter/select inspect
Attached patch FixSplinter Review
Assignee: dom-inspector → bzbarsky
Status: NEW → ASSIGNED
Attachment #146440 - Flags: superreview?(peterv)
Attachment #146440 - Flags: review?(timeless)
Priority: -- → P1
Summary: select element by click doesn't work for content in an iframe nested in a frame → [FIX]select element by click doesn't work for content in an iframe nested in a frame
Target Milestone: --- → mozilla1.8alpha
Comment on attachment 146440 [details] [diff] [review]
Fix

convert the tabs to spaces before committing.
Attachment #146440 - Flags: review?(timeless) → review+
Comment on attachment 146440 [details] [diff] [review]
Fix

What timeless said.
Attachment #146440 - Flags: superreview?(peterv) → superreview+
Checked in, with the tabs changed to spaces.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
*** Bug 260511 has been marked as a duplicate of this bug. ***
Attachment #146440 - Flags: approval1.7.x?
Comment on attachment 146440 [details] [diff] [review]
Fix

a=asa for 1.7.x checkin.
Attachment #146440 - Flags: approval1.7.x? → approval1.7.x+
Fixed on 1.7 branch.
Keywords: fixed1.7.x
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

Created:
Updated:
Size: