Closed Bug 285588 Opened 19 years ago Closed 19 years ago

In <globalOverlay.js>, (2) "Warning: function goQuitApplication does not always return a value"

Categories

(Core :: XUL, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla1.8beta2

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

Attachments

(1 file)

[Mozilla Thunderbird, version 1.0.1 (20050309)] (nightly) (W98SE)

{{
Warning: function goQuitApplication does not always return a value
Source File: chrome://global/content/globalOverlay.js
Line: 64, Column: 19
Source Code:
       return false;          

Warning: function goQuitApplication does not always return a value
Source File: chrome://global/content/globalOverlay.js
Line: 68, Column: 13
Source Code:
  return true;
}}
Came from
{{
1.7	ben%bengoodger.com	2003-11-30 21:40		Ensure that
window closing that would cause the app to quit goes through the same codepath
as an actual application quit. (The bug was that you would get the download
cancel confirmation prompt when shutting down via File->Exit, but not when you
closed all windows)
}}

Could you (super-)review/check in this patch ? Thanks.
Assignee: events → gautheri
Status: NEW → ASSIGNED
Attachment #177009 - Flags: superreview?(bryner)
Attachment #177009 - Flags: review?(bryner)
Component: Event Handling → XP Toolkit/Widgets
Comment on attachment 177009 [details] [diff] [review]
(Av1) <globalOverlay.js>
[Checked in: Comment 3]

r+sr=bryner
Attachment #177009 - Flags: superreview?(bryner)
Attachment #177009 - Flags: superreview+
Attachment #177009 - Flags: review?(bryner)
Attachment #177009 - Flags: review+
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Attachment #177009 - Attachment description: (Av1) <globalOverlay.js> → (Av1) <globalOverlay.js> [Checked in: Comment 3]
Attachment #177009 - Attachment is obsolete: true
Target Milestone: --- → mozilla1.8beta2
*** Bug 290356 has been marked as a duplicate of this bug. ***
Attachment #177009 - Attachment is obsolete: false
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: