[#SRC-381] Enhance capture screenshot to stream screenshot to client

Selenium Remote Control

Enhance capture screenshot to stream screenshot to client

Created: 07/Nov/07 08:41 AM Updated: 12/Aug/08 05:35 PM

Component/s:

Server

Affects Version/s:

None

Fix Version/s:

1.0 beta 1

Original Estimate:

Unknown

Remaining Estimate:

Unknown

Time Spent:

Unknown

File Attachments:

captureScreenshotToString-corechanges.patch (1.0 kb)
2.

captureScreenshotToString-serverchanges.patch (3 kb)

Issue Links:

Relationship

This issue Relates to:
	~~SRC-395~~ Add getScreenshot in additikon to ca...

Description

« Hide

When using selenium rc with selenium grid it's inconvenient to have to save screenshots locally on the grid members for a number of reasons.

Could the capture screenshot method somehow stream the screenshot down to the client. Not sure if selenium rc supports sending binary results back to the client; if not, it could be mime encoded. In my case this would allow the test executing to decide where to store the image (in my case I want to pulish to a cruise artifact dir).

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Rob Dupuis - 07/Nov/07 08:43 AM

Err when I said mime encoded I meant base 64 encoded. Brain wrong.

[ Permlink | « Hide ]

Rob Dupuis - 02/Jan/08 12:47 PM

This should be closed in favour of ~~SRC-395~~ which includes a patch.

[ Permlink | « Hide ]

Erik Beans - 27/May/08 08:53 PM

Here is a patch that modifies selenium to add an API method for capturing a screenshot and returning it as a string which is a base64 encoded png file. This uses the Robot class to capture the current screen. The API method is called captureScreenshotToString.

[ Permlink | « Hide ]

Jason Huggins - 28/May/08 03:06 PM

I closed ~~SRC-395~~ as a duplicate of this issue.

My reasons:
1) This issue came first. :-)
2) The implementation behind this issue is instantly cross-language, returning a Base64 encoded string version of the screenshot, instead of a Java-specific BufferedImage.

Read my comments in ~~SRC-395~~ for more of my commentary. But basically, the good bits in ~~SRC-395~~, should get reworked as enhancements to this patch. That way, all supported client libraries will get the feature at the same time.

[ Permlink | « Hide ]

Jason Huggins - 28/May/08 07:29 PM

Code has been commited to subversion for this patch:
Core: svn revision 2084
RC: svn revision 2306

Remaining task is add some documentation to the wiki on how to use the new feature. Happy captureScreenshottingToFile, everyone!