@jgraham r?

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

src/command.rs, line 573 [r1] (raw file):

impl Parameters for WindowPositionParameters {
    fn from_json(body: &Json) -> WebDriverResult<WindowPositionParameters> {
        let data = try_opt!(body.as_object(), ErrorStatus::UnknownError, "Message body was not an object");

I think these should be broken over multiple lines.

src/response.rs, line 22 [r1] (raw file):

impl WebDriverResponse {
    pub fn to_json_string(self) -> String {
        let s = match self {

Please revert. The () are unneeded.

src/response.rs, line 87 [r1] (raw file):

impl WindowPositionResponse {
    pub fn new(x: u64, y: u64) -> WindowPositionResponse {
        WindowPositionResponse { x: x, y: y }

Use multiple lines.

Comments from Reviewable

Review status: 1 of 3 files reviewed at latest revision, 3 unresolved discussions.

src/response.rs, line 87 [r1] (raw file):

Comments from Reviewable

Review status: 1 of 3 files reviewed at latest revision, 3 unresolved discussions.

src/command.rs, line 573 [r1] (raw file):

src/response.rs, line 22 [r1] (raw file):

Comments from Reviewable

Review status: 1 of 3 files reviewed at latest revision, 3 unresolved discussions.

src/response.rs, line 87 [r1] (raw file):

Comments from Reviewable

@jgraham Mind taking another look?

@jgraham ping

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable