Closed
Bug 1095299
Opened 11 years ago
Closed 11 years ago
Filepicker does not show directories if ui.allow_platform_file_picker is false
Categories
(Toolkit :: General, defect)
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: h.figge, Assigned: MatsPalmgren_bugz)
References
Details
(Keywords: regression)
Attachments
(2 files)
|
17.55 KB,
image/png
|
Details | |
|
1.33 KB,
patch
|
efaust
:
review+
|
Details | Diff | Splinter Review |
With ui.allow_platform_file_picker set to false using File->Open in the browser on SM-Trunk opens the filepicker, but the field behind 'Look in' is empty and cannot be changed. Workaround: put the path before the filename. The error console shows
Error: ReferenceError: numFilters is not defined
Source File: chrome://global/content/filepicker.js Line: 82
Regression window is
Last good: 2014-11-04 16:15:00 PST
First bad: 2014-11-05 16:39:00 PST
| Assignee | ||
Comment 1•11 years ago
|
||
It's broken in Firefox too. The dropdown list is empty and clicking
on the Home button (in the upper right) displays an error message.
| Assignee | ||
Comment 2•11 years ago
|
||
Regression range FF mozilla-central: 2014-11-05-03-02-03 -- 2014-11-06-03-02-01
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=62990ec7ad78&tochange=2114ef80f6ae
Component: Download & File Handling → General
Product: SeaMonkey → Toolkit
Summary: SM-Trunk: Filepicker does not show directories if ui.allow_platform_file_picker is false → Filepicker does not show directories if ui.allow_platform_file_picker is false
| Assignee | ||
Comment 3•11 years ago
|
||
Bisecting m-i gives this changeset that broke it:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a9a7f16c817b
Blocks: 611388
| Assignee | ||
Comment 4•11 years ago
|
||
Attachment #8519492 -
Flags: review?(efaustbmo)
| Reporter | ||
Comment 5•11 years ago
|
||
(In reply to Mats Palmgren (:mats) from comment #4)
> Created attachment 8519492 [details] [diff] [review]
> fix
Works for my SM-Trunk Linux x86_64
Updated•11 years ago
|
Attachment #8519492 -
Flags: review?(efaustbmo) → review+
| Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 6•11 years ago
|
||
Assignee: nobody → mats
Keywords: checkin-needed
Comment 7•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•