Opened 14 years ago

Closed 14 years ago

Last modified 14 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@… 14 years ago.

Download all attachments as: .zip

Change History (7)

Changed 14 years ago by jason.chambers@…

Attachment: mythfrontend.log​ added

comment:1 Changed 14 years ago by stuartm

Milestone: unknown → 0.24
Owner: changed from Isaac Richards to stuartm
Status: new → accepted

comment:2 Changed 14 years ago by stuartm

Milestone: 0.24 → unknown
Owner: changed from stuartm to greg
Status: accepted → assigned

comment:3 Changed 14 years ago by beirdo

Owner: changed from greg to beirdo

comment:4 Changed 14 years ago by beirdo

Resolution: → fixed
Status: assigned → closed

(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 14 years ago by beirdo

Milestone: unknown → 0.24

comment:6 Changed 14 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.