Bug 243015 - Rdesktop window is not embedded into KRDC
Summary: Rdesktop window is not embedded into KRDC
Status: RESOLVED UPSTREAM
Alias: None
Product: krdc
Classification: Applications
Component: RDP (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Urs Wolfer
URL:
Keywords:
: 243180 247368 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-06-28 11:10 UTC by Pascal d'Hermilly
Modified: 2010-11-07 19:47 UTC (History)
5 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 Pascal d'Hermilly 2010-06-28 11:10:30 UTC
Version:           unspecified (using Devel) 
OS:                Linux

I start a connection to a windows server I usually connect to using krdc. However with kdrc 4.4.90 the rdesktop window is not embedded into KRDC. is is in a standalone window with no window-decoration (close, maximze,drag etc.)

Reproducible: Always




OS: Linux (x86_64) release 2.6.32-22-generic
Compiler: cc
Comment 1 Paulo Dias 2010-06-29 00:21:09 UTC
I can confirm this too, rdp sessions worked has a charm in 4.4, since beta2 (and now rc1), rdp sessions dont stay embedded in the krdc window, and full screen ones just stay full screen without decoration but missplaced and dont respect the topbar commands.

even if i resize or move the krdc window, the rdp window stays detached.

very annoying since krdc is a must for me. feel free to contact me for further testing.
Comment 2 Martin Sandsmark 2010-06-29 00:23:02 UTC
Are you using Qt 4.7? If so, it's a known regression in Qt.
Comment 3 Martin Sandsmark 2010-06-29 00:25:20 UTC
FWIW: http://bugreports.qt.nokia.com/browse/QTBUG-10809
Comment 4 Paulo Dias 2010-06-29 00:34:46 UTC
Yeah qt4 4.7 = 4.7.0~beta1+git20100522.

k, so just need to wait for a qt patch.. good to know. tks.
Comment 5 Pascal d'Hermilly 2010-06-29 09:27:05 UTC
I'm just using the standard Kubuntu beta packages. (http://www.kubuntu.org/node/157)
However a --version parameter shows that they indeed use QT 4.7

pascal@pascal-t61:~$ krdc --version
Qt: 4.7.0                                                                                                                                                            
KDE Udviklingsplatform: 4.4.90 (KDE 4.4.90 (KDE 4.5 RC1))                                                                                                            
KRDC: 4.4.90 (KDE 4.4.90 (KDE 4.5 RC1))
Comment 6 Urs Wolfer 2010-06-29 17:52:11 UTC
I'll let this bug open until it's fixed in Qt. Otherwise we will receive tons of reports for this issue...
Comment 7 Urs Wolfer 2010-06-29 17:52:19 UTC
*** Bug 243180 has been marked as a duplicate of this bug. ***
Comment 8 Luis Zaldivar 2010-06-30 01:52:57 UTC
Strangely enough, VNC works ok.
Comment 9 Søren Holm 2010-08-03 16:14:46 UTC
Apparently it has been fixed for a while.

commit 9a1b0695277a3864b42d082095962f8742cdcf04
Author: Denis Dzyubenko <denis.dzyubenko@nokia.com>
Date:   Thu Jul 1 12:00:05 2010

    Fixed QX11EmbedContainer.
    
    Enforce the creation of the native window id for the focusproxy widget inside
    the container to make sure we won't get a CreateNotify event to try to embed
    the focusproxy itself.
    
    Task-number: QTBUG-10809
    Reviewed-by: Bradley T. Hughes

diff --git a/src/gui/kernel/qx11embed_x11.cpp b/src/gui/kernel/qx11embed_x11.cpp
index b527e72..9f1b1f8 100644
--- a/src/gui/kernel/qx11embed_x11.cpp
+++ b/src/gui/kernel/qx11embed_x11.cpp
@@ -1070,6 +1070,7 @@ QX11EmbedContainer::QX11EmbedContainer(QWidget *parent)
     d->focusProxy = new QWidget(this);
     d->focusProxy->setAttribute(Qt::WA_NativeWindow);
     d->focusProxy->setAttribute(Qt::WA_DontCreateNativeAncestors);
+    d->focusProxy->createWinId();
     d->focusProxy->setGeometry(-1, -1, 1, 1);
 
     // We need events from the window (activation status) and
Comment 10 karaluh 2010-08-11 15:09:00 UTC
*** Bug 247368 has been marked as a duplicate of this bug. ***
Comment 11 Urs Wolfer 2010-11-07 19:47:20 UTC
Fixed in Qt.