statusText is set twice in Downloader::onStopRequest

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

({fixed1.9.1})

unspecified
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

btw: this makes test_0030_general.js pass with a XULRunner by itself
Attachment #373395 - Flags: review?(dtownsend) → review+
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/176afb7235f8
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
This code path is in the test suite already so adding in‑testsuite even though this is just removing unnecessary code
Flags: in-testsuite+
Comment on attachment 373395 [details] [diff] [review]
patch rev1

Drivers, this patch removes code that sets a value that is immediately set ot a new value. With this code removed the xpcshell test test_0030_general.js passes on standalone XULRunner and likely will also pass for fennec (bug 477597)
Attachment #373395 - Flags: approval1.9.1?
Comment on attachment 373395 [details] [diff] [review]
patch rev1

a191=beltzner
Attachment #373395 - Flags: approval1.9.1? → approval1.9.1+
You need to log in before you can comment on or make changes to this bug.