1322444 - I see this assertion when running dom/url/tests/test_bloburl_location.html

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Andrea Marchesini [:baku]]

MEMORY STAT | vsize 2956MB | residentFast 182MB
[Parent 30372] ###!!! ASSERTION: Only call this with blob URIs: 'IsBlobURI(aURI)', file /home/baku/Sources/m/foobar/src/dom/file/nsHostObjectProtocolHandler.cpp, line 889
#01: NS_GetBlobForBlobURI(nsIURI*, mozilla::dom::BlobImpl**) (/home/baku/Sources/m/foobar/src/dom/file/nsHostObjectProtocolHandler.cpp:889 (discriminator 1))
#02: nsHostObjectURI::Deserialize(mozilla::ipc::URIParams const&) (/home/baku/Sources/m/foobar/src/dom/file/nsHostObjectURI.cpp:156)
#03: mozilla::ipc::DeserializeURI(mozilla::ipc::URIParams const&) (/home/baku/Sources/m/foobar/src/ipc/glue/URIUtils.cpp:120)
#04: mozilla::dom::ContentParent::RecvSetURITitle(mozilla::ipc::URIParams const&, nsString const&) (/home/baku/Sources/m/foobar/src/dom/ipc/ContentParent.cpp:3155)
#05: mozilla::dom::PContentParent::OnMessageReceived(IPC::Message const&) (/home/baku/Sources/m/foobar/build/ipc/ipdl/PContentParent.cpp:4085)
#06: mozilla::ipc::MessageChannel::DispatchAsyncMessage(IPC::Message const&) (/home/baku/Sources/m/foobar/src/ipc/glue/MessageChannel.cpp:1750)
#07: mozilla::ipc::MessageChannel::DispatchMessage(IPC::Message&&) (/home/baku/Sources/m/foobar/src/ipc/glue/MessageChannel.cpp:1690)
#08: mozilla::ipc::MessageChannel::RunMessage(mozilla::ipc::MessageChannel::MessageTask&) (/home/baku/Sources/m/foobar/src/ipc/glue/MessageChannel.cpp:1572)
#09: mozilla::ipc::MessageChannel::MessageTask::Run() (/home/baku/Sources/m/foobar/src/ipc/glue/MessageChannel.cpp:1598)
#10: nsThread::ProcessNextEvent(bool, bool*) (/home/baku/Sources/m/foobar/src/xpcom/threads/nsThread.cpp:1213)
#11: NS_ProcessNextEvent(nsIThread*, bool) (/home/baku/Sources/m/foobar/src/xpcom/glue/nsThreadUtils.cpp:381)
#12: mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) (/home/baku/Sources/m/foobar/src/ipc/glue/MessagePump.cpp:96)
#13: MessageLoop::RunInternal() (/home/baku/Sources/m/foobar/src/ipc/chromium/src/base/message_loop.cc:233)
#14: MessageLoop::RunHandler() (/home/baku/Sources/m/foobar/src/ipc/chromium/src/base/message_loop.cc:226)
#15: MessageLoop::Run() (/home/baku/Sources/m/foobar/src/ipc/chromium/src/base/message_loop.cc:204)
#16: nsBaseAppShell::Run() (/home/baku/Sources/m/foobar/src/widget/nsBaseAppShell.cpp:158)
#17: nsAppStartup::Run() (/home/baku/Sources/m/foobar/src/toolkit/components/startup/nsAppStartup.cpp:283)
#18: XREMain::XRE_mainRun() (/home/baku/Sources/m/foobar/src/toolkit/xre/nsAppRunner.cpp:4485)
#19: XREMain::XRE_main(int, char**, nsXREAppData const*) (/home/baku/Sources/m/foobar/src/toolkit/xre/nsAppRunner.cpp:4618)
#20: XRE_main (/home/baku/Sources/m/foobar/src/toolkit/xre/nsAppRunner.cpp:4709)
#21: do_main(int, char**, char**, nsIFile*) (/home/baku/Sources/m/foobar/src/browser/app/nsBrowserApp.cpp:328)
#22: main (/home/baku/Sources/m/foobar/src/browser/app/nsBrowserApp.cpp:461)
#23: __libc_start_main (/build/glibc-Qz8a69/glibc-2.23/csu/../csu/libc-start.c:325)

Comment 1

[Andrea Marchesini [:baku]]

Attachment: blobUrl_assertions.patch

The reason why we have this assertion is because, ages ago, the check of the type of url was based on the scheme 'blob/media/etc'. Now we only use 'blob' and we check the internal type stored in the DataInfo data struct. If the blobURL doesn't exist, we have this assertion triggered.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a335dd5b87e5
No assertions needed for NS_GetBlobForBlobURI, r=qdot

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a335dd5b87e5