Fix acceptance of multiple mime types in the Mac file picker.

Fix acceptance of multiple mime types in the Mac file picker.

This mostly reverts r135025 and does it correctly based on how it really works.

BUG=127749
TEST=as in bug


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137162

[Avi (use Gerrit), 2012-05-15 15:24:47]

You reviewed the first version, so you get this one.

[Nico, 2012-05-15 15:38:50]

lgtm

https://chromiumcodereview.appspot.com/10388145/diff/1/chrome/browser/ui/coco...
File chrome/browser/ui/cocoa/select_file_dialog_mac.mm (right):

https://chromiumcodereview.appspot.com/10388145/diff/1/chrome/browser/ui/coco...
chrome/browser/ui/cocoa/select_file_dialog_mac.mm:228: NSString* type_extension
= base::SysUTF8ToNSString(ext_list[j]);
nit: it might be worth it to have a function that maps a std::string file
extension to an CFStringRef uti and then call that here and further down

[Avi (use Gerrit), 2012-05-15 15:54:12]

https://chromiumcodereview.appspot.com/10388145/diff/1/chrome/browser/ui/coco...
File chrome/browser/ui/cocoa/select_file_dialog_mac.mm (right):

https://chromiumcodereview.appspot.com/10388145/diff/1/chrome/browser/ui/coco...
chrome/browser/ui/cocoa/select_file_dialog_mac.mm:228: NSString* type_extension
= base::SysUTF8ToNSString(ext_list[j]);
On 2012/05/15 15:38:50, Nico wrote:
> nit: it might be worth it to have a function that maps a std::string file
> extension to an CFStringRef uti and then call that here and further down

Done.

[commit-bot: I haz the power, 2012-05-15 16:03:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/10388145/2002

[commit-bot: I haz the power, 2012-05-15 17:35:33]

Change committed as 137162