Hmmm ... it doesn't leak, but I agree it doesn't really look that nice :)

Maybe something like this would be better:

nsCOMPtr<nsIURIContentListener> uriContentListener = 
do_QueryInterface(chrome);
webBrowser->SetParentURIContentListener(uriContentListener);

  -Pelle

Boris Zbarsky wrote:
> Pelle Johnsen wrote:
>> chrome->QueryInterface(nsIURIContentListener::GetIID(), 
>> (void**)&uriContentListener);
>> rv = webBrowser->SetParentURIContentListener(uriContentListener);
> 
> That looks like a leak to me, but does make it look like the reason 
> things didn't work before is because the object went away...
> 
> -Boris
_______________________________________________
dev-embedding mailing list
dev-embedding@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-embedding

Reply via email to