Web Automation: don't return an error if resizing/moving a window has no effect

[BJ Burg]

The specification has no such demands, and it just causes tests to fail unnecessarily. Remove the error case.

[BJ Burg]

Created attachment 333721 [details]
Proposed Fix

[Tim Horton]

Comment on attachment 333721 [details]
Proposed Fix

What's the testing story? I would have expected the original code to come with a test, but I don't see you removing one?

[BJ Burg]

(In reply to Tim Horton from comment #2)
> Comment on attachment 333721 [details]
> Proposed Fix
> 
> What's the testing story? I would have expected the original code to come
> with a test, but I don't see you removing one?

The Set Window Rect W3C endpoint for safaridriver is blocked by other reviews, but once it's hooked up, I hit this in my branch while running conformance tests. It expects setting window rect to be idempotent without error if you get the window frame then set the window frame to the same rect.

https://github.com/w3c/web-platform-tests/blob/master/webdriver/tests/set_window_rect.py#L261

[WebKit Commit Bot]

Comment on attachment 333721 [details]
Proposed Fix

Clearing flags on attachment: 333721

Committed r228434: <https://trac.webkit.org/changeset/228434>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/37513710>