<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>8658</bug_id>
          
          <creation_ts>2006-04-29 10:25:55 -0700</creation_ts>
          <short_desc>Assertion failure in -[WebPluginContainerCheck _isForbiddenFileLoad] (bridge is null) when clicking QuickTime object with href</short_desc>
          <delta_ts>2006-05-01 14:21:01 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Plug-ins</component>
          <version>420+</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, Regression</keywords>
          <priority>P1</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>mitz</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>40717</commentid>
    <comment_count>0</comment_count>
    <who name="">mitz</who>
    <bug_when>2006-04-29 10:25:55 -0700</bug_when>
    <thetext>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&apos;s dataSource is NULL.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>40718</commentid>
    <comment_count>1</comment_count>
      <attachid>8032</attachid>
    <who name="">mitz</who>
    <bug_when>2006-04-29 10:26:26 -0700</bug_when>
    <thetext>Created attachment 8032
Test case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>40721</commentid>
    <comment_count>2</comment_count>
      <attachid>8034</attachid>
    <who name="">mitz</who>
    <bug_when>2006-04-29 11:02:21 -0700</bug_when>
    <thetext>Created attachment 8034
Set the plugin controller&apos;s data source (no test yet)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>40728</commentid>
    <comment_count>3</comment_count>
      <attachid>8036</attachid>
    <who name="">mitz</who>
    <bug_when>2006-04-29 12:34:25 -0700</bug_when>
    <thetext>Created attachment 8036
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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>40729</commentid>
    <comment_count>4</comment_count>
      <attachid>8037</attachid>
    <who name="">mitz</who>
    <bug_when>2006-04-29 12:35:30 -0700</bug_when>
    <thetext>Created attachment 8037
orange.mov for the manual test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>40764</commentid>
    <comment_count>5</comment_count>
      <attachid>8036</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-04-29 20:41:39 -0700</bug_when>
    <thetext>Comment on attachment 8036
Patch, including manual test and change log

Looks right to me. r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>40815</commentid>
    <comment_count>6</comment_count>
    <who name="Tim Omernick">timo</who>
    <bug_when>2006-05-01 14:21:01 -0700</bug_when>
    <thetext>This is in Radar as &lt;rdar://problem/4531779&gt;.

Fix landed to TOT, revision 14142.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>8032</attachid>
            <date>2006-04-29 10:26:26 -0700</date>
            <delta_ts>2006-04-29 10:26:26 -0700</delta_ts>
            <desc>Test case</desc>
            <filename>8658 reduction.html</filename>
            <type>text/html</type>
            <size>336</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">PHA+Q2xpY2sgdGhlIGJ1dHRvbjwvcD4KPEVNQkVEIHNyYz0iaHR0cDovL21vdmllcy5hcHBsZS5j
b20vdHJhaWxlcnMvaW1hZ2VzL2hkX2J0bjJfNDgwcC5tb3YiIHdpZHRoPSI2NyIgaGVpZ2h0PSIy
NyIgcGx1Z2luc3BhZ2U9Imh0dHA6Ly93d3cuYXBwbGUuY29tL3F1aWNrdGltZS9kb3dubG9hZC8i
IGJnY29sb3I9IkZGRkZGRiIgY29udHJvbGxlcj0iZmFsc2UiIGhyZWY9Imh0dHA6Ly9pbWFnZXMu
YXBwbGUuY29tL21vdmllcy9zb255X3BpY3R1cmVzL21vbnN0ZXJfaG91c2UvbW9uc3Rlcl9ob3Vz
ZS10bHIxXzQ4MHAubW92IiB0YXJnZXQ9IlF1aWNrVGltZVBsYXllciI+PC9FTUJFRD4K
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>8034</attachid>
            <date>2006-04-29 11:02:21 -0700</date>
            <delta_ts>2006-04-29 12:35:30 -0700</delta_ts>
            <desc>Set the plugin controller&apos;s data source (no test yet)</desc>
            <filename>8658_r0.patch</filename>
            <type>text/plain</type>
            <size>438</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC9XZWJWaWV3L1dlYkhUTUxWaWV3Lm0KPT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViS2l0
L1dlYlZpZXcvV2ViSFRNTFZpZXcubQkocmV2aXNpb24gMTQxMjQpCisrKyBXZWJLaXQvV2ViVmll
dy9XZWJIVE1MVmlldy5tCSh3b3JraW5nIGNvcHkpCkBAIC0zMTQzLDYgKzMxNDMsNyBAQCBkb25l
OgogewogICAgIEFTU0VSVCghX3ByaXZhdGUtPmRhdGFTb3VyY2UpOwogICAgIF9wcml2YXRlLT5k
YXRhU291cmNlID0gW2RhdGFTb3VyY2UgcmV0YWluXTsKKyAgICBbX3ByaXZhdGUtPnBsdWdpbkNv
bnRyb2xsZXIgc2V0RGF0YVNvdXJjZTpkYXRhU291cmNlXTsKIH0KIAogLSAodm9pZClkYXRhU291
cmNlVXBkYXRlZDooV2ViRGF0YVNvdXJjZSAqKWRhdGFTb3VyY2UK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>8036</attachid>
            <date>2006-04-29 12:34:25 -0700</date>
            <delta_ts>2006-04-29 20:41:39 -0700</delta_ts>
            <desc>Patch, including manual test and change log</desc>
            <filename>8658_r1.patch</filename>
            <type>text/plain</type>
            <size>3550</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiAxNDEyNCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTQgQEAKKzIwMDYtIyMtIyMgIE1pdHogUGV0dGVsICA8b3BlbmRhcndpbi5vcmdA
bWl0enBldHRlbC5jb20+CisKKyAgICAgICAgLSBtYW51YWwgdGVzdCBmb3IgaHR0cDovL2J1Z3pp
bGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNnaT9pZD04NjU4CisgICAgICAgICAgQXNzZXJ0
aW9uIGZhaWx1cmUgaW4gLVtXZWJQbHVnaW5Db250YWluZXJDaGVjayBfaXNGb3JiaWRkZW5GaWxl
TG9hZF0KKyAgICAgICAgICAoYnJpZGdlIGlzIG51bGwpIHdoZW4gY2xpY2tpbmcgUXVpY2tUaW1l
IG9iamVjdCB3aXRoIGhyZWYKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K
KworICAgICAgICAqIG1hbnVhbC10ZXN0cy9wbHVnaW4tY29udHJvbGxlci1kYXRhc291cmNlLmh0
bWw6IEFkZGVkLgorICAgICAgICAqIG1hbnVhbC10ZXN0cy9yZXNvdXJjZXMvb3JhbmdlLm1vdjog
QWRkZWQuCisKIDIwMDYtMDQtMjggIERhdmlkIEh5YXR0ICA8aHlhdHRAYXBwbGUuY29tPgogCiAg
ICAgICAgIE1ha2UgaW1hZ2UgYnVmZmVyIHNpemVzIGluIHRoZSBsb2FkZXIgbW9yZSBhY2N1cmF0
ZSwgc2luY2Ugd2Uga25vdyB3ZSBzdG9yZQpJbmRleDogV2ViQ29yZS9tYW51YWwtdGVzdHMvcGx1
Z2luLWNvbnRyb2xsZXItZGF0YXNvdXJjZS5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvbWFu
dWFsLXRlc3RzL3BsdWdpbi1jb250cm9sbGVyLWRhdGFzb3VyY2UuaHRtbAkocmV2aXNpb24gMCkK
KysrIFdlYkNvcmUvbWFudWFsLXRlc3RzL3BsdWdpbi1jb250cm9sbGVyLWRhdGFzb3VyY2UuaHRt
bAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwyMyBAQAorPCFET0NUWVBFIEhUTUwgUFVCTElDICIt
Ly9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIgorICAgICAgICAiaHR0cDovL3d3dy53My5vcmcvVFIv
aHRtbDQvc3RyaWN0LmR0ZCI+Cis8aHRtbCBsYW5nPSJlbiI+Cis8aGVhZD4KKzwvaGVhZD4KKzxi
b2R5PgorPHA+PGI+QlVHIElEOjwvYj4gPGEgaHJlZj0iaHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3
aW4ub3JnL3Nob3dfYnVnLmNnaT9pZD04NjU4Ij5CdWd6aWxsYSBidWcgODY1ODwvYT4gQXNzZXJ0
aW9uIGZhaWx1cmUgaW4gLVtXZWJQbHVnaW5Db250YWluZXJDaGVjayBfaXNGb3JiaWRkZW5GaWxl
TG9hZF0gKGJyaWRnZSBpcyBudWxsKSB3aGVuIGNsaWNraW5nIFF1aWNrVGltZSBvYmplY3Qgd2l0
aCBocmVmPC9wPgorCis8cCBpZD0idGVzdCIgc3R5bGU9ImJhY2tncm91bmQtY29sb3I6c2t5Ymx1
ZTsgcGFkZGluZzozcHg7Ij48Yj5TVEVQUyBUTyBURVNUOjwvYj4gCitDbGljayB0aGUgc21hbGwg
b3JhbmdlIHNxdWFyZSBiZWxvdy4KKzwvcD4KKworPHAgaWQ9InN1Y2Nlc3MiIHN0eWxlPSJiYWNr
Z3JvdW5kLWNvbG9yOnBhbGVncmVlbjsgcGFkZGluZzozcHg7Ij48Yj5URVNUIFBBU1M6PC9iPiAK
K1F1aWNrVGltZSBQbGF5ZXIgd2lsbCBvcGVuIDxpPm9yYW5nZS5tb3Y8L2k+LgorPC9wPgorCis8
cCBpZD0iZmFpbHVyZSIgc3R5bGU9ImJhY2tncm91bmQtY29sb3I6I0ZGMzMwMDsgcGFkZGluZzoz
cHg7Ij48Yj5URVNUIEZBSUw6PC9iPiAgCitOb3RoaW5nIHdpbGwgaGFwcGVuIGluIHJlc3BvbnNl
IHRvIGNsaWNraW5nIHRoZSBvcmFuZ2Ugc3F1YXJlLgorPC9wPgorCis8RU1CRUQgaWQ9ImUiIHNy
Yz0icmVzb3VyY2VzL29yYW5nZS5tb3YiIHdpZHRoPSIxNSIgaGVpZ2h0PSIxNSIgY29udHJvbGxl
cj0iZmFsc2UiIGhyZWY9IiMiIHRhcmdldD0iUXVpY2tUaW1lUGxheWVyIj48L0VNQkVEPgorPC9i
b2R5PgorPC9odG1sPgpJbmRleDogV2ViQ29yZS9tYW51YWwtdGVzdHMvcmVzb3VyY2VzL29yYW5n
ZS5tb3YKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PQpDYW5ub3QgZGlzcGxheTogZmlsZSBtYXJrZWQgYXMgYSBiaW5hcnkg
dHlwZS4Kc3ZuOm1pbWUtdHlwZSA9IGFwcGxpY2F0aW9uL29jdGV0LXN0cmVhbQoKUHJvcGVydHkg
Y2hhbmdlcyBvbjogcmVzb3VyY2VzL29yYW5nZS5tb3YKX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpOYW1lOiBzdm46bWlt
ZS10eXBlCiAgICsgYXBwbGljYXRpb24vb2N0ZXQtc3RyZWFtCgpJbmRleDogV2ViS2l0L0NoYW5n
ZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXQvQ2hhbmdlTG9nCShyZXZpc2lvbiAxNDEyNCkKKysr
IFdlYktpdC9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxNCBAQAorMjAwNi0j
Iy0jIyAgTWl0eiBQZXR0ZWwgIDxvcGVuZGFyd2luLm9yZ0BtaXR6cGV0dGVsLmNvbT4KKworICAg
ICAgICAtIGZpeCBodHRwOi8vYnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hvd19idWcuY2dpP2lk
PTg2NTgKKyAgICAgICAgICBBc3NlcnRpb24gZmFpbHVyZSBpbiAtW1dlYlBsdWdpbkNvbnRhaW5l
ckNoZWNrIF9pc0ZvcmJpZGRlbkZpbGVMb2FkXQorICAgICAgICAgIChicmlkZ2UgaXMgbnVsbCkg
d2hlbiBjbGlja2luZyBRdWlja1RpbWUgb2JqZWN0IHdpdGggaHJlZgorCisgICAgICAgIFJldmll
d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogV2ViVmlldy9XZWJIVE1MVmlldy5t
OgorICAgICAgICAoLVtXZWJIVE1MVmlldyBzZXREYXRhU291cmNlOl0pOiBTZXQgdGhlIHBsdWdp
bkNvbnRyb2xsZXIncyBkYXRhU291cmNlLgorCiAyMDA2LTA0LTI4ICBEYXZpZCBIeWF0dCAgPGh5
YXR0QGFwcGxlLmNvbT4KIAogICAgICAgICBEb3VibGUgdGhlIGNhY2hlIHNpemUgdG8gYWNjb3Vu
dCBmb3Igb3VyIHJldmlzZWQgKG1vcmUgYWNjdXJhdGUpCkluZGV4OiBXZWJLaXQvV2ViVmlldy9X
ZWJIVE1MVmlldy5tCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC9XZWJWaWV3L1dlYkhUTUxWaWV3Lm0J
KHJldmlzaW9uIDE0MTI0KQorKysgV2ViS2l0L1dlYlZpZXcvV2ViSFRNTFZpZXcubQkod29ya2lu
ZyBjb3B5KQpAQCAtMzE0Myw2ICszMTQzLDcgQEAgZG9uZToKIHsKICAgICBBU1NFUlQoIV9wcml2
YXRlLT5kYXRhU291cmNlKTsKICAgICBfcHJpdmF0ZS0+ZGF0YVNvdXJjZSA9IFtkYXRhU291cmNl
IHJldGFpbl07CisgICAgW19wcml2YXRlLT5wbHVnaW5Db250cm9sbGVyIHNldERhdGFTb3VyY2U6
ZGF0YVNvdXJjZV07CiB9CiAKIC0gKHZvaWQpZGF0YVNvdXJjZVVwZGF0ZWQ6KFdlYkRhdGFTb3Vy
Y2UgKilkYXRhU291cmNlCg==
</data>
<flag name="review"
          id="2225"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>8037</attachid>
            <date>2006-04-29 12:35:30 -0700</date>
            <delta_ts>2006-04-29 12:35:30 -0700</delta_ts>
            <desc>orange.mov for the manual test</desc>
            <filename>orange.mov</filename>
            <type>video/quicktime</type>
            <size>875</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">AAAAIGZ0eXBxdCAgIAUDAHF0ICAAAAAAAAAAAAAAAAAAAAL1bW9vdgAAAGxtdmhkAAAAAMB5ZdfA
eWXXAAACWAAAAlgAAQAAAQAAAAAAAAAAAAAAAAEAAAAAAAAAAAAAAAAAAAABAAAAAAAAAAAAAAAA
AABAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAgAAAml0cmFrAAAAXHRraGQAAAAPwHll
wcB5ZdcAAAABAAAAAAAAAlgAAAAAAAAAAAAAAAAAAAAAAAEAAAAAAAAAAAAAAAAAAAABAAAAAAAA
AAAAAAAAAABAAAAAAA8AAAAPAAAAAAAYbG9hZAAAAAAAAAAAAAAAAAAAAQAAAAAkZWR0cwAAABxl
bHN0AAAAAAAAAAEAAAJYAAAAAAABAAAAAAG9bWRpYQAAACBtZGhkAAAAAMB5ZdfAeWXXAAACWAAA
AlgAAAAAAAAAOmhkbHIAAAAAbWhscnZpZGVhcHBsEAAAAAABAYIZQXBwbGUgVmlkZW8gTWVkaWEg
SGFuZGxlcgAAAVttaW5mAAAAFHZtaGQAAAABAQOAAIAAgAAAAAA5aGRscgAAAABkaGxyYWxpc2Fw
cGwQAAABAAEBmBhBcHBsZSBBbGlhcyBEYXRhIEhhbmRsZXIAAAAkZGluZgAAABxkcmVmAAAAAAAA
AAEAAAAMYWxpcwAAAAEAAADic3RibAAAAH5zdHNkAAAAAAAAAAEAAABuZ2lmIAAAAAAAAAABAAAA
AGFwcGwAAAAAAAAEAAAPAA8ASAAAAEgAAAAAAAAAAQNHSUYAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAgAAAAAAACAAAABgAD///////+AAP//ZmYAAAAAABhzdHRzAAAAAAAAAAEAAAABAAAC
WAAAABxzdHNjAAAAAAAAAAEAAAABAAAAAQAAAAEAAAAUc3RzegAAAAAAAAAuAAAAAQAAABRzdGNv
AAAAAAAAAAEAAAM9AAAADHVkdGEAAAAAAAAAGHVkdGEAAAAMV0xPQwAaAAQAAAAAAAAACGZyZWUA
AAAId2lkZQAAAEZtZGF0AAAACHdpZGUAAAAAbWRhdEdJRjg5YQ8ADwCA/wD/////ZgAsAAAAAA8A
DwAAAg2Mj6nL7Q+jnLTai2cBADs=
</data>

          </attachment>
      

    </bug>

</bugzilla>