Bug 146834 - [EFL] http/tests/media/media-stream/disconnected-frame-already.html is crashing after r185903
Summary: [EFL] http/tests/media/media-stream/disconnected-frame-already.html is crashi...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-10 04:08 PDT by Hunseop Jeong
Modified: 2015-08-03 08:38 PDT (History)
6 users (show)

See Also:


Attachments
Crash log (2.95 MB, image/png)
2015-07-10 10:35 PDT, Hunseop Jeong
no flags Details
Patch (15.90 KB, patch)
2015-07-30 08:39 PDT, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hunseop Jeong 2015-07-10 04:08:00 PDT
Expected : Passed
Actual : Crash
Comment 1 youenn fablet 2015-07-10 05:36:53 PDT
Can you provide a debug crash log?
Comment 2 Hunseop Jeong 2015-07-10 06:34:33 PDT
I'm out of office.
I'll upload the crash log.
Comment 3 Hunseop Jeong 2015-07-10 10:35:03 PDT
Created attachment 256587 [details]
Crash log

EFL port doesn't work the copy&paste :( so I upload the capture of log.
Comment 4 youenn fablet 2015-07-10 14:09:37 PDT
Thanks.

It may be that I went too far in removing one check within webkitGetUserMedia.

I am not sure why this is happening for EFL port and not the other ports. I am away next two weeks, I can study that at my return.

In the meantime, you can look at the first removed check in the revision within NavigatorUserMedia::webkitGetUserMedia.
Comment 5 youenn fablet 2015-07-30 08:39:11 PDT
Created attachment 257835 [details]
Patch
Comment 6 Eric Carlson 2015-07-31 09:06:07 PDT
Comment on attachment 257835 [details]
Patch

Nice refactoring!
Comment 7 WebKit Commit Bot 2015-08-03 08:38:43 PDT
Comment on attachment 257835 [details]
Patch

Clearing flags on attachment: 257835

Committed r187737: <http://trac.webkit.org/changeset/187737>
Comment 8 WebKit Commit Bot 2015-08-03 08:38:47 PDT
All reviewed patches have been landed.  Closing bug.