785955 - Device Storage - Filter out mimetypes based on device storage type

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect)

Description

[Doug Turner (:dougt)]

Currently, when you enumerate a directory, we return files that shouldn't be returned.  For example, if you ask for a 'pictures' device storage object and in that directory there are files that are not pictures, currently we return those.  What I'd like to do is to filter those files out of the enumeration.

This will be very helpful when we start enumerating the SDCard for Gaia/B2G

Comment 1

[Doug Turner (:dougt)]

Attachment: patch v.1

I'd like to keep the DEBUG_ISTYPE code in for a while.  This will help aide in gaia debugging.

Comment 2

[Johnny Stenback (:jst)]

Blocking.

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 655657 [details] [diff] [review]
patch v.1

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

Let me know what you think about using the properties file instead. I think the current approach would be ok, but we'd need to check that the registry on B2G contains good defaults.

::: dom/devicestorage/nsDeviceStorage.cpp
@@ +150,5 @@
>    return true;
>  }
>  
> +bool
> +DeviceStorageFile::IsType(nsAString& aType)

You'd likely get better results by using the values from the chrome://global/content/filepicker.properties file. I.e. the imageFilter, audioFilter and videoFilter properties there. That's what we're using in the filepicker and has the nice advantage that it doesn't depend on the OS registry and so will produce more consistent results.

Comment 4

[Doug Turner (:dougt)]

Yes, we definitely will have to do that.  I can do that as a followup, if you don't mind.  I filled 786922

Comment 5

[Doug Turner (:dougt)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9c497a6e9e0f

Comment 6

[Philipp von Weitershausen [:philikon]]

https://hg.mozilla.org/mozilla-central/rev/9c497a6e9e0f