mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fdo#67011: Run Display::Yield through KDEXLib::Yield.

Browse Source 
Drag'n'Drop is handled in a second thread, which tries to "yield
the display" while the main Qt thread probably is already
yielding. Both need the YieldMutex, which freezes the application
until the D'n'D thread times out.

Trying to yield the display throught the application yield
results in a recursive loop.

So this catches and breaks the recursion, but just inside the Qt
thread, so other processes can "yield on the display", instead
of simply disabling the Display::Yield.

Change-Id: Ifba91aa89fe5b0a89cc94820935dc996a065112f
Reviewed-on: https://gerrit.libreoffice.org/6750
Tested-by: Jan-Marek Glogowski <glogow@fbihome.de>
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
This commit is contained in:
Jan-Marek Glogowski
2013-11-21 12:40:57 +01:00
parent 8d74243a65
commit 95f60222e7
1 changed files with 16 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
vcl/unx/kde4/KDESalDisplay.cxx
View File

@@ -25,6 +25,8 @@
#include <assert.h>
#include <unx/saldata.hxx>
#include <qthread.h>
SalKDEDisplay* SalKDEDisplay::selfptr = NULL;
SalKDEDisplay::SalKDEDisplay( Display* pDisp )
@@ -48,18 +50,26 @@ SalKDEDisplay::~SalKDEDisplay()
void SalKDEDisplay::Yield()
{
if( DispatchInternalEvent() )
// We yield the display throught the main Qt thread.
// Actually this Yield may call the Display::Yield, which results in an
// unlimited cycle.
static bool break_cyclic_yield_recursion = false;
bool is_qt_gui_thread = ( qApp->thread() == QThread::currentThread() );
if( DispatchInternalEvent() || break_cyclic_yield_recursion )
return;
if( is_qt_gui_thread )
break_cyclic_yield_recursion = true;
DBG_ASSERT( static_cast<SalYieldMutex*>(GetSalData()->m_pInstance->GetYieldMutex())->GetThreadId() ==
osl::Thread::getCurrentIdentifier(),
"will crash soon since solar mutex not locked in SalKDEDisplay::Yield" );
XEvent event;
XNextEvent( pDisp_, &event );
if( checkDirectInputEvent( &event ))
return;
qApp->x11ProcessEvent( &event );
static_cast<KDEXLib*>(GetXLib())->Yield( true, false );
if( is_qt_gui_thread )
break_cyclic_yield_recursion = false;
}
// HACK: When using Qt event loop, input methods (japanese, etc.) will get broken because