Skip to content

Commit

Permalink
Merge r220786 - [GTK][WPE] WebKitAutomationSession should set session…
Browse files Browse the repository at this point in the history
… client to nullptr on dispose

https://bugs.webkit.org/show_bug.cgi?id=175564

Reviewed by Carlos Alberto Lopez Perez.

WebAutomationSession expects its client to be nullptr in the destructor.

* UIProcess/API/glib/WebKitAutomationSession.cpp:
(webkitAutomationSessionDispose):
  • Loading branch information
carlosgcampos committed Aug 17, 2017
1 parent caa7220 commit ea30e69
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 0 deletions.
12 changes: 12 additions & 0 deletions Source/WebKit/ChangeLog
@@ -1,3 +1,15 @@
2017-08-15 Carlos Garcia Campos <cgarcia@igalia.com>

[GTK][WPE] WebKitAutomationSession should set session client to nullptr on dispose
https://bugs.webkit.org/show_bug.cgi?id=175564

Reviewed by Carlos Alberto Lopez Perez.

WebAutomationSession expects its client to be nullptr in the destructor.

* UIProcess/API/glib/WebKitAutomationSession.cpp:
(webkitAutomationSessionDispose):

2017-08-15 Carlos Garcia Campos <cgarcia@igalia.com>

WebDriver: timeout when JavaScript alert is shown in onload handler
Expand Down
2 changes: 2 additions & 0 deletions Source/WebKit/UIProcess/API/glib/WebKitAutomationSession.cpp
Expand Up @@ -199,6 +199,8 @@ static void webkitAutomationSessionDispose(GObject* object)
{
WebKitAutomationSession* session = WEBKIT_AUTOMATION_SESSION(object);

session->priv->session->setClient(nullptr);

if (session->priv->applicationInfo) {
webkit_application_info_unref(session->priv->applicationInfo);
session->priv->applicationInfo = nullptr;
Expand Down

0 comments on commit ea30e69

Please sign in to comment.