Hallvord R. M. Steen [:hallvors] Reporter
	Description • 11 years ago

Test cases:
http://w3c-test.org/web-platform-tests/master/XMLHttpRequest/responsexml-document-properties.htm

Spec clarification: http://lists.w3.org/Archives/Public/public-webapps/2013JulSep/0636.html

	Thomas Wisniewski Assignee
	Comment 1 • 8 years ago

This patch fixes the referrer and lastModified attributes of responseXML for the tests, covering two of the four failing cases in the WPT. Bug 918768 is about the remaining document.cookie issues, so I'll leave those changes for that bug.

Try shows no failures: https://treeherder.mozilla.org/#/jobs?repo=try&revision=bab3c914c5a2

Note that the existing web platform test does not actually get a Last-Modified HTTP header with the response, so the Last-Modified test will never pass as-is. As such the test will have to be corrected upstream before this patch lands, or I'll have to tweak it to still expect a failure from that test for now. I'll attach a second patch with a version of the test that works.

	Thomas Wisniewski Assignee
	Comment 2 • 8 years ago

Whoops, sorry, that first comment has the WPT-fixing patch, not the one that helps Firefox pass the tests. This is the one to review.

	Thomas Wisniewski Assignee
	Comment 3 • 8 years ago

Anne, could you take a peek at my comments and patch in comment 1 in this bug? Thanks!

	Anne (:annevk)
	Comment 4 • 8 years ago

This looks wrong. According to https://html.spec.whatwg.org/multipage/dom.html#dom-document-lastmodified even if that header is not present, document.lastModified still returns a value. It seems Chrome fails this too however so maybe we should change what this property returns for documents that are not the result of a navigation?

	Thomas Wisniewski Assignee
	Comment 5 • 8 years ago

Alright, here's a patch with just the responseXML.referrer changes.

I've created bug 1280454 to deal with lastModified once a decision is made on what to do there.

	Thomas Wisniewski Assignee
	Comment 6 • 8 years ago

Rebased.

	Pulsebot
	Comment 7 • 8 years ago

Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3f85f05c1e20
set responseXML.referrer to the empty string. r=jst

	Carsten Book [:Tomcat]
	Comment 8 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/3f85f05c1e20