Title: [229002] releases/WebKitGTK/webkit-2.20/Source/WebKit
- Revision
- 229002
- Author
- [email protected]
- Date
- 2018-02-26 03:14:53 -0800 (Mon, 26 Feb 2018)
Log Message
Merge r228434 - Web Automation: don't return an error if resizing/moving a window has no effect
https://bugs.webkit.org/show_bug.cgi?id=182742
Reviewed by Tim Horton.
* UIProcess/Automation/WebAutomationSession.cpp:
(WebKit::WebAutomationSession::resizeWindowOfBrowsingContext):
(WebKit::WebAutomationSession::moveWindowOfBrowsingContext):
Unify the behavior for all ports. Don't raise an error if the command was
idempotent (by mistake or not). It should not be an error to re-maximize a window.
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.20/Source/WebKit/ChangeLog (229001 => 229002)
--- releases/WebKitGTK/webkit-2.20/Source/WebKit/ChangeLog 2018-02-26 11:14:46 UTC (rev 229001)
+++ releases/WebKitGTK/webkit-2.20/Source/WebKit/ChangeLog 2018-02-26 11:14:53 UTC (rev 229002)
@@ -1,3 +1,16 @@
+2018-02-13 Brian Burg <[email protected]>
+
+ Web Automation: don't return an error if resizing/moving a window has no effect
+ https://bugs.webkit.org/show_bug.cgi?id=182742
+
+ Reviewed by Tim Horton.
+
+ * UIProcess/Automation/WebAutomationSession.cpp:
+ (WebKit::WebAutomationSession::resizeWindowOfBrowsingContext):
+ (WebKit::WebAutomationSession::moveWindowOfBrowsingContext):
+ Unify the behavior for all ports. Don't raise an error if the command was
+ idempotent (by mistake or not). It should not be an error to re-maximize a window.
+
2018-02-20 Brian Burg <[email protected]>
ASSERT under WebAutomationSession::setProcessPool() when running W3C test suite a second time
Modified: releases/WebKitGTK/webkit-2.20/Source/WebKit/UIProcess/Automation/WebAutomationSession.cpp (229001 => 229002)
--- releases/WebKitGTK/webkit-2.20/Source/WebKit/UIProcess/Automation/WebAutomationSession.cpp 2018-02-26 11:14:46 UTC (rev 229001)
+++ releases/WebKitGTK/webkit-2.20/Source/WebKit/UIProcess/Automation/WebAutomationSession.cpp 2018-02-26 11:14:53 UTC (rev 229002)
@@ -342,19 +342,7 @@
return callback->sendSuccess();
page->setWindowFrame(newFrame);
-
-#if PLATFORM(GTK)
callback->sendSuccess();
-#else
- // If nothing changed at all, it's probably fair to report that something went wrong.
- // (We can't assume that the requested frame size will be honored exactly, however.)
- page->getWindowFrameWithCallback([callback = WTFMove(callback), originalFrame](WebCore::FloatRect updatedFrame) {
- if (originalFrame == updatedFrame)
- callback->sendFailure(STRING_FOR_PREDEFINED_ERROR_NAME_AND_DETAILS(InternalError, "The window size was expected to have changed, but did not."));
- else
- callback->sendSuccess();
- });
-#endif
});
#endif
}
@@ -390,19 +378,7 @@
return callback->sendSuccess();
page->setWindowFrame(newFrame);
-
-#if PLATFORM(GTK)
callback->sendSuccess();
-#else
- // If nothing changed at all, it's probably fair to report that something went wrong.
- // (We can't assume that the requested frame size will be honored exactly, however.)
- page->getWindowFrameWithCallback([callback = WTFMove(callback), originalFrame](WebCore::FloatRect updatedFrame) {
- if (originalFrame == updatedFrame)
- callback->sendFailure(STRING_FOR_PREDEFINED_ERROR_NAME_AND_DETAILS(InternalError, "The window position was expected to have changed, but did not."));
- else
- callback->sendSuccess();
- });
-#endif
});
#endif
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
