Bug 109456 – [WK2][EFL][WTR] Regression(r141836): WTR crashes on exit

• Component: WebKit EFL
• Status: RESOLVED FIXED

[Mikhail Pozdnyakov]

SSIA.
#1  0x00007fffeff96d33 in WebKit::WebPageProxy::close (this=0x68e2a0) at /media/ssd/WebKit/Source/WebKit2/UIProcess/WebPageProxy.cpp:459
#2  0x00007ffff0160a36 in WebKit::WebView::~WebView (this=0xe96550, __in_chrg=<optimized out>) at /media/ssd/WebKit/Source/WebKit2/UIProcess/efl/WebView.cpp:54
#3  0x00007ffff0160a84 in WebKit::WebView::~WebView (this=0xe96550, __in_chrg=<optimized out>) at /media/ssd/WebKit/Source/WebKit2/UIProcess/efl/WebView.cpp:55
#4  0x00007fffefee5224 in WTF::ThreadSafeRefCounted<WebKit::APIObject>::deref (this=0xe96558) at /media/ssd/WebKit/Source/WTF/wtf/ThreadSafeRefCounted.h:137
#5  0x00007fffeff27a76 in WKRelease (typeRef=0xe96550) at /media/ssd/WebKit/Source/WebKit2/Shared/API/c/WKType.cpp:47
#6  0x0000000000435ea3 in WTR::PlatformWebView::~PlatformWebView (this=0x496d90, __in_chrg=<optimized out>)
    at /media/ssd/WebKit/Tools/WebKitTestRunner/efl/PlatformWebViewEfl.cpp:80
#7  0x0000000000424c7f in WTF::deleteOwnedPtr<WTR::PlatformWebView> (ptr=0x496d90) at /media/ssd/WebKit/Source/WTF/wtf/OwnPtrCommon.h:63
#8  0x0000000000423adf in WTF::OwnPtr<WTR::PlatformWebView>::~OwnPtr (this=0x7fffffffdd48, __in_chrg=<optimized out>)
    at /media/ssd/WebKit/Source/WTF/wtf/OwnPtr.h:63
#9  0x000000000041d947 in WTR::TestController::~TestController (this=0x7fffffffdce0, __in_chrg=<optimized out>)
    at /media/ssd/WebKit/Tools/WebKitTestRunner/TestController.cpp:115
#10 0x0000000000436547 in main (argc=2, argv=0x7fffffffdef8) at /media/ssd/WebKit/Tools/WebKitTestRunner/efl/main.cpp:53

[Mikhail Pozdnyakov]

Ahh.. several different problem lead to exact same crash..!! (at the same time )

[Mikhail Pozdnyakov]

Created attachment 187615 [details]
patch

Kill two crashes with one patch

[Benjamin Poulain]

Comment on attachment 187615 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=187615&action=review

> Source/WebKit2/ChangeLog:8
> +        WebView dtor now consideres the situation if its WebPageProxy

dtor->destructor.
consideres->considers

> Source/WebKit2/UIProcess/efl/WebView.cpp:54
> +    // Can be closed already from outside the class.

This comments does not add enough information to be useful.
You should either remove the comment, or full explain the events that leads to such situation.

> Tools/WebKitTestRunner/efl/main.cpp:52
> +        // Prefer the not installed web and plugin processes.

What??

[Mikhail Pozdnyakov]

Comment on attachment 187615 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=187615&action=review

>> Source/WebKit2/UIProcess/efl/WebView.cpp:54
>> +    // Can be closed already from outside the class.
> 
> This comments does not add enough information to be useful.
> You should either remove the comment, or full explain the events that leads to such situation.

ok.

>> Tools/WebKitTestRunner/efl/main.cpp:52
>> +        // Prefer the not installed web and plugin processes.
> 
> What??

that's not my comment, changing/removing it wasn't within scope of this patch :)

Thanks for review!

[Mikhail Pozdnyakov]

Created attachment 187804 [details]
patch v2

Took comments from Benjamin into consideration.

[Kenneth Rohde Christiansen]

Comment on attachment 187804 [details]
patch v2

View in context: https://bugs.webkit.org/attachment.cgi?id=187804&action=review

> Tools/WebKitTestRunner/efl/main.cpp:49
> +        // Test controller has own ptr containing WebView and Webview must be deleted

Spelling Webview* -> WebView

[Mikhail Pozdnyakov]

Created attachment 188062 [details]
patch v3

Fixed misspelling in the comment.

[WebKit Review Bot]

Comment on attachment 188062 [details]
patch v3

Clearing flags on attachment: 188062

Committed r142748: <http://trac.webkit.org/changeset/142748>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.