369756 - Cocoa file picker doesn't set mSelectedType in all cases

Status: RESOLVED INVALID

(Core :: Widget: Cocoa, defect)

Description

[Alex Vincent [:WeirdAl]]

Spun off of bug 368285.  Only the "PutLocalFiles" cas sets mSelectedType.

Comment 1

[Alex Vincent [:WeirdAl]]

specifically bug 368285 comment 2.

Comment 2

[Alex Vincent [:WeirdAl]]

also worth noting (but probably not needing its own bug) is http://lxr.mozilla.org/seamonkey/source/widget/src/cocoa/nsFilePicker.mm#577 and #588 : "do we still need this?" is definitely "yes", because the IDL requires it.

Comment 3

[Alex Vincent [:WeirdAl]]

Additional test case, unresolved:  what if the chrome code calling on this does the following:

fp.appendFilter("XULRunner application", "*.ini");
fp.appendFilter("Some other ini file", "*.ini");
var buttonStatus = fp.show();

How should fp.filterIndex be resolved?

Comment 4

[Mounir Lamouri (:mounir)]

(In reply to comment #0)
> Spun off of bug 368285.  Only the "PutLocalFiles" cas sets mSelectedType.

Actually, the real question is why PutLocalFiles is setting mSelectType?

Comment 5

[Stephen A Pohl [:spohl]]

Closing this since Components.classes is no longer accessible to web content.