Skip to content
Snippets Groups Projects
Commit aa9f5488 authored by Alex Gaynor's avatar Alex Gaynor
Browse files

Bug 1367560 - Remove an assertion from the Directory constructor that the...

Bug 1367560 - Remove an assertion from the Directory constructor that the directory exists on disk from DEBUG builds; r=baku

As long as Directory objects can be constructed in the content process, trying
to access the file system conceptually does not make sense because of
sandboxing. The canary in the coal mine for this was tests which started
crashing on Directory construction as we further locked down read access in the
sandbox.

MozReview-Commit-ID: HZitALYpO5H

--HG--
extra : rebase_source : 1f7e0361e40e45a9c23d5d78bdcdc1696d44d298
parent 221a21cb
No related branches found
No related tags found
No related merge requests found
......@@ -86,12 +86,6 @@ Directory::Create(nsISupports* aParent, nsIFile* aFile,
MOZ_ASSERT(aParent);
MOZ_ASSERT(aFile);
#ifdef DEBUG
bool isDir;
nsresult rv = aFile->IsDirectory(&isDir);
MOZ_ASSERT(NS_SUCCEEDED(rv) && isDir);
#endif
RefPtr<Directory> directory = new Directory(aParent, aFile, aFileSystem);
return directory.forget();
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment