Closed Bug 753755 Opened 8 years ago Closed 8 years ago

[Page Thumbnails] Disabling thumbnails should not create empty folders

Categories

(Firefox :: Tabbed Browser, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 15

People

(Reporter: jmjjeffery, Assigned: ttaubert)

References

Details

Attachments

(1 file)

Turning off Thumbnails for the New Tab page should not create 'Empty Folders'

1. Add hidden pref: browser.pagethumbnails.capturing_disabled
2. Set value to 'True' 

With Firefox closed delete the thumbnails folder in your Profile.
Browse around and check back in your Profile - a New Thumbnail Folder was created on restart of the browser as one would expect. 

What one should not expect is to find numerous 'empty folders' being created as you continue to browse over time.  

Over just a few days after clearing the Thumbnails Folder there are now 271 Empty Folders.  I fear that something could creep into the folders and would remain buried and would be very hard to locate something hidden there by perhaps a malicious program.

Enabling the Hidden Pref should turn off creating of folders.
Component: General → Tabbed Browser
QA Contact: general → tabbed.browser
Summary: Disabling thumbnails should not create empty folders → [Page Thumbnails] Disabling thumbnails should not create empty folders
See Also: → 754671
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
OS: Windows 7 → All
Hardware: x86_64 → All
Attached patch patch v1Splinter Review
This adds a {createPath: bool} flag to PageThumbsStorage.getFileForURL() so that we stop creating the file's path implicitly.
Attachment #623677 - Flags: review?(dietrich)
Comment on attachment 623677 [details] [diff] [review]
patch v1

Review of attachment 623677 [details] [diff] [review]:
-----------------------------------------------------------------

yay fileutils
Attachment #623677 - Flags: review?(dietrich) → review+
Pushed to fx-team:

https://hg.mozilla.org/integration/fx-team/rev/92317d7fe1ee

and merged:

https://hg.mozilla.org/mozilla-central/rev/92317d7fe1ee
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
You need to log in before you can comment on or make changes to this bug.