Bug 38950

Summary: 19 media tests are crashing on Windows Release
Product: WebKit Reporter: Jer Noble <jer.noble>
Component: MediaAssignee: Jer Noble <jer.noble>
Status: RESOLVED FIXED    
Severity: Normal CC: rniwa
Priority: P2 Keywords: InRadar, LayoutTestFailure, PlatformOnly, Regression
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows 7   
URL: http://build.webkit.org/builders/Windows%20Release%20%28Tests%29/builds/12469
Attachments:
Description Flags
QTMovieWin.vcproj: link against JavaScriptCore instead of WTF.
none
QTMovieWin.vcproj: link against JavaScriptCore instead of WTF. darin: review+

Description Jer Noble 2010-05-11 16:54:44 PDT
Since r59001, 19 media tests have been crashing on Windows Release.
Comment 1 Jer Noble 2010-05-11 17:14:16 PDT
Created attachment 55780 [details]
QTMovieWin.vcproj: link against JavaScriptCore instead of WTF.

Linking directly against WTF.lib instead of indirectly through JavaScriptCore.lib causes free and malloc to duplicated inside QTMovieWin.dll.  This causes a malloc/free mismatch which shows up only in Release builds.
Comment 2 Jer Noble 2010-05-11 17:15:10 PDT
This originally showed up in r59001 when QTMovie, QTMovieGWorld, and QTMovieVisualContext were changed to inherit from WTF::RefCounted, so that each could be contained in a RefPtr.
Comment 3 Jer Noble 2010-05-11 17:17:08 PDT
Testing 93 test cases.
media .............................................................................................
48.38s total testing time

all 93 test cases succeeded
Comment 4 Jer Noble 2010-05-11 17:20:58 PDT
Created attachment 55782 [details]
QTMovieWin.vcproj: link against JavaScriptCore instead of WTF.

This time, with a ChangeLog.
Comment 5 Jer Noble 2010-05-11 17:43:04 PDT
Committed r59190: <http://trac.webkit.org/changeset/59190>
Comment 6 Ryosuke Niwa 2011-11-14 23:02:57 PST
Fixed the change log entry in http://trac.webkit.org/changeset/100243.