screenshots from bug report in 0.14 are weird

[totaam]

Issue migrated from trac ticket # 637

component: client | priority: major | resolution: fixed | keywords: win32

2014-08-17 19:26:16: eblade created the issue

---

http://s27.postimg.org/yqsl3m7mr/Screenshot.jpg

That doesn't really look much of anything like my screen. It looks like it captured one monitor, put it in the middle of another monitor, failed to capture most of the windows, and the windows it did capture are split in two.

[totaam]

2014-08-17 19:26:25: eblade uploaded file Screenshot.jpg (2184.0 KiB)

[totaam]

2014-08-17 19:27:08: eblade changed component from android to client

[totaam]

2014-08-17 19:27:43: eblade commented

---

Windows 8.1, triple monitor, left is 1080x1920, middle is 1920x1200, right is 1920x1080

[[Image(windows8-triplemonitor-bugreport.jpg)]]

[totaam]

2014-08-18 01:52:46: totaam uploaded file windows8-triplemonitor-bugreport.jpg (167.7 KiB)

smaller version of the screenshot we can inline in the ticket

[totaam]

2014-08-18 01:53:58: totaam commented

---

Do you have the ability to take a more correct screenshot using some other tool so that I can have a better idea of what things are meant to look like?

[totaam]

2014-08-18 02:08:32: eblade uploaded file screenshot-2.jpg (542.3 KiB)

[totaam]

2014-08-18 02:11:44: eblade commented

---

what it looks like happened, is all the area that the left monitor takes up above the other two got filled with garbage .. the area below the right two got filled with black.. the right hand monitor's image got swapped to the left.. and the open windows got sliced up and moved all over. In the original screenshot, the Opera window with the Xpra wiki in it should have been full screen, but it's showing bits of other windows that were below it, and I think the window that is sliced in two parts there was actually on the right hand monitor instead of the center.

[totaam]

2014-08-18 07:27:01: totaam changed status from new to assigned

[totaam]

2014-08-18 07:27:01: totaam changed owner from antoine to totaam

[totaam]

2014-08-18 07:27:01: totaam commented

---

Sounds to me like the screen capture code in GTK has bugs on windows 8 and/or multi monitors setup.

Either way, this should be dealt with before too long as part of #389.
Another solution would be the GTK3 port (#90) - assuming that screen capture works better there... which is not a given.

[totaam]

2014-08-18 16:22:36: eblade commented

---

Probably multi-monitors. GTK is really screwey in that respect, especially once you start mixing radically different resolutions.

The successful screenshot that I attached was taken using windows Ctrl-PrintScreen and then pasted to GIMP. GIMP itself could not grab the whole thing, and would only grab a single display and a tiny piece (about 30 px or so) of the right or the left depending on which monitor the window was on when I hit the Create->From Screenshot button.

[totaam]

2015-04-24 13:11:01: antoine commented

---

I'll be able to deal with this once I get a MS Windows test system setup (works OK in virtualbox)

[totaam]

2015-07-22 13:24:15: totaam changed status from assigned to new

[totaam]

2015-07-22 13:24:15: totaam changed owner from totaam to eblade

[totaam]

2015-07-22 13:24:15: totaam commented

---

r10002 probably fixes this, I now have a Windows PC to test on and enough monitors but I am still waiting for the delivery of the HDMI cable I need for testing!

I would like to backport this change because it also drastically improves the performance of the win32 shadow server.

Beta build available for testing: [http://xpra.org/beta/windows/]

@afarr / @eblade: does this work for you?
Bonus points for testing the [/wiki/ShadowServer shadow server]:

xpra_cmd shadow :0 --bind-tcp=0.0.0.0:10000

Then connect to this window box as normal on port 10000.

[totaam]

2015-07-23 01:33:30: afarr commented

---

Testing on a windows 8.1 client 0.16.0 r10002 with two displays against a fedora 20 server 0.16.0 9919 (there weren't pieces of this on the server side, I hope).

Client display set up:

2015-07-22 16:52:43,976  desktop size is 5120x2160 with 1 screen(s):
2015-07-22 16:52:43,976   '1\WinSta-Default' (1354x571 mm - DPI: 96x96) workarea: 5120x2120
2015-07-22 16:52:43,976     DISPLAY1 3840x2160 at 1280x0 (621x341 mm - DPI: 157x160) workarea: 3840x2120
2015-07-22 16:52:43,976     DISPLAY2 1280x720 (597x336 mm - DPI: 54x54) workarea: 1280x680

When I have windows on Display 1, bug report screenshot only captures Display 1. Moving the xpra windows to Display 2, the bug report window opens with no content. All subsequent attempts to create a bug report also produce a blank window - until I disconnect/reconnect.

Upon reconnecting, I am able to use the bug report tool to take another screen shot, but it again takes only a shot of Display 1 and again brings up an empty window if I try to open the bug report tool a second time, despite having thought to move the xpra windows to Display 2.

I'll attach xpra info (used your windows client but our fedora server build) and a screenshot of the little greyed out bug report window.

[totaam]

2015-07-23 01:35:21: afarr uploaded file ticket637-second-bug-report-screenshot-try_bug-report-window-has-no-content.PNG (288.8 KiB)

screenshot of blank bug report window

[totaam]

2015-07-23 01:36:30: afarr uploaded file ticket637_xpra-info_bug-report-window-blank.txt (110.6 KiB)

xpra info with blank bug report tool window

[totaam]

2015-07-23 01:38:38: afarr commented

---

Blank bug report window next to an xpra firefox window.

[[Image(ticket637-second-bug-report-screenshot-try_bug-report-window-has-no-content.PNG)]]

[totaam]

2015-07-24 11:07:34: antoine changed owner from eblade to afarr

[totaam]

2015-07-24 11:07:34: antoine commented

---

So, I got an HDMI cable in the end and it turns out that the Pillow screen capture is also buggy: Python PIL library— Is there a way to select which screen ImageGrab.grab() grabs in a multi-monitor setup?

So r10026 implements screen capture using native pywin32 calls instead (+fixup in r10053). This is suitable for backporting.
The shadow_server.py can be used for testing the screen capture quickly and it will save a file named "screenshot.png" in the current working directory.

With r10027 onwards, the bug report tool can also be accessed from the launcher.
As for the blank bug report window, this has nothing to do with multiple monitors or win32, it just shows up as blank whenever you open it for the second time around from the same process. This is fixed in r10029. (will also backport)

@afarr / @eblade: can you still break it somehow? (new beta uploaded)

[totaam]

2015-08-01 02:03:19: afarr commented

---

Well, aside from trying to declare that copying the entire contents of the bug report generated to clipboard failing to allow the pasting of a screen shot into a .rtf file, I couldn't seem to break it.

Screenshot with multi-dispaly desktop captured both displays, with black space to indicate the portion of the smaller window which had no content (while putting the whole "desktop" into a box).

Bug tool window appeared as expected and functioned as expected three times in a row. Not too shabby. Can probably close, unless eblade can come up with a way to break it.

[totaam]

2015-08-19 06:21:12: antoine changed status from new to closed

[totaam]

2015-08-19 06:21:12: antoine set resolution to fixed

[totaam]

2015-08-19 06:21:12: antoine commented

---

Looks fine to me. Closing.
I will re-test once my new cables are delivered (so I can test with multiple displays)

[totaam]

2015-08-19 16:06:31: eblade uploaded file Screenshot.png (2890.7 KiB)

screenshot from win10 confirming fix

[totaam]

2015-08-19 16:08:51: eblade commented

---

Looks good from here now. Attached verification (screenshot.png) that it's working from Win10.

[totaam]

2015-08-19 16:09:47: eblade commented

---

oh, the left monitor seems to extend slightly below the area where the taskbar is. nothing major, though.