Before, the frame was creating the file picker. When the file picker creation has been moved to the content, this method hasn't been moved.
Created attachment 477002 [details] [diff] [review] Patch v1
Created attachment 477019 [details] [diff] [review] Patch v1.1 Hmmm, some tests were expecting appendFilters to be called... and I wrote that :( I wonder why we need AppendFilters(filterAll) to be called. Shouldn't we expect the file picker to revert to that value otherwise? GTK file picker looks nicer without adding this filter (it removes a useless button)...
Attachment #477019 - Flags: review?(Olli.Pettay) → review+
Attachment #477019 - Flags: approval2.0? → approval2.0+
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Target Milestone: mozilla2.0b8 → mozilla2.0b7
You need to log in before you can comment on or make changes to this bug.