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"
+#include "testing/gtest/include/gtest/gtest.h"
 
 using extensions::FileSystemChooseFileFunction;
 
 class FileSystemApiTest : public PlatformAppBrowserTest {
  public:
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     PlatformAppBrowserTest::SetUpCommandLine(command_line);
     test_root_folder_ = test_data_dir_.AppendASCII("api_test")
         .AppendASCII("file_system");
   }
@@ skipping 176 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_;
 }
+
+class FileSystemApiUnitTest : public testing::Test {

[benwells, 2012/07/26 08:10:23]

This should go in file file_system_unittest.cc, and be compiled into a different
executable than browser_tests (probably unit_tests_.

[thorogood, 2012/07/27 03:14:38]

Done.

+};
+
+TEST_F(FileSystemApiUnitTest,
+    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);
+}