In Editor, IFRAME is editable instead of page (editor not created)

VERIFIED DUPLICATE of bug 14156

Status

()

Core
Editor
P3
normal
VERIFIED DUPLICATE of bug 14156
17 years ago
15 years ago

People

(Reporter: rubydoo123, Assigned: Kathleen Brade)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: editorbase-, URL)

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
What happens is the file never completes loading in the editor. All the content 
is there, but the throbber is still spinning. This means there is no 
nsIHTMLEditor created for the document. This is probably why the NS_NO_INTERFACE 
error keeps getting thrown from nsEditorShell. After more digging, if the IFRAME 
is commented out, then the page loads and is editable.

Code snippet to use for testing:

<html>
<head><title>test</title></head>
<body>
<p>Here is an iframe:</p>
<iframe frameborder=1 width=20 height=20>IFRAME element</iframe><br>
</body>
</html>

Comment 1

17 years ago
I believe iframes are out... but I may be wrong. Nisheeth are these handled in 
the DOM? 
Assignee: gagan → nisheeth
The HTML spec says (sec. 16.5):

  The information to be inserted inline is designated by the src attribute of
  this element. The contents of the IFRAME element, on the other hand, should
  only be displayed by user agents that do not support frames or are configured
  not to display frames.

IFRAMEs currently work fine, but this bug exists whether the IFRAME has a SRC
attribute or not.  However, the *contents* of the IFRAME do become editable.

Attaching a testcase.

TO REPRODUCE:
 * load attached testcase
 * Go to "File | Edit Page"

ACTUAL RESULTS:
 * The contents of the iframe are editable, the page is not.

EXPECTED RESULTS:
 * The page should be editable.  The contents of the iframe should not (?).

This seems more like an editor bug than a Layout bug.  (Note that I never saw
the throbber keep spinning.  Maybe that is fixed.)
Summary: Web page containing IFRAME, never loads → In Editor, IFRAME is editable instead of page
Created attachment 9592 [details]
testcase

Comment 4

17 years ago
Beth, the page loads fine now, but as David says, instead of the page becoming 
editable, the IFRAME inside the page becomes editable.  This seems like a editor 
specific issue, so re-assigning this back to you.
Assignee: nisheeth → beppe
Component: Networking → Editor
(Reporter)

Comment 5

17 years ago
asking Simon if he has any ideas as to why the page becomes uneditable, but the 
IFRAME is editable.
Assignee: beppe → sfraser

Comment 6

17 years ago
Yes, I do. We'll make the editor on the last IFRAME that is loaded, blowing away 
previous editors in the process. Fixing this requires some thought.

Testcase, please?
Status: NEW → ASSIGNED

Comment 7

17 years ago
Actually, I think fixing this will requires at least some of the work for 
embedding to be complete (bug 35639).
(Reporter)

Comment 8

17 years ago
Simon -- tests cases are already in here, see the sample code in the file and 
David Baron attached a test case
(Reporter)

Comment 9

17 years ago
setting to m18
Target Milestone: --- → M18

Comment 10

17 years ago
cc: valeski, dougt for embedding info. To fix this bug, I need to be able to hang 
an editor off a docShell (or equivalent), which leads us down the embedding path. 
Have you guys started down that path yet?
Depends on: 34477
Target Milestone: M18 → M20
(Reporter)

Comment 11

17 years ago
added correctness keyword and bringing it back to m19
Keywords: correctness
Target Milestone: M20 → M19
(Reporter)

Comment 12

17 years ago
moving to future per review with bij and beppe
Target Milestone: M19 → Future

Comment 13

17 years ago
*** Bug 60869 has been marked as a duplicate of this bug. ***

Comment 14

16 years ago
Editor issue.
Assignee: sfraser → brade
Status: ASSIGNED → NEW

Updated

15 years ago
QA Contact: tever → sujay
(Assignee)

Comment 15

15 years ago
In a recent build I now see this:
page loads fine but it not editable; clicking on links in the iframe load those
pages
Keywords: nsbeta1
OS: Windows 98 → All
Whiteboard: editorbase
Target Milestone: Future → ---
(Assignee)

Comment 16

15 years ago
nsbeta1+
editorbase- for now but this is subject to change later
Keywords: nsbeta1 → nsbeta1+
Summary: In Editor, IFRAME is editable instead of page → In Editor, IFRAME is editable instead of page (editor not created)
Whiteboard: editorbase → editorbase-
(Assignee)

Comment 17

15 years ago
add http://news.bbc.co.uk/ as a testcase (iframe with java)
(Assignee)

Comment 18

15 years ago
duping

*** This bug has been marked as a duplicate of 14156 ***
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → DUPLICATE
(Reporter)

Comment 19

15 years ago
verified dup
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.