Bug 118882 - [EFL][WK2] test of ewk_context_new failed in test_ewk2_context
Summary: [EFL][WK2] test of ewk_context_new failed in test_ewk2_context
Status: RESOLVED LATER
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 115332
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-18 20:27 PDT by Donggwan Kim
Modified: 2013-07-24 19:01 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Donggwan Kim 2013-07-18 20:27:24 PDT
ASSERTION FAILED: !result
/Source/WTF/wtf/ThreadingPthreads.cpp(344) : void WTF::Mutex::lock()
1   0x7ffe433017b8 WTFCrash
2   0x7ffe4333345d WTF::Mutex::lock()
3   0x7ffe42f361f2 WTF::Locker<WTF::Mutex>::Locker(WTF::Mutex&)
4   0x7ffe42470e6e WorkQueue::performWork()
5   0x7ffe4247156c WorkQueue::workQueueThread(WorkQueue*)
6   0x7ffe433112f1
7   0x7ffe43332d88
8   0x7ffe417a7e9a
9   0x7ffe40fbeccd clone
Segmentation fault (core dumped)
Comment 1 Sergio Correia (qrwteyrutiyoup) 2013-07-18 20:54:11 PDT
Byungwoo has fixed some problems with the WorkQueue in https://bugs.webkit.org/show_bug.cgi?id=115332, still under review. With his patch applied, this crash goes away, although we still get the following message:

Source/WebKit2/UIProcess/API/efl/ewk_context.cpp:330 ewk_context_additional_plugin_path_set() safety check failed: path == NULL

Let's get Chris to take another look at that patch :)
Comment 2 Donggwan Kim 2013-07-18 23:17:49 PDT
I will check this bug again after it is fixed.
Comment 3 Chris Dumez 2013-07-18 23:25:35 PDT
(In reply to comment #1)
> Byungwoo has fixed some problems with the WorkQueue in https://bugs.webkit.org/show_bug.cgi?id=115332, still under review. With his patch applied, this crash goes away, although we still get the following message:
> 
> Source/WebKit2/UIProcess/API/efl/ewk_context.cpp:330 ewk_context_additional_plugin_path_set() safety check failed: path == NULL
> 
> Let's get Chris to take another look at that patch :)

As I said earlier, an owner needs to sign off on it before I can review because the patch is touching generic wk2 files.
Comment 4 Sergio Correia (qrwteyrutiyoup) 2013-07-19 04:35:20 PDT
(In reply to comment #3)
> (In reply to comment #1)
> > Byungwoo has fixed some problems with the WorkQueue in https://bugs.webkit.org/show_bug.cgi?id=115332, still under review. With his patch applied, this crash goes away, although we still get the following message:
> > 
> > Source/WebKit2/UIProcess/API/efl/ewk_context.cpp:330 ewk_context_additional_plugin_path_set() safety check failed: path == NULL
> > 
> > Let's get Chris to take another look at that patch :)
> 
> As I said earlier, an owner needs to sign off on it before I can review because the patch is touching generic wk2 files.

Ah, I see. Apologies, then. I saw your comment in the other bug, but I was under the impression a reviewer would need to do some kind of informal review and then an owner would sign it off :)