484621 - Port Bug 471962: When saving an inner frame as file only, the POST data of the outer page is sent to the address of the inner page

Status: VERIFIED FIXED

(SeaMonkey :: Download & File Handling, defect)

Description

[Justin Wood (:Callek)]

+++ This bug was initially created as a clone of Bug #483959 +++

{
TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/toolkit/content/tests/browser/browser_bug471962.js | The saved inner frame does not contain outer POST data
}

I'll fix 1.9.1 for SeaMonkey, and 1.8.1 for SeaMonkey, this may also affect 1.8.1 TB, doubt it is worth fixing there, but if someone identifies it as a problem I'll do so.

Comment 1

[Justin Wood (:Callek)]

[sg:low] per Bug 471962

Comment 2

[Justin Wood (:Callek)]

Attachment: Patch for SeaMonkey c-c [checked-in]

This is a direct port of the patch in Bug 471962. Does make their test pass for us.

Comment 3

[Justin Wood (:Callek)]

Attachment: SeaMonkey version, 1.8.1

The SeaMonkey 1.1.x version of the above

Comment 4

[Justin Wood (:Callek)]

Attachment: Toolkit (Thunderbird) version, 1.8.1

I doubt normal builds of TB need this, but for the benefit of XULRunner on 1.8.1 and linux distro's can't hurt to push the toolkit fix while I'm here. (r? on bz since he reviewed original patch).

Comment 5

[Serge Gautherie (:sgautherie)]

V.Fixed, as SeaMonkey tinderboxes pass bug 483959 test now.

Comment 6

[Daniel Veditz [:dveditz]]

Comment on attachment 368809 [details] [diff] [review]
Toolkit (Thunderbird) version, 1.8.1

Approved for 1.8.1.22, a=dveditz for release-drivers

Comment 7

[Daniel Veditz [:dveditz]]

I guess I didn't need to approve the seamonkey-only patch

Comment 8

[Justin Wood (:Callek)]

dveditz, can you please commit these two 1.8.x patches, just moved today and won't have net access where I can utilize my ssh key for a while.

Comment 9

[Frank Wein [:mcsmurf]]

Checking in toolkit/content/contentAreaUtils.js;
/cvsroot/mozilla/toolkit/content/contentAreaUtils.js,v  <--  contentAreaUtils.js
new revision: 1.77.2.8; previous revision: 1.77.2.7
done
Checking in xpfe/communicator/resources/content/contentAreaUtils.js;
/cvsroot/mozilla/xpfe/communicator/resources/content/contentAreaUtils.js,v  <--
 contentAreaUtils.js
new revision: 1.134.2.7; previous revision: 1.134.2.6
done