21820 - [DOGFOOD]'Edit Page' results in non-editable page

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[Simon Fraser [no longer active]]

If you do 'Edit Page', the resulting page in editor has a number of problems:
1. Sometimes, content is duplicated
2. Part of all of the page is not editable.

Comment 1

[Simon Fraser [no longer active]]

These problems go away if you turn off the timer-based content notifications with
the pref

user_pref("content.notify.ontimer", false);

Comment 2

[Simon Fraser [no longer active]]

cc: vidur

Comment 3

[Simon Fraser [no longer active]]

Adding dogfood, pls consider for PDT+ status

Comment 4

[Simon Fraser [no longer active]]

Mmm, this problem does not seem to occur when loading the page through a file:
(or chrome:) URL, only http. Even the simplest page with at least one tag in the
body will show the problem.

Comment 5

[Simon Fraser [no longer active]]

More info: the content model looks good, so the problem is in content -> frame
construction.

Comment 6

[Simon Fraser [no longer active]]

This looks very much like it's caused by bug 21459. What's happening is that the
OnEndDocumentLoad is being called too early, so the editor gets instantiated
before the document is fully reflowed. Because of this, the editor is making the
bogus <BR> node, which confuses the content sink and causes teh doubled content.

Comment 7

[Simon Fraser [no longer active]]

Good, rick's fix for 21459 fixed this too.

Comment 8

[sujay]

verified in 12/16 build.