1402526 - Assertion failure: aOldParent in [@ mozilla::EditorBase::MoveNode]

Status: RESOLVED FIXED

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

Description

[Tyson Smith [:tsmith]]

Attachment: test_case.html

Assertion failure: aOldParent, at /src/obj-firefox/dist/include/mozilla/SelectionState.h:341

#0 mozilla::AutoMoveNodeSelNotify::AutoMoveNodeSelNotify(mozilla::RangeUpdater&, nsINode*, int, nsINode*, int)  /src/obj-firefox/dist/include/mozilla/SelectionState.h:343:5
#1 mozilla::EditorBase::MoveNode(nsIContent*, nsINode*, int)  /src/editor/libeditor/EditorBase.cpp:1825:25
#2 mozilla::HTMLEditRules::WillAlign(mozilla::dom::Selection&, nsTSubstring<char16_t> const&, bool*, bool*)  /src/editor/libeditor/HTMLEditRules.cpp:4853:22
#3 mozilla::HTMLEditRules::WillDoAction(mozilla::dom::Selection*, mozilla::RulesInfo*, bool*, bool*)  /src/editor/libeditor/HTMLEditRules.cpp:666:14
#4 mozilla::HTMLEditor::Align(nsTSubstring<char16_t> const&)  /src/editor/libeditor/HTMLEditor.cpp:2216:24
#5 nsMultiStateCommand::DoCommandParams(char const*, nsICommandParams*, nsISupports*)  /src/editor/composer/nsComposerCommands.cpp:637:10
#6 nsControllerCommandTable::DoCommandParams(char const*, nsICommandParams*, nsISupports*)  /src/dom/commandhandler/nsControllerCommandTable.cpp:162:26
#7 nsBaseCommandController::DoCommandWithParams(char const*, nsICommandParams*)  /src/dom/commandhandler/nsBaseCommandController.cpp:152:25
#8 nsCommandManager::DoCommand(char const*, nsICommandParams*, mozIDOMWindowProxy*)  /src/dom/commandhandler/nsCommandManager.cpp:210:29
#9 nsHTMLDocument::ExecCommand(nsTSubstring<char16_t> const&, bool, nsTSubstring<char16_t> const&, nsIPrincipal&, mozilla::ErrorResult&)  /src/dom/html/nsHTMLDocument.cpp:3357:18
#10 mozilla::dom::HTMLDocumentBinding::execCommand(JSContext*, JS::Handle<JSObject*>, nsHTMLDocument*, JSJitMethodCallArgs const&)  /src/obj-firefox/dom/bindings/HTMLDocumentBinding.cpp:891:21
#11 mozilla::dom::GenericBindingMethod(JSContext*, unsigned int, JS::Value*)  /src/dom/bindings/BindingUtils.cpp:3055:13
#12 js::CallJSNative(JSContext*, bool (*)(JSContext*, unsigned int, JS::Value*), JS::CallArgs const&)  /src/js/src/jscntxtinlines.h:293:15
#13 js::InternalCallOrConstruct(JSContext*, JS::CallArgs const&, js::MaybeConstruct)  /src/js/src/vm/Interpreter.cpp:495:16
#14 InternalCall(JSContext*, js::AnyInvokeArgs const&)  /src/js/src/vm/Interpreter.cpp:540:12
#15 Interpret(JSContext*, js::RunState&)  /src/js/src/vm/Interpreter.cpp:3084:18
#16 js::RunScript(JSContext*, js::RunState&)  /src/js/src/vm/Interpreter.cpp:435:12
#17 js::InternalCallOrConstruct(JSContext*, JS::CallArgs const&, js::MaybeConstruct)  /src/js/src/vm/Interpreter.cpp:513:15
#18 InternalCall(JSContext*, js::AnyInvokeArgs const&)  /src/js/src/vm/Interpreter.cpp:540:12
#19 js::Call(JSContext*, JS::Handle<JS::Value>, JS::Handle<JS::Value>, js::AnyInvokeArgs const&, JS::MutableHandle<JS::Value>)  /src/js/src/vm/Interpreter.cpp:559:10
#20 JS::Call(JSContext*, JS::Handle<JS::Value>, JS::Handle<JS::Value>, JS::HandleValueArray const&, JS::MutableHandle<JS::Value>)  /src/js/src/jsapi.cpp:2965:12
#21 mozilla::dom::EventHandlerNonNull::Call(JSContext*, JS::Handle<JS::Value>, mozilla::dom::Event&, JS::MutableHandle<JS::Value>, mozilla::ErrorResult&)  /src/obj-firefox/dom/bindings/EventHandlerBinding.cpp:260:37
#22 void mozilla::dom::EventHandlerNonNull::Call<nsISupports*>(nsISupports* const&, mozilla::dom::Event&, JS::MutableHandle<JS::Value>, mozilla::ErrorResult&, char const*, mozilla::dom::CallbackObject::ExceptionHandling, JSCompartment*)  /src/obj-firefox/dist/include/mozilla/dom/EventHandlerBinding.h:362:12
#23 mozilla::JSEventHandler::HandleEvent(nsIDOMEvent*)  /src/dom/events/JSEventHandler.cpp:215:12
#24 mozilla::EventListenerManager::HandleEventSubType(mozilla::EventListenerManager::Listener*, nsIDOMEvent*, mozilla::dom::EventTarget*)  /src/dom/events/EventListenerManager.cpp:1112:51
#25 mozilla::EventListenerManager::HandleEventInternal(nsPresContext*, mozilla::WidgetEvent*, nsIDOMEvent**, mozilla::dom::EventTarget*, nsEventStatus*)  /src/dom/events/EventListenerManager.cpp:1283:20
#26 mozilla::EventTargetChainItem::HandleEvent(mozilla::EventChainPostVisitor&, mozilla::ELMCreationDetector&)  /src/dom/events/EventDispatcher.cpp:313:17
#27 mozilla::EventTargetChainItem::HandleEventTargetChain(nsTArray<mozilla::EventTargetChainItem>&, mozilla::EventChainPostVisitor&, mozilla::EventDispatchingCallback*, mozilla::ELMCreationDetector&)  /src/dom/events/EventDispatcher.cpp:462:16
#28 mozilla::EventDispatcher::Dispatch(nsISupports*, nsPresContext*, mozilla::WidgetEvent*, nsIDOMEvent*, nsEventStatus*, mozilla::EventDispatchingCallback*, nsTArray<mozilla::dom::EventTarget*>*)  /src/dom/events/EventDispatcher.cpp:822:9
#29 nsDocumentViewer::LoadComplete(nsresult)  /src/layout/base/nsDocumentViewer.cpp:1081:7
#30 nsDocShell::EndPageLoad(nsIWebProgress*, nsIChannel*, nsresult)  /src/docshell/base/nsDocShell.cpp:7741:21
#31 nsDocShell::OnStateChange(nsIWebProgress*, nsIRequest*, unsigned int, nsresult)  /src/docshell/base/nsDocShell.cpp:7539:7
#32 non-virtual thunk to nsDocShell::OnStateChange(nsIWebProgress*, nsIRequest*, unsigned int, nsresult)  /src/docshell/base/nsDocShell.cpp:7436:13
#33 nsDocLoader::DoFireOnStateChange(nsIWebProgress*, nsIRequest*, int&, nsresult)  /src/uriloader/base/nsDocLoader.cpp:1320:3
#34 nsDocLoader::doStopDocumentLoad(nsIRequest*, nsresult)  /src/uriloader/base/nsDocLoader.cpp:861:14
#35 nsDocLoader::DocLoaderIsEmpty(bool)  /src/uriloader/base/nsDocLoader.cpp:750:9
#36 nsDocLoader::OnStopRequest(nsIRequest*, nsISupports*, nsresult)  /src/uriloader/base/nsDocLoader.cpp:632:5
#37 non-virtual thunk to nsDocLoader::OnStopRequest(nsIRequest*, nsISupports*, nsresult)  /src/uriloader/base/nsDocLoader.cpp:488:14
#38 mozilla::net::nsLoadGroup::RemoveRequest(nsIRequest*, nsISupports*, nsresult)  /src/netwerk/base/nsLoadGroup.cpp:629:28
#39 nsDocument::DoUnblockOnload()  /src/dom/base/nsDocument.cpp:9171:18
#40 nsDocument::UnblockOnload(bool)  /src/dom/base/nsDocument.cpp:9093:9
#41 nsDocument::DispatchContentLoadedEvents()  /src/dom/base/nsDocument.cpp:5597:3
#42 mozilla::detail::RunnableMethodImpl<nsDocument*, void (nsDocument::*)(), true, (mozilla::RunnableKind)0>::Run()  /src/obj-firefox/dist/include/nsThreadUtils.h:1192:13
#43 nsThread::ProcessNextEvent(bool, bool*)  /src/xpcom/threads/nsThread.cpp:1039:14
#44 NS_ProcessNextEvent(nsIThread*, bool)  /src/xpcom/threads/nsThreadUtils.cpp:521:10
#45 mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*)  /src/ipc/glue/MessagePump.cpp:97:21
#46 MessageLoop::RunInternal()  /src/ipc/chromium/src/base/message_loop.cc:326:10
#47 MessageLoop::Run()  /src/ipc/chromium/src/base/message_loop.cc:299:3
#48 nsBaseAppShell::Run()  /src/widget/nsBaseAppShell.cpp:158:27
#49 nsAppStartup::Run()  /src/toolkit/components/startup/nsAppStartup.cpp:288:30
#50 XREMain::XRE_mainRun()  /src/toolkit/xre/nsAppRunner.cpp:4701:22
#51 XREMain::XRE_main(int, char**, mozilla::BootstrapConfig const&)  /src/toolkit/xre/nsAppRunner.cpp:4865:8
#52 XRE_main(int, char**, mozilla::BootstrapConfig const&)  /src/toolkit/xre/nsAppRunner.cpp:4960:21
#53 do_main(int, char**, char**)  /src/browser/app/nsBrowserApp.cpp:236:22
#54 main  /src/browser/app/nsBrowserApp.cpp:309:16
#55 __libc_start_main /build/glibc-bfm8X4/glibc-2.23/csu/../csu/libc-start.c:291
#56 _start (/home/user/workspace/browsers/m-c-1505727316-asan-debug/firefox+0x41e9e4)

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Goes back further than mozregression can bisect :(

Comment 2

[Makoto Kato [:m_kato]]

I will fix this by bug 1425091

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5f74d7b82fe3
Add crashtest. r=me a=test-only

Comment 4

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/5f74d7b82fe3