Postpone setting up file handler's file permissions if handler is running lazy background page.

chrome/browser/chromeos/extensions/external_filesystem_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/platform_file.h"
 #include "base/path_service.h"
 #include "base/scoped_temp_dir.h"
 #include "base/stringprintf.h"
 #include "chrome/browser/chromeos/gdata/gdata_file_system_proxy.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
+#include "chrome/common/chrome_switches.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "webkit/fileapi/file_system_context.h"
 #include "webkit/fileapi/file_system_mount_point_provider.h"
 #include "chrome/browser/chromeos/gdata/gdata_util.h"
 #include "webkit/chromeos/fileapi/remote_file_system_proxy.h"
 
 using ::testing::_;
 using content::BrowserContext;
 
 // These should match the counterparts in remote.js.
@@ skipping 87 matching lines @@
     "Got unexpected error: File handler error: SECURITY_ERR";
 }
 
 class FileSystemExtensionApiTest : public ExtensionApiTest {
  public:
   FileSystemExtensionApiTest() : test_mount_point_("/tmp") {
   }
 
   virtual ~FileSystemExtensionApiTest() {}
 
+   void SetUpCommandLine(CommandLine* command_line) {

[tbarzic, 2012/04/13 02:58:30]

I messed up indent here..

+    ExtensionApiTest::SetUpCommandLine(command_line);
+    command_line->AppendSwitch(switches::kEnableExperimentalExtensionApis);
+  }
+
   // Adds a local mount point at at mount point /tmp.
   void AddTmpMountPoint() {
     fileapi::ExternalFileSystemMountPointProvider* provider =
         BrowserContext::GetFileSystemContext(browser()->profile())->
             external_provider();
     provider->AddLocalMountPoint(test_mount_point_);
   }
 
  private:
   FilePath test_mount_point_;
@@ skipping 53 matching lines @@
   ASSERT_TRUE(RunComponentExtensionTest("local_filesystem")) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest, FileBrowserTest) {
   AddTmpMountPoint();
   ASSERT_TRUE(RunExtensionTest("filesystem_handler")) << message_;
   ASSERT_TRUE(RunExtensionSubtest(
       "filebrowser_component", "read.html", kComponentFlags)) << message_;
 }
 
+IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest, FileBrowserTestLazy) {
+  AddTmpMountPoint();
+  ASSERT_TRUE(RunExtensionTest("filesystem_handler_lazy_background"))
+      << message_;
+  ASSERT_TRUE(RunExtensionSubtest(
+      "filebrowser_component", "read.html", kComponentFlags)) << message_;
+}
+
 IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest, FileBrowserTestWrite) {
   AddTmpMountPoint();
   ASSERT_TRUE(RunExtensionTest("filesystem_handler_write")) << message_;
   ASSERT_TRUE(RunExtensionSubtest(
       "filebrowser_component", "write.html", kComponentFlags)) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest,
                        FileBrowserTestWriteReadOnly) {
   AddTmpMountPoint();
@@ skipping 36 matching lines @@
       .WillOnce(MockCreateSnapshotFile(
           base::PLATFORM_FILE_OK,
           test_file_info_,
           // Returns the path to the temporary file on the local drive.
           test_file_path_,
           scoped_refptr<webkit_blob::ShareableFileReference>(NULL)));
   ASSERT_TRUE(RunExtensionSubtest(
       "filebrowser_component", "remote.html#" + GetPathOnMountPoint(""),
       kComponentFlags)) << message_;
 }