Closed Bug 923715 Opened 11 years ago Closed 11 years ago

windows should ignore closed windows

Categories

(Add-on SDK Graveyard :: General, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mossop, Assigned: mossop)

Details

Attachments

(1 file)

The window enumerator returns windows that have already been closed sometimes, these should probably be ignored. See the comment at http://hg.mozilla.org/mozilla-central/diff/da5496557024/addon-sdk/source/lib/sdk/window/utils.js
Assignee: nobody → dtownsend+bugmail
Attachment #814629 - Flags: review?(evold)
Comment on attachment 814629 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1261

Just mentioned a minor issue in the pull request.
Attachment #814629 - Flags: review?(evold) → review-
Comment on attachment 814629 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1261

Updated
Attachment #814629 - Flags: review- → review?(evold)
Attachment #814629 - Flags: review?(evold) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/a9d4a94dcd8ccefcc9b4ae164f1ec375e3aa53b9
Bug 923715: Ignore closed windows in the windows iterator.

https://github.com/mozilla/addon-sdk/commit/53503fcf6df8364e1dda9adf7595c74026015f2a
Merge pull request #1261 from Mossop/bug923715

Bug 923715: Ignore closed windows in the windows iterator. r=erikvold
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: