386730 - After reloading a page with a single contenteditable element, every element on that page is editable

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Description

[Albert Brand]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a7pre) Gecko/2007070304 Minefield/3.0a7pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a7pre) Gecko/2007070304 Minefield/3.0a7pre

See the testcase URL (from bug #237964). Open the URL, and press reload. Click on a text outside the contenteditable area. Start typing: the text you just typed is appended. Deleting, pasting is also possible. Notice that the caret doesn't appear outside the contenteditable area.

Reproducible: Always



Expected Results:  
Text that isn't set as contenteditable should never be editable, even after a reload.

Comment 1

[Albert Brand]

Also noted: after a page reload, disabling contenteditable through the testcase button doesn't work, which may be closely tied to this bug.

Comment 2

[Peter Van der Beken [:peterv]]

Attachment: v1

Comment 3

[Peter Van der Beken [:peterv]]

Comment on attachment 270886 [details] [diff] [review]
v1

aMakeWholeDocumentEditable should apply to subsequent loads in the same editor.

Comment 4

[Peter Van der Beken [:peterv]]

Attachment: v1.1

Hmm, aMakeWholeDocumentEditable should apply to subsequent loads in the same editor and we shouldn't touch the editable flag on the document on teardown if we didn't touch it when setting up the editor.