Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3150)

Unified Diff: chrome/browser/extensions/extension_local_filesystem_apitest.cc

Issue 9737001: Fix crash in AppNotifyChannelSetup when incognito profiles are used. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: forgot a file Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_local_filesystem_apitest.cc
diff --git a/chrome/browser/extensions/extension_local_filesystem_apitest.cc b/chrome/browser/extensions/extension_local_filesystem_apitest.cc
index 1e1dcdd96196740d6eb9ac44ed088d4df94372a9..344d015f11f78c3b2e0e5f9994b0d9dfa13a02a6 100644
--- a/chrome/browser/extensions/extension_local_filesystem_apitest.cc
+++ b/chrome/browser/extensions/extension_local_filesystem_apitest.cc
@@ -30,6 +30,10 @@ namespace {
// The ID of the file browser extension.
const char kFileBrowserExtensionId[] = "ddammdhioacbehjngdmkjcjbnfginlla";
+// Flags used to run the tests with a COMPONENT extension.
+const char kComponentFlags = ExtensionApiTest::kFlagEnableFileAccess |
+ ExtensionApitest::kFlagLoadAsComponent;
+
// Returns the expected URL for the given path.
GURL GetExpectedURL(const std::string& path) {
return GURL(
@@ -180,15 +184,15 @@ IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest, LocalFileSystem) {
IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest, FileBrowserTest) {
AddTmpMountPoint();
ASSERT_TRUE(RunExtensionTest("filesystem_handler")) << message_;
- ASSERT_TRUE(RunComponentExtensionSubtest("filebrowser_component",
- "read.html")) << 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(RunComponentExtensionSubtest("filebrowser_component",
- "write.html")) << message_;
+ ASSERT_TRUE(RunExtensionSubtest(
+ "filebrowser_component", "write.html", kComponentFlags)) << message_;
}
IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest,
@@ -196,8 +200,8 @@ IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest,
AddTmpMountPoint();
ASSERT_TRUE(RunComponentExtensionTest("filesystem_handler_write"))
<< message_;
- ASSERT_TRUE(RunComponentExtensionSubtest("filebrowser_component",
- "write.html")) << message_;
+ ASSERT_TRUE(RunExtensionSubtest(
+ "filebrowser_component", "write.html", kComponentFlags)) << message_;
}
IN_PROC_BROWSER_TEST_F(RemoteFileSystemExtensionApiTest, RemoteMountPoint) {
@@ -230,8 +234,8 @@ IN_PROC_BROWSER_TEST_F(RemoteFileSystemExtensionApiTest, RemoteMountPoint) {
test_file_path_,
scoped_refptr<webkit_blob::ShareableFileReference>(NULL)));
- ASSERT_TRUE(RunComponentExtensionSubtest("filebrowser_component",
- "remote.html")) << message_;
+ ASSERT_TRUE(RunExtensionSubtest(
+ "filebrowser_component", "remote.html", kComponentFlags)) << message_;
}
#endif

Powered by Google App Engine
This is Rietveld 408576698