[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    Bug#28033: Reproductible critical crash in konqueror 2.2 alpha 2
From:       David Faure <david () mandrakesoft ! com>
Date:       2001-07-03 11:31:07
[Download RAW message or body]

On Tuesday 03 July 2001 14:15, Malte Starostik wrote:
> On Dienstag, 3. Juli 2001 12:42, David Faure wrote:
> > On Tuesday 03 July 2001 02:51, Malte Starostik wrote:
> > > On Dienstag, 3. Juli 2001 07:49, Ignacio Monge wrote:
> > > > En Tue, 3 Jul 2001 00:56:05 +0200
> > > >
> > > > Malte Starostik <malte@kde.org> escribió:
> > > > > On Montag, 2. Juli 2001 23:51, ignaciomonge@navegalia.com wrote:
> > > > > > Package: konqueror
> > > > > > Version: 3.0 (using KDE 2.2.0 alpha2)
> > > > > > Severity: normal
> > > > > > Installed from:    Linux-Mandrake 8.1 (Cooker) - 2.2-0.alpha2.4mdk
> > > > > > Compiler:          gcc version 2.96 20000731 (Linux-Mandrake 8.1
> > > > > > 2.96-0.54mdk) OS:                Linux (i686) release 2.4.5-ac22
> > > > > > OS/Compiler notes:
> > > >
> > > > 	Ok, sorry about my horrible english. I'll try to be cleaner.
> > >
> > > No need to be sorry here, I understood your text very well, just not
> > > whether you meant the small preview thumbnails or not :)
> > >
> > > > 	I see this bug doing that:
> > > > 	1.- Open konqueror (it doesn't matter if previews is or not
> > > > activated). 2.- Click on an image file to open in current konqueror,
> > > > just press right click mouse button at the beginnign of image loading
> > > > and open the file menu (you must be quick and do it before the load
> > > > it's completed). 3.- Let the image loads on konqueror (the file menu is
> > > > over the image and you can't select anything on it).
> > > > 	4.- Load completed. Now select one option on menu (now you can do it),
> > > > i.e, Propieties.
> > > > 	5.- Finally, konqueror crashes.
> > >
> > > Here's one with debugging symbols (not for Qt, sorry).
> > > Without any knowledge about the popupmenu-handling nor time to look at it
> > > now, is it that the view has been destroyed but the popup didn't have it
> > > as parent?
> >
> > " press right click mouse button at the beginnign of image loading and open
> > the file menu" There is no file menu in the RMB.... is this about the "New"
> > submenu ? Or do you mean that you open the RMB, and then open the main File
> > menu (thus closing the RMB menu) ?
> LMB click an icon in the icon view and immediately RMB click it (before the 
> icon view is replaced by the part that renders whatever file you clicked).
> 
> Ehm, just noticed: it's not even neccessary to click on a file that will be 
> embedded, a directory icon will do, too.

Now, this is very surprising because KonqMainWindow::slotPopupMenu says

  // the page is currently loading something -> Don't enter a local event loop
  // by launching a popupmenu!
  if ( m_currentView->run() != 0 )
      return;

to fix this very problem.

Oh, but ....... hmmm... when clicking on a directory we have no KonqRun, since
we know the mimetype... I guess we manage to pop the menu up during the
QTimer(0) that happens between BrowserExtension::slotOpenURLRequest()
and BrowserExtension::slotEmitOpenURLRequestDelayed().
ARGL.

Maybe we could add BrowserExtension::hasOpenURLRequest(), that says true
if d->m_delayedURL isn't empty, and disable the popup in that case too....
Can you try this ? (I'm trying to hack koffice and I'm submerged by mail)

-- 
David FAURE, david@mandrakesoft.com, faure@kde.org
http://perso.mandrakesoft.com/~david/, http://www.konqueror.org/
KDE, Making The Future of Computing Available Today

[prev in list] [next in list] [prev in thread] [next in thread]