Bug 8658 - Assertion failure in -[WebPluginContainerCheck _isForbiddenFileLoad] (bridge is null) when clicking QuickTime object with href
: Assertion failure in -[WebPluginContainerCheck _isForbiddenFileLoad] (bridge ...
Status: RESOLVED FIXED
Product: WebKit
Classification: Unclassified
Component: Plug-ins
: 420+
: Macintosh Mac OS X 10.4
: P1 Normal
Assigned To: Nobody
: InRadar, Regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-04-29 10:25 PDT by mitz@webkit.org
Modified: 2006-05-01 14:21 PDT (History)
0 users

See Also:


Attachments
Test case (336 bytes, text/html)
2006-04-29 10:26 PDT, mitz@webkit.org
no flags Details
Set the plugin controller's data source (no test yet) (438 bytes, patch)
2006-04-29 11:02 PDT, mitz@webkit.org
no flags Details | Formatted Diff | Diff
Patch, including manual test and change log (3.47 KB, patch)
2006-04-29 12:34 PDT, mitz@webkit.org
darin: review+
Details | Formatted Diff | Diff
orange.mov for the manual test (875 bytes, video/quicktime)
2006-04-29 12:35 PDT, mitz@webkit.org
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description mitz@webkit.org 2006-04-29 10:25:55 PDT
With a debug build of TOT, clicking the QuickTime object in the attached test case gives the following message:

ASSERTION FAILED: bridge (WebKit/WebKit/Plugins/WebPluginContainerCheck.m:95 -[WebPluginContainerCheck _isForbiddenFileLoad])

The WebPluginController's dataSource is NULL.
Comment 1 mitz@webkit.org 2006-04-29 10:26:26 PDT
Created attachment 8032 [details]
Test case
Comment 2 mitz@webkit.org 2006-04-29 11:02:21 PDT
Created attachment 8034 [details]
Set the plugin controller's data source (no test yet)
Comment 3 mitz@webkit.org 2006-04-29 12:34:25 PDT
Created attachment 8036 [details]
Patch, including manual test and change log

I think the only way -[WebPluginContainerCheck _isForbiddenFileLoad] is reached is when a plugin calls a WebPlugInContainer private API, so the test has to use the QT plugin (which knows that API). I tried to make an automated test, but it had the side effect of opening QT Player on success.
Comment 4 mitz@webkit.org 2006-04-29 12:35:30 PDT
Created attachment 8037 [details]
orange.mov for the manual test
Comment 5 Darin Adler 2006-04-29 20:41:39 PDT
Comment on attachment 8036 [details]
Patch, including manual test and change log

Looks right to me. r=me
Comment 6 Tim Omernick 2006-05-01 14:21:01 PDT
This is in Radar as <rdar://problem/4531779>.

Fix landed to TOT, revision 14142.