1843361 - Crash in [@ mozilla::EditorBase::InitializeSelection]

Status: ASSIGNED

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

Description

[Sean Feng [:sefeng]]

Crash report: https://crash-stats.mozilla.org/report/index/8718934d-e183-46d4-ac90-5977b0230703

MOZ_CRASH Reason: MOZ_DIAGNOSTIC_ASSERT(IsTextEditor())

Top 10 frames of crashing thread:

0  xul.dll  mozilla::EditorBase::InitializeSelection  editor/libeditor/EditorBase.cpp:5447
1  xul.dll  mozilla::EditorBase::PostCreateInternal  editor/libeditor/EditorBase.cpp:422
2  xul.dll  mozilla::HTMLEditor::PostCreate  editor/libeditor/HTMLEditor.cpp:491
3  xul.dll  nsEditingSession::SetupEditorOnWindow  editor/composer/nsEditingSession.cpp:435
4  xul.dll  nsEditingSession::MakeWindowEditable  editor/composer/nsEditingSession.cpp:165
5  xul.dll  mozilla::dom::Document::EditingStateChanged  dom/base/Document.cpp:6201
6  xul.dll  mozilla::dom::Document::DeferredContentEditableCountChange  dom/base/Document.cpp:6361
7  xul.dll  mozilla::dom::DeferredContentEditableCountChangeEvent::Run  dom/base/Document.cpp:6301
8  xul.dll  nsContentUtils::RemoveScriptBlocker  dom/base/nsContentUtils.cpp:6042
8  xul.dll  mozilla::dom::Document::EndUpdate  dom/base/Document.cpp:7967

Comment 1

[Sean Feng [:sefeng]]

I wonder if this is related to Bug 1817723, so needinfo myself to take a look.

Comment 2

[Sean Feng [:sefeng]]

Actually this is not :)

Masayuki, thoughts on the assertion?

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(away 4/27 - 5/6)]

I don't have concrete idea how to reproduce it, but I could fix the assertion failure.
(But perhaps, we should add other MOZ_DIAGNOSTIC_ASSERTs when the odd state is created.)