-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get the barest bones of webdriver integration ready for keeners. #5415
Conversation
Critic review: https://critic.hoppipolla.co.uk/r/4403 This is an external review system which you may optionally use for the code review of your pull request. In order to help critic track your changes, please do not make in-place history rewrites (e.g. via |
This will allow me to file issues for people to work on integration in pieces, and will force us to keep webdriver-rust up to date with rustc upgrades. |
Assigned @pcwalton. |
@@ -226,6 +231,7 @@ pub fn from_cmdline_args(args: &[String]) -> bool { | |||
getopts::optflag("z", "headless", "Headless mode"), | |||
getopts::optflag("f", "hard-fail", "Exit on task failure instead of displaying about:failure"), | |||
getopts::optflagopt("", "devtools", "Start remote devtools server on port", "6000"), | |||
getopts::optflagopt("", "webdriver", "Start remote devtools server on port", "7000"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/devtools/webdriver/
?
r? @pcwalton |
r+ |
📌 Commit 4dfb65e has been approved by |
💔 Test failed - gonk |
@bors-servo: r+ |
📌 Commit 5728799 has been approved by |
@bors-servo: force |
@bors-servo: p=2 |
⌛ Testing commit 5728799 with merge e3f93d5... |
⌛ Testing commit 5728799 with merge f373645... |
⌛ Testing commit 5728799 with merge 1f57c6d... |
Requires mozilla/webdriver-rust#6.