Opened 10 years ago

Closed 9 years ago

Last modified 9 years ago

#8071 closed defect (fixed)

Screenshot from http://<frontendhost>:6547/MythFE/GetScreenShot does not work

Reported by: jason.chambers@… Owned by: beirdo
Priority: minor Milestone: 0.24
Component: MythTV - General Version: 0.22-fixes
Severity: medium Keywords:
Cc: Ticket locked: no

Description

I noticed that [16532] (referenced in 0.22 release notes) mentions a screenshot can be captured via http://<frontendhost>:6547/MythFE/GetScreenShot.

However when trying this on my frontends my browser just gets an empty body (no content, but also no errors).

But I can successfully take screenshot using the "jump snapshot" command when connecting via telnet to port 6546, as well as by assigning and using a key on my remote.

The frontend log file shows some warnings and errors (e.g. "QPixmap: It is not safe to use pixmaps outside the GUI thread") when connecting to http://<frontendhost>:6547/MythFE/GetScreenShot is attached.

(NB some of the messages are captured on stderr, not stdout).

Attachments (1)

mythfrontend.log (1014 bytes) - added by jason.chambers@… 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by jason.chambers@…

Attachment: mythfrontend.log added

comment:1 Changed 10 years ago by stuartm

Milestone: unknown0.24
Owner: changed from Isaac Richards to stuartm
Status: newaccepted

comment:2 Changed 10 years ago by stuartm

Milestone: 0.24unknown
Owner: changed from stuartm to greg
Status: acceptedassigned

comment:3 Changed 9 years ago by beirdo

Owner: changed from greg to beirdo

comment:4 Changed 9 years ago by beirdo

Resolution: fixed
Status: assignedclosed

(In [26240]) Create a signal/slot in the MythMainWindow? to handle remote screen shots. As QPixmap doesn't want to capture from anything but the GUI thread, it is necessary to actually create it in the GUI thread. I used Qt::BlockingQueuedConnection? to block the WorkerThread? until the screen shot has completed.

Also changed it to use PNG rather than JPG for the remote screen shots as this can be helpful for people like themers without lossy encoding.

Fixes #8071

comment:5 Changed 9 years ago by beirdo

Milestone: unknown0.24

comment:6 Changed 9 years ago by beirdo

(In [26242]) In the frontend remote screen shots:

  • changed optional Width and Height parameters to width and height respectively (lower case)
  • added an optional format parameter to indicate image format, defaulting to png. Supports jpg, png, gif.

Refs #8071

Note: See TracTickets for help on using tickets.