Bug 8941

Summary: document.cookie undefined for documents of type text/javascript
Product: WebKit Reporter: Geoffrey Garen <ggaren>
Component: WebCore JavaScriptAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: a.bob, andersca, oster
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.google.com/calendar/ping
Bug Depends on:    
Bug Blocks: 8370    
Attachments:
Description Flags
Inherit from HTMLDocument hyatt: review+

Geoffrey Garen
Reported 2006-05-16 10:32:02 PDT
1. navigate to www.google.com/calendar/ping 2. in the url field, enter "javascript:alert(document.cookie)" --> undefined in Safari; defined in FF Maciej mentioned that the problem probably results from the fact that the document has type text/javascript instead of type text/html. We should figure out which document types need to support cookies (text/xml? text/xhtml?), and support them. Anders, Maciej thought that your TextDocument change would fix this, but that doesn't seem to be the case.
Attachments
Inherit from HTMLDocument (1.52 KB, patch)
2006-05-16 10:51 PDT, Anders Carlsson 		hyatt: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2006-05-16 10:51:41 PDT
Created attachment 8352 [details] Inherit from HTMLDocument This change fixes it for me. I thought I had included it when landing ImageDocument (there's even a ChangeLog entry about it)
Dave Hyatt
Comment 2 2006-05-16 11:22:27 PDT
Comment on attachment 8352 [details] Inherit from HTMLDocument r=me
Darin Adler
Comment 3 2006-05-16 11:47:35 PDT
Comment on attachment 8352 [details] Inherit from HTMLDocument We need to move things like cookie out of HTMLDocument and into Document. The only things that should be in HTMLDocument are things we don't need in a true XHTML document.
Anders Carlsson
Comment 4 2006-05-16 12:01:48 PDT
Yes, but I think that's a separate issue. Both Opera and Firefox use HTMLDocuments for text views and I think we should too
Note You need to log in before you can comment on or make changes to this bug.