Bug 304626 - Get rid of remaining sync dbus calls to kactivitymanagerd
Summary: Get rid of remaining sync dbus calls to kactivitymanagerd
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: activities (show other bugs)
Version: git master
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
: 322406 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-08-05 16:38 UTC by Thomas Lübking
Modified: 2018-10-04 19:46 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Lübking 2012-08-05 16:38:22 UTC
void Workspace::activityPopupAboutToShow()
{
.....
    foreach (const QString &id, openActivities_) {
^^^^^^^^^ LOOP!!
        KActivities::Info activity(id);
        QString name = activity.name();
SYNC DBUS CALL ^^^^^^^^^^^^^^^^
        name.replace('&', "&&");
        action = activity_popup->addAction(KIcon(activity.icon()), name);
                                     SYNC DBUS CALL ^^^^^^^^^^^^^^^^
...
}

ie. for eg. 4 activities we timeout 4 minutes - i'd like to say sth. about hiding such in library functions that look like member accesses....
Comment 1 Thomas Lübking 2013-07-15 19:34:44 UTC
*** Bug 322406 has been marked as a duplicate of this bug. ***
Comment 2 Martin Flöser 2015-01-13 11:57:56 UTC
that should be fixed in 5.x, shouldn't it?
Comment 3 David Edmundson 2018-10-04 19:46:19 UTC
Ran with Q_DBUS_BLOCKING_CALL_MAIN_THREAD_WARNING_MS=0 kwin_x11 --replace      

not an issue anymore.