Updates file type selector for fileSystem API

chrome/browser/extensions/api/file_system/file_system_apitest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "chrome/browser/extensions/api/file_system/file_system_api.h"
 #include "chrome/browser/extensions/platform_app_browsertest_util.h"
 
 using extensions::FileSystemChooseFileFunction;
@@ skipping 184 matching lines @@
 }
 
 IN_PROC_BROWSER_TEST_F(FileSystemApiTest, FileSystemApiIsWritableTest) {
   FilePath test_file = TempFilePath("writable.txt", true);
   ASSERT_FALSE(test_file.empty());
   FileSystemChooseFileFunction::SkipPickerAndAlwaysSelectPathForTest(
       &test_file);
   ASSERT_TRUE(RunPlatformAppTest(
       "api_test/file_system/is_writable_file_entry")) << message_;
 }
+
+// TODO(thorogood): Not really a browser test, but this is the easiest way to

[benwells, 2012/07/26 05:40:45]

This should be a unit test, not a browser test

[thorogood, 2012/07/26 07:39:05]

Done.

+// quickly define it alongside our other tests.
+IN_PROC_BROWSER_TEST_F(FileSystemApiTest,
+    FileSystemChooseFileFunctionFileTypeInfoTest) {
+  // AcceptsAllTypes is ignored when no other extensions are available.
+  scoped_ptr<SelectFileDialog::FileTypeInfo> file_type_info(
+      new SelectFileDialog::FileTypeInfo());
+  bool acceptsAllTypes = false;
+  FileSystemChooseFileFunction::BuildFileTypeInfo(file_type_info.get(),
+      NULL, &acceptsAllTypes);
+  EXPECT_TRUE(file_type_info->include_all_files);
+  EXPECT_TRUE(file_type_info->extensions.empty());
+
+  // Test grouping of multiple types.
+  file_type_info.reset(new SelectFileDialog::FileTypeInfo());
+  std::vector<std::string> accepts;
+  accepts.push_back(".jso,application/x-chrome-extension");
+  acceptsAllTypes = false;
+  FileSystemChooseFileFunction::BuildFileTypeInfo(file_type_info.get(),
+      &accepts, &acceptsAllTypes);
+  EXPECT_FALSE(file_type_info->include_all_files);
+  EXPECT_EQ(file_type_info->extensions.size(), (size_t) 1);
+}