Thomas Goldstein Reporter
	Description • 16 years ago

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090110 Shiretoko/3.1b3pre (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090110 Shiretoko/3.1b3pre (.NET CLR 3.5.30729)

For example, go to: https://addons.mozilla.org/en-US/firefox/addon/1865
Start the install, and then, while the download bar fills, go to another page (go back, or go to Google, whatever).
Result: the download will stop, and a "Download error" message box will pop up.

Reproducible: Always

	Dave Garrett
	Comment 1 • 16 years ago

because: Download error
-228

I can confirm this. Have to be quick on the trigger to click to another page if it you download fast, but it does error if I go to my homepage before getting to the end. (during download or "wait" period in the beginning) Tested in 3.1b3pre on Linux.

	Dave Garrett
	Comment 2 • 16 years ago

Not sure if this is on the add-ons manager end or the AMO end. Might also be a dupe, but I can't find one.

	Dave Garrett
	Comment 3 • 16 years ago

After a bit of testing in a new profile, I've been able to reproduce this in both 3.1b3pre and 3.2a1pre but not 3.0.5.

	Ria Klaassen (not reading all bugmail)
	Comment 4 • 16 years ago

Seems to be something from last week November.

	Fred Wenzel [:wenzel]
	Comment 5 • 16 years ago

(In reply to comment #2)
> Not sure if this is on the add-ons manager end or the AMO end.

That sounds a lot like a client bug.

	Ria Klaassen (not reading all bugmail)
	Comment 6 • 16 years ago

Regression range is http://hg.mozilla.org/mozilla-central/pushloghtml?startdate=2008-11-28+05%3A00%3A00&enddate=2008-11-28+21%3A00%3A00
There is a Bug 462739, a Bug 454546 and a Bug 463882. 
BTW, for some reason I need to delete the profile after every test.

	Dave Garrett
	Comment 7 • 16 years ago

(In reply to comment #6)
> BTW, for some reason I need to delete the profile after every test.

Odd, I had no such problem here.

	Ria Klaassen (not reading all bugmail)
	Comment 8 • 16 years ago

(In reply to comment #7)
> (In reply to comment #6)
> > BTW, for some reason I need to delete the profile after every test.
> 
> Odd, I had no such problem here.

Clicking Cancel on the dialog in a unaffected build gave a false negative in a affected build.

	Dave Townsend [:mossop] Assignee
	Comment 9 • 16 years ago

Will backout bug 462739 to resolve this

	Dave Townsend [:mossop] Assignee
	Comment 10 • 16 years ago

Actually have a potential fix for this, but still working on developing unit tests for it.

	Dave Townsend [:mossop] Assignee
	Comment 11 • 16 years ago

I don't think we need this for b3, but we'll probably want to relnote it if it isn't fixed by then.

	Dave Townsend [:mossop] Assignee
	Comment 12 • 16 years ago

This patch works by removing the request from the page's load group once the request has begun. This is after any necessary cookies have been added to the headers so does not regress bug 462739, but means that when the page is navigated away from the load group will no longer cancel the xpi request.

Tests for this exist in the test suite in bug 474763

	Daniel Veditz [:dveditz]
	Comment 13 • 16 years ago

Comment on attachment 359502 [details] [diff] [review]
patch rev 1

r/sr=dveditz

	Dave Townsend [:mossop] Assignee
	Comment 14 • 16 years ago

Landed: http://hg.mozilla.org/mozilla-central/rev/cd824b1dd0bb

	Dave Townsend [:mossop] Assignee
	Comment 15 • 16 years ago

Landed on branch: http://hg.mozilla.org/releases/mozilla-1.9.1/rev/d1d3a589d096

	Dave Townsend [:mossop] Assignee
	Comment 16 • 16 years ago

Comment on attachment 359502 [details] [diff] [review]
patch rev 1

We need this on branch if we also want to fix bug 462739 there

	Daniel Veditz [:dveditz]
	Comment 17 • 16 years ago

Comment on attachment 359502 [details] [diff] [review]
patch rev 1

Approved for 1.9.0.8, a=dveditz for release-drivers

	Dave Townsend [:mossop] Assignee
	Comment 18 • 16 years ago

Landed on branch:

Checking in xpinstall/src/CertReader.cpp;
/cvsroot/mozilla/xpinstall/src/CertReader.cpp,v  <--  CertReader.cpp
new revision: 1.13; previous revision: 1.12
done
Checking in xpinstall/src/nsXPInstallManager.cpp;
/cvsroot/mozilla/xpinstall/src/nsXPInstallManager.cpp,v  <--  nsXPInstallManager.cpp
new revision: 1.167; previous revision: 1.166
done

	Al Billings [:abillings - ex-MoCo]
	Comment 19 • 16 years ago

(In reply to comment #0)
> User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US;
> rv:1.9.1b3pre) Gecko/20090110 Shiretoko/3.1b3pre (.NET CLR 3.5.30729)
> Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US;
> rv:1.9.1b3pre) Gecko/20090110 Shiretoko/3.1b3pre (.NET CLR 3.5.30729)
> 
> For example, go to: https://addons.mozilla.org/en-US/firefox/addon/1865
> Start the install, and then, while the download bar fills, go to another page
> (go back, or go to Google, whatever).
> Result: the download will stop, and a "Download error" message box will pop up.
> 
> Reproducible: Always

I cannot confirm this bug with these repro steps in 1.9.0.7 on XP (which is not surprising based on comment 3).