Bug 13636 - <OBJECT> tags seem mistakenly included in the document's applet collection
Summary: <OBJECT> tags seem mistakenly included in the document's applet collection
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL: http://homestarrunner.com
Keywords: InRadar
Depends on:
Blocks: 13634
  Show dependency tree / graph
 
Reported: 2007-05-09 00:49 PDT by Brady Eidson
Modified: 2007-05-10 02:21 PDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brady Eidson 2007-05-09 00:49:42 PDT 
Both in Safari 2.0 and ToT, it seems OBJECT tags were incorrectly included in a page's applet collection.  This prevented (and continues to prevent) back/forward caching of certain pages.

Visit the above url in Safari/ToT WebKit and in Firefox
run "javascript:alert(document.applets[0].outerHTML);" from the location field

Firefox has 0, Safari/ToT have 1 - view the page source to see the big glaring OBJECT

Since Firefox correctly reports 0, it's hard to believe we ever did this for compatibility with FFX - however, I have not tried WinIE
Comment 1 Brady Eidson 2007-05-09 00:51:35 PDT 
Correction - since FFX doesn't support outerhtml, amend the javascript to be "javascript:alert(document.applets[0]);"

It gives you back an undefined collection, whereas Safari hands back a collection (with 1 member)
Comment 2 Brady Eidson 2007-05-09 00:52:48 PDT 
ummm check that - I meant a null ELEMENT, and Safari gives back an HTMLObjectElement
Comment 3 Brady Eidson 2007-05-09 01:15:25 PDT 
http://trac.webkit.org/projects/webkit/changeset/12250 might be of interest
Comment 4 Brady Eidson 2007-05-09 10:07:56 PDT 
<rdar://problem/5190816>
Comment 5 Brady Eidson 2007-05-10 02:18:08 PDT 
Landed in r21359
Comment 6 Eric Seidel (no email) 2007-05-10 02:21:02 PDT 
Here is a link to the change:
http://trac.webkit.org/projects/webkit/changeset/21359