added password encrypt/decrypt stubs, recursive frame/iframe searches us...

[jguglielmi]

Added encrypt/decrypt stubs, recursive frame search capabilities (no more need for selectFrame) and contextMenu support.

[raboof]

A shame about all the whitespace changes - I added #147 to address that separately

[raboof]

Does searchAllFrames impact performance? It seems to make things a bit more complicated.

[raboof]

Where is encryptString actually used? Could we move this to a separate fixture?

[raboof]

It would be nice to have a example/test demonstrating the added features

[jguglielmi]

It is a recursive call, but I hardly noticed anything.
On Jun 28, 2015 4:59 PM, "Arnout Engelen" [email protected] wrote:

Does searchAllFrames impact performance? It seems to make things a bit
more complicated.

—
Reply to this email directly or view it on GitHub
#122 (comment).

[jguglielmi]

Sure. It is used within an OASIS REST call. There is also a decrypt as well.
On Jun 28, 2015 4:59 PM, "Arnout Engelen" [email protected] wrote:

Where is encryptString actually used? Could we move this to a separate
fixture?

—
Reply to this email directly or view it on GitHub
#122 (comment).

[raboof]

I think the encryption/decryption methods should move to their own fixture, outside of Xebium.

[jguglielmi]

Got it!
On Jun 28, 2015 5:50 PM, "Arnout Engelen" [email protected] wrote:

In src/main/java/com/xebia/incubator/xebium/SeleniumDriverFixture.java
#122 (comment):

@@ -456,6 +524,81 @@ private boolean executeDoCommand(final String methodName, final String[] values)
}
}

• private String encryptString(String password) { //encrypt exposed passwords or information stub

I think the encryption/decryption methods should move to their own
fixture, outside of Xebium.

—
Reply to this email directly or view it on GitHub
https://github.com/xebia/Xebium/pull/122/files#r33427425.