74056 - Lots of assertions when leaving page with images

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[Simon Fraser [no longer active]]

With imglib2, I'm hitting an assertion lots of times when leaving a page with
many images.

To repro:
1. Build with PSM (not sure if this is necessary)
2. Go to the URL above
3. Click on one of the main links.

As we tear down the page you are leaving, we hit an assertion here:

###!!! ASSERTION: GetURI failed: 'NS_SUCCEEDED(res)', file
nsSecureBrowserUIImpl.cpp, line 359

many times (once of each image on the page?).

Assertion stack:

 Calling chain using A6/R1 links
  Back chain  ISA  Caller
  00000000    PPC  3ED40E34  
  0F668230    PPC  3ED206D4  main+001AC
  0F6681D0    PPC  3ED1DA50  main1(int, char**, nsISupports*)+008D0
  0F667F60    PPC  3D19CEF4  nsAppShellService::Run()+00054
  0F667F10    PPC  3B500E80  nsAppShell::Run()+0004C
  0F667ED0    PPC  3B501774  nsMacMessagePump::DoMessagePump()+00044
  0F667E80    PPC  3B501FD0  nsMacMessagePump::DispatchEvent(int,
EventRecord*)+001B0
  0F667E30    PPC  3B52147C  Repeater::DoRepeaters(const EventRecord&)+0003C
  0F667DE0    PPC  3B4E3400  nsMacNSPREventQueueHandler::RepeatAction(const
EventRecord&)+000
14
  0F667DA0    PPC  3B4E36AC  nsMacNSPREventQueueHandler::ProcessPLEventQueue()+00244
  0F667D00    PPC  3D2D7C64  nsEventQueueImpl::ProcessPendingEvents()+00068
  0F667C90    PPC  3D349128  PL_ProcessPendingEvents+000AC
  0F667C40    PPC  3D34937C  PL_HandleEvent+00054
  0F667C00    PPC  3CA224B4  HandlePLEvent(OnLinkClickEvent*)+00014
  0F667BC0    PPC  3CA22550  OnLinkClickEvent::HandleEvent()+00050
  0F667B80    PPC  3CA22F2C 
HandleLinkClickEvent__10nsWebShellFP10nsIContent10nsLinkVerbPCwP
CwP14nsIInputStreamP14nsIInputStream+002DC
  0F667A30    PPC  3CA02278  nsDocShell::InternalLoad(nsIURI*, nsIURI*,
nsISupports*, int, in
t, const char*, nsIInputStream*, nsIInputStream*, unsigned int, nsISHEntry*)+00368
  0F667900    PPC  3C9E9FF8  nsDocShell::StopLoad()+00094
  0F667880    PPC  3B715628  nsURILoader::Stop(nsISupports*)+001A0
  0F6677F0    PPC  3B71C33C  nsDocLoaderImpl::Stop()+00164
  0F667780    PPC  3BBBB684  nsLoadGroup::Cancel(unsigned int)+001B4
  0F6676B0    PPC  3BBBC740 
RemoveRequest__11nsLoadGroupFP10nsIRequestP11nsISupportsUiPCw+00
434
  0F667590    PPC  3B71D058 
OnStopRequest__15nsDocLoaderImplFP10nsIRequestP11nsISupportsUiPC
w+000C0
  0F667510    PPC  3B71D9D8  nsDocLoaderImpl::doStopURLLoad(nsIRequest*,
unsigned int)+00094
  0F6674B0    PPC  3B71FDB4  nsDocLoaderImpl::FireOnStateChange(nsIWebProgress*,
nsIRequest*,
 int, unsigned int)+003A4
  0F6673E0    PPC  3BA4B564 
nsSecureBrowserUIImpl::OnStateChange(nsIWebProgress*, nsIRequest
*, int, unsigned int)+003C8
  0F667070    PPC  3D2AD818  nsDebug::Assertion(const char*, const char*, const
char*, int)+0
005C


Stack for the reason we assert:

...
  09367D10    PPC  07DE3A5C 
OnStopRequest__15nsDocLoaderImplFP10nsIRequestP11nsISupportsUiPC
w+00088
  09367CB0    PPC  07DE4394  nsDocLoaderImpl::doStopURLLoad(nsIRequest*,
unsigned int)+00094
  09367C50    PPC  07DE6648  nsDocLoaderImpl::FireOnStateChange(nsIWebProgress*,
nsIRequest*,
 int, unsigned int)+003A4
  09367B80    PPC  0957AAD8 
nsSecureBrowserUIImpl::OnStateChange(nsIWebProgress*, nsIRequest
*, int, unsigned int)+00308
  09367810    PPC  0C571558  DummyChannel::GetURI(nsIURI**)+0005C

We assert because mOwner on the dummy channel is NULL.

Comment 1

[Stuart Parmenter]

fixed