Closed
Bug 1261678
Opened 9 years ago
Closed 9 years ago
wrong variable name in nsActivityManager.js::getProcessesByContext()
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(thunderbird46 wontfix, thunderbird47 fixed, thunderbird48 fixed, thunderbird_esr4546+ fixed)
RESOLVED
FIXED
Thunderbird 48.0
People
(Reporter: aceman, Assigned: aceman)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
jorgk-bmo
:
review+
jorgk-bmo
:
approval-comm-aurora+
jorgk-bmo
:
approval-comm-beta-
rkent
:
approval-comm-esr45+
|
Details | Diff | Splinter Review |
There is a typo in the function nsActivityManager.js::getProcessesByContext(), the variable is called 'activity', not 'value' in this instance.
getProcessesByContext: function(aContextType, aContextObj, aCount) {
let list = [];
- for each (let [, activity] in Iterator(this._activities)) {
+ for (let id in this._activities) {
+ let value = this._activities;
if (activity instanceof Ci.nsIActivityProcess &&
activity.contextType == aContextType &&
activity.contextObj == aContextObj) {
list.push(activity);
}
}
Caused in bug 824104
Attachment #8737602 -
Flags: review?(mozilla)
The incorrect line quoted in comment 0 has since been fixed to 'let value = this._activities[id]', but the variable name of 'value' remained.
tracking-thunderbird45:
--- → ?
Comment 3•9 years ago
|
||
Comment on attachment 8737602 [details] [diff] [review]
1261678.patch
Review of attachment 8737602 [details] [diff] [review]:
-----------------------------------------------------------------
Grrr, we already fixed this in bug 1231887 attachment 8697587 [details] [diff] [review] and it was still not right.
What are the effects of this? This needs uplift right into ESR 45, right?
Attachment #8737602 -
Flags: review?(mozilla)
Attachment #8737602 -
Flags: review+
Attachment #8737602 -
Flags: approval-comm-esr45?
Attachment #8737602 -
Flags: approval-comm-beta?
Attachment #8737602 -
Flags: approval-comm-aurora+
Updated•9 years ago
|
status-thunderbird45:
--- → affected
status-thunderbird46:
--- → affected
status-thunderbird47:
--- → affected
status-thunderbird48:
--- → affected
status-thunderbird_esr45:
--- → affected
tracking-thunderbird45:
? → ---
tracking-thunderbird_esr45:
--- → +
I don't know if it has any effect but the function basically does nothing now, expect for maybe spamming the console with 'undeclared variable activity'.
Yes, I'd suggest this for TB45. That is the time when the bug will arrive to stable users. So lack of bug reports till now (from only nightly users) does not mean the bug is harmless.
Keywords: checkin-needed
Comment 5•9 years ago
|
||
https://hg.mozilla.org/comm-central/rev/ba30dd5df13d6c0ec3adb79f07b7a3bc4d9064a5
Bug 1261678 - fix wrong copy-pasted variable name in nsActivityManager.js::getProcessesByContext() (value => activity). r=jorgk
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 48.0
Comment 6•9 years ago
|
||
Aurora (TB 47):
https://hg.mozilla.org/releases/comm-aurora/rev/504faf2bbf34
Comment 7•9 years ago
|
||
AFAICT although we should land this in a point release, it is not serious enough to block 45.0
Comment 8•9 years ago
|
||
Updated•9 years ago
|
Attachment #8737602 -
Flags: approval-comm-esr45? → approval-comm-esr45+
Updated•8 years ago
|
Attachment #8737602 -
Flags: approval-comm-beta? → approval-comm-beta-
Updated•8 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•