I'm trying to keep the options in sync with the selenium-server options, and I'd rather not have to maintain a mess of compat options.

--jason


On Jan 17, 2009, at 4:13 PM, Dan Tran wrote:

To make it some what backward compatible

can you set default value for singleWindow is "true"?  (
multipleWindows default value was false, means singleWidows is true)

Thanks

-D

On Sat, Jan 17, 2009 at 12:55 AM,  <[email protected]> wrote:
Revision 8720 Author user57 Date 2009-01-17 02:55:22 -0600 (Sat, 17 Jan
2009)

Log Message

(MSELENIUM-41) Changedc multiWindow to singleWindow

Modified Paths

trunk/mojo/selenium-maven-plugin/src/main/groovy/org/codehaus/mojo/ selenium/StartServerMojo.groovy

Diff

Modified:
trunk/mojo/selenium-maven-plugin/src/main/groovy/org/codehaus/mojo/ selenium/StartServerMojo.groovy
(8719 => 8720)

---
trunk/mojo/selenium-maven-plugin/src/main/groovy/org/codehaus/mojo/ selenium/StartServerMojo.groovy 2009-01-17
08:31:46 UTC (rev 8719)
+++
trunk/mojo/selenium-maven-plugin/src/main/groovy/org/codehaus/mojo/ selenium/StartServerMojo.groovy 2009-01-17
08:55:22 UTC (rev 8720)
@@ -97,14 +97,13 @@
    String verifyBrowser

    /**
- * Flag to control if we start Selenium RC in multiWindow mode or not.
The multiWindow mode
- * is useful for applications using frames/iframes which otherwise
cannot be tested as the
- * same window is used for displaying both the Selenium tests and the
AUT.
+ * Puts you into a mode where the test web site executes in a frame.
This mode should only be
+     * selected if the application under test does not use frames.
     *
-     * @parameter expression="${multiWindow}" default-value="false"
-     * @since 1.0-beta-3
+     * @parameter expression="${singleWindow}" default-value="false"
+     * @since 1.0-rc-1
     */
-    boolean multiWindow
+    boolean singleWindow

    /**
     * Sets the browser mode (e.g. "*iexplore" for all sessions).
@@ -320,8 +319,8 @@
                    arg(value: '-debug')
                }

-                if (multiWindow) {
-                    arg(value: '-multiWindow')
+                if (singleWindow) {
+                    arg(value: '-singleWindow')
                }

                if (forcedBrowserMode) {

________________________________

To unsubscribe from this list please visit:

http://xircles.codehaus.org/manage_email

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email




---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email


Reply via email to