summaryrefslogtreecommitdiffstats
path: root/webkit/fileapi
Commit message (Collapse)AuthorAgeFilesLines
...
* Cleanup SandboxedFileSystem* and merge them into FileSystem* for simplicity.kinuko@chromium.org2011-02-148-407/+259
| | | | | | | | | | | Based on our rough discussion over emails, I just went ahead and did the cleanup. BUG=none TEST=none Review URL: http://codereview.chromium.org/6471018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74786 0039d316-1c4b-4281-b951-d872f2087c98
* Add 1st cut of FileSystemUsageTracker that tracks the usage changes in ↵kinuko@chromium.org2011-02-108-29/+362
| | | | | | | | | | | | | | FileSystem API. For now it has no meaningful implementation yet; mostly just for defining a few interfaces. BUG= TEST=FileSystemUsageTrackerTest.DummyTest Review URL: http://codereview.chromium.org/6426001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74429 0039d316-1c4b-4281-b951-d872f2087c98
* Several bits of cleanup in FileSystem*URLRequestJob:adamk@chromium.org2011-02-095-21/+20
| | | | | | | | | | | | - Remove startup_error_ member and delayed error reporting; URLRequestJob properly enqueues NotifyDone calls so that they complete after Start() runs. - Remove origin_url_ members, replace with locals. - Re-organize .h file to call out FilterContext method. Review URL: http://codereview.chromium.org/6458011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74339 0039d316-1c4b-4281-b951-d872f2087c98
* First crack at FileSystemURLRequestJob for handling filesystem: URLs.adamk@chromium.org2011-02-0810-0/+1159
| | | | | | | | Disabled behind a switch, "--enable-filesystem-url-scheme". Review URL: http://codereview.chromium.org/6262015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74082 0039d316-1c4b-4281-b951-d872f2087c98
* Roll WebKit DEPS past WebKit move. Update gyp files and include paths to ↵abarth@chromium.org2011-01-175-11/+11
| | | | | | reflect the move. Consolidate how we DEPS in WebKit source files. Cross fingers. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71586 0039d316-1c4b-4281-b951-d872f2087c98
* net: Remove typedef net::URLRequestContext URLRequestContext;tfarina@chromium.org2011-01-152-4/+4
| | | | | | | | | BUG=64263 TEST=compiled locally, trybots Review URL: http://codereview.chromium.org/6338002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71522 0039d316-1c4b-4281-b951-d872f2087c98
* Add some initial OWNERS files for src/webkit/.darin@chromium.org2011-01-141-0/+3
| | | | | | | | R=ben Review URL: http://codereview.chromium.org/6327005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71492 0039d316-1c4b-4281-b951-d872f2087c98
* Added WARN_UNUSED_RESULT to ScopedTempDir methods.cbentzel@chromium.org2011-01-132-4/+2
| | | | | | | | | BUG=NONE TEST=all targets build, tests pass. Review URL: http://codereview.chromium.org/6042009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71322 0039d316-1c4b-4281-b951-d872f2087c98
* net: Add namespace net to the remaining files under url_request directory.tfarina@chromium.org2011-01-062-5/+5
| | | | | | | | | | | | It just adds the 'namespace net' to these files and a typedef for them, because there are many entries to fix in one pass. They will be fixed later. BUG=64263 TEST=trybots Review URL: http://codereview.chromium.org/6056007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70592 0039d316-1c4b-4281-b951-d872f2087c98
* Remove BrowserFileSystemContext class and merge it into ↵kinuko@chromium.org2010-12-144-12/+42
| | | | | | | | | | | SandboxedFileSystemContext BUG=60243 TEST=none Review URL: http://codereview.chromium.org/5633008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69089 0039d316-1c4b-4281-b951-d872f2087c98
* Move more code from headers to implementation.erg@google.com2010-12-072-0/+3
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5624002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68534 0039d316-1c4b-4281-b951-d872f2087c98
* Delete FileSystem API data when uninstalling extensions.kinuko@chromium.org2010-12-033-7/+31
| | | | | | | | | BUG=63700 TEST=load an extension that keeps writing to file system and unload it. See if the origin data under the profile is deleted. Review URL: http://codereview.chromium.org/5272010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68143 0039d316-1c4b-4281-b951-d872f2087c98
* net: Remove typedef net::URLRequest URLRequest;tfarina@chromium.org2010-11-303-17/+20
| | | | | | | | | BUG=64263 TEST=compiled locally, trybots Review URL: http://codereview.chromium.org/5384002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67762 0039d316-1c4b-4281-b951-d872f2087c98
* Make FileSystemOperation's lifetime more explicit.kinuko@chromium.org2010-11-305-197/+180
| | | | | | | | | | | | | | | In the current code calling dispatcher->DidXxx in an operation's DidXxx method MAY indirectly delete the operation itself depending on the dispatcher's implementation. I was confused by this several times and I want to make this flow more explicit. This patch lets FileSystemOperation control its lifetime by itself so that each callback dispatcher implementation does not need to take care of it. Also moved BrowserFileSystemCallbackDispatcher into file_system_dispatcher_host.cc as it's only used in it and its implementation is tightly coupled with the DispatcherHost. BUG=60243 TEST=none Review URL: http://codereview.chromium.org/4821005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67732 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 67385 - On windows filepaths need \\?\ prefix to allow extended file ↵kinuko@chromium.org2010-11-304-48/+4
| | | | | | | | | | | | | | | | | path names. Fix to bug 63574. Turned out that we need more changes to make extended paths work. BUG=63574 TEST=file_system_operation_unittest.cc and manually tested on FAT32. Review URL: http://codereview.chromium.org/5259003 TBR=kkanetkar@chromium.org Review URL: http://codereview.chromium.org/5357009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67703 0039d316-1c4b-4281-b951-d872f2087c98
* net: Add namespace net to URLRequest and URLRequestJob classes.tfarina@chromium.org2010-11-281-2/+5
| | | | | | | | | BUG=64263 TEST=compiled locally and trybots Review URL: http://codereview.chromium.org/5298008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67497 0039d316-1c4b-4281-b951-d872f2087c98
* On windows filepaths need \\?\ prefix to allow extended file path names. Fix ↵kkanetkar@chromium.org2010-11-254-4/+48
| | | | | | | | | | | | to bug 63574. BUG=63574 TEST=file_system_operation_unittest.cc and manually tested on FAT32. Review URL: http://codereview.chromium.org/5259003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67385 0039d316-1c4b-4281-b951-d872f2087c98
* Extend simple_file_system to use SandboxedFileSystemOperationkinuko@chromium.org2010-11-232-19/+4
| | | | | | | | | | | | | | so that most of the code paths that run in chromium can be tested in test_shell. Also removed SandboxedFileSystemContext::CheckIfFilePathIsSafe that is not used at all. BUG=60243 TEST=fast/filesystem Review URL: http://codereview.chromium.org/4879001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67158 0039d316-1c4b-4281-b951-d872f2087c98
* FileSystem code cleanup 2nd cut - introduce SandboxedFileSystemOperationkinuko@chromium.org2010-11-106-43/+424
| | | | | | | | | | | | | 1. Introduced SandboxedFileSystemOperation. 2. Factored out most of the PathManager/QuotaManager related code from FileSystemDispatcherHost to the SandboxedFileSystemOperation. BUG=60243 TEST=none Review URL: http://codereview.chromium.org/4054003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65598 0039d316-1c4b-4281-b951-d872f2087c98
* FileSystem code cleanup 1st cut - does some class renaming.kinuko@chromium.org2010-11-095-60/+80
| | | | | | | | | | | | Renamed FileSystemQuota to FileSystemQuotaManager. Removed PlatFormErrorToFileError in simple_file_system.cc (in favor of webkit_flue::PlatformErrorToFileError) BUG=60243 TEST=none Review URL: http://codereview.chromium.org/4017007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65506 0039d316-1c4b-4281-b951-d872f2087c98
* Support FileSystem API in extensionskinuko@chromium.org2010-11-061-0/+3
| | | | | | | | | BUG=61534 TEST=see if the feature works in an extension Review URL: http://codereview.chromium.org/4439002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65304 0039d316-1c4b-4281-b951-d872f2087c98
* Moving Entry from base::file_util_proxy namespace to within class ↵kkanetkar@chromium.org2010-11-024-10/+12
| | | | | | | | | | | | base::FileUtilProxy BUG=None TEST=None Review URL: http://codereview.chromium.org/4261001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64806 0039d316-1c4b-4281-b951-d872f2087c98
* Convert implicit scoped_refptr constructor calls to explicit ones, part 1thakis@chromium.org2010-11-011-2/+2
| | | | | | | | | | | This CL was created automatically by this clang rewriter: http://codereview.appspot.com/2776043/ . I manually fixed a few rough spots of the rewriter output (doh1-3) and fixed all presubmit errors. BUG=28083 TEST=None Review URL: http://codereview.chromium.org/4192012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64573 0039d316-1c4b-4281-b951-d872f2087c98
* Change a couple of occurances of ASSERT_TRUE(false) to FAIL()michaeln@chromium.org2010-10-301-2/+2
| | | | | | | | | | TEST=none BUG=none TBR=phajdan.jr Review URL: http://codereview.chromium.org/3896005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64532 0039d316-1c4b-4281-b951-d872f2087c98
* Release GetFileSystemRootPath callback on the correct thread.kinuko@chromium.org2010-10-281-0/+1
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/4142009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64304 0039d316-1c4b-4281-b951-d872f2087c98
* This is the second try at submitting ↵ericu@google.com2010-10-251-0/+4
| | | | | | | | | | | | | | | http://codereview.chromium.org/3991007/show; I've removed the extra renderer cleanup that turned out to break in the test environment. Workers who want to use FileWriter need to be granted permission to access files in the user's FileSystem directory. BUG=none; you need tests I haven't checked in yet to repro this problem. TEST=none; tests are in review, but will go in after this fix. Review URL: http://codereview.chromium.org/4057004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63781 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 63619 - Workers who want to use FileWriter need to be granted ↵ericu@google.com2010-10-231-4/+0
| | | | | | | | | | | | | | permission to access files in the user's FileSystem directory. BUG=none; you need tests I haven't checked in yet to repro this problem. TEST=none; tests are in review, but will go in after this fix. Review URL: http://codereview.chromium.org/3991007 TBR=ericu@google.com Review URL: http://codereview.chromium.org/3996009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63623 0039d316-1c4b-4281-b951-d872f2087c98
* Workers who want to use FileWriter need to be granted permission to access ↵ericu@google.com2010-10-231-0/+4
| | | | | | | | | | | files in the user's FileSystem directory. BUG=none; you need tests I haven't checked in yet to repro this problem. TEST=none; tests are in review, but will go in after this fix. Review URL: http://codereview.chromium.org/3991007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63619 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Even more ctor/virtual deinlining.erg@google.com2010-10-193-2/+11
| | | | | | | | | | | (Only 424k off Linux debug .a files). BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3859003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63059 0039d316-1c4b-4281-b951-d872f2087c98
* WebFileWriterBase unit testsmichaeln@chromium.org2010-10-192-5/+428
| | | | | | | | | | BUG=none TEST=webfilewriter_base_unittest.cc Review URL: http://codereview.chromium.org/3806009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63007 0039d316-1c4b-4281-b951-d872f2087c98
* Hide the FileSystem directory under 'unpredictable' location (part 2)kinuko@chromium.org2010-10-154-141/+495
| | | | | | | | | BUG=58361 TEST=FileSystemPathManager.* Review URL: http://codereview.chromium.org/3724001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62698 0039d316-1c4b-4281-b951-d872f2087c98
* Rename FileUtilProxy::Create to EnsureFileExistskinuko@chromium.org2010-10-152-26/+19
| | | | | | | | | | | | | | | | | | per follow-up discussion on http://codereview.chromium.org/3717001/show Rename FileUtilProxy::Create to EnsureFileExists to make it clear that it doesn't open the file (or doesn't leave a file handle opened). Also fixes CreatePlatformFile to set |created| correctly when PLATFORM_FILE_CREATE flag is given (I think this is the correct behavior). BUG=none TEST=FileSystemOperationTest.* Review URL: http://codereview.chromium.org/3743004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62683 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: More ctor/dtor cleanup.erg@google.com2010-10-142-1/+7
| | | | | | | | | | | (3.5 megs off debug linux .a files) BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3806005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62654 0039d316-1c4b-4281-b951-d872f2087c98
* SimpleFileWriter for test_shell and DRT.michaeln@chromium.org2010-10-144-0/+218
| | | | | | | | | BUG=none TEST=manual Review URL: http://codereview.chromium.org/3683004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62486 0039d316-1c4b-4281-b951-d872f2087c98
* Switch FileSystemOperation::Write back to using FileUtilProxy::CreateOrOpen, ↵ericu@google.com2010-10-131-1/+1
| | | | | | | | | | | which returns a file handle, instead of FileUtilProxy::Create, which doesn't. BUG=59073 TEST=tests are currently disabled until the TestShell implementation goes in, but you can try WebKit/LayoutTests/fast/filesystem/file-writer-gc-blob.html in Chromium with --enable-file-system. Review URL: http://codereview.chromium.org/3758002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62436 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor out path-related methods from file_system_host_context into a ↵kinuko@chromium.org2010-10-124-0/+494
| | | | | | | | | | | | | | | | separated module. No functionality changes; code relocation/cleanup only. This change is necessary to improve test coverage for another coming change (hide the FileSystem's root directory under 'unpredictrable' location). BUG=none TEST=FileSystemPathManager.* Review URL: http://codereview.chromium.org/3703003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62259 0039d316-1c4b-4281-b951-d872f2087c98
* FileWriter: Defend against being deleted in the FileWriterDelegate ↵michaeln@chromium.org2010-10-121-4/+5
| | | | | | | | | | | completion callback. BUG=none TEST=none Review URL: http://codereview.chromium.org/3690003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62231 0039d316-1c4b-4281-b951-d872f2087c98
* Make sure we close the file_handle when we create (but not open) a new filekinuko@chromium.org2010-10-111-2/+2
| | | | | | | | | | | | 1. Add FileUtilProxy::Create that does not leave a file_handle opened. 2. Update the file_util_operation to use FileUtilProxy::Create instead of FileUtilProxy::CreateOrOpen so that no other tasks get queued in before we close the handle. BUG=58424,58473 TEST=fast/filesystem/ Review URL: http://codereview.chromium.org/3717001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62192 0039d316-1c4b-4281-b951-d872f2087c98
* Replace DCHECK in the FileSystemQuota test code with ASSERT_TRUE.kinuko@chromium.org2010-10-081-3/+3
| | | | | | | | | | BUG=none TEST=FileSystemQuotaTest.* TBR=phajdan Review URL: http://codereview.chromium.org/3638002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62017 0039d316-1c4b-4281-b951-d872f2087c98
* Allow unlimited quota for apps for FileSystem accesskinuko@chromium.org2010-10-084-0/+225
| | | | | | | | | | | | | | | | | | | | - allow unlimited access for apps/extensions that have "unlimited_storage" permission. - disallow any write access that may increase the filesystem usage (i.e. copy/move/create/write). for others. - allow unlimit access for file:/// URIs only if --allow-file-from-files flag is given. BUG=57211 TEST=FileSystemQuota.* TEST=Load a remote test page and verify that it throws QUOTA_EXCEEDED_ERR (22) for any write access. TEST=Load an app/extension page that has "unlimited_storage" permission and verify that any write access is allowed. TEST=Disable or uninstall the app/extension and do the same. Verify that it throws QUOTA_EXCEEDED_ERR (22) for any write access. TEST=Launch chromium without --allow-file-from-files flag, load a local test page (with file:/// URI) and verify that it throws SECURITY_ERR (18) for requesetFileSystem. TEST=Launch chromium with --allow-file-from-files flag and do the same. Verify that requestFileSystem returns a valid filesystem and any write access is allowed. Review URL: http://codereview.chromium.org/3561016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61934 0039d316-1c4b-4281-b951-d872f2087c98
* Resubmit of http://codereview.chromium.org/3476002/show but without the ↵ericu@google.com2010-10-076-11/+307
| | | | | | | | | | crashing bug. BUG=none TEST=none Review URL: http://codereview.chromium.org/3618016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61879 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 61833 - Add the actual writing-files part of FileWriter.ericu@google.com2010-10-075-305/+11
| | | | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3476002 TBR=ericu@google.com Review URL: http://codereview.chromium.org/3630002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61842 0039d316-1c4b-4281-b951-d872f2087c98
* Add the actual writing-files part of FileWriter.ericu@google.com2010-10-075-11/+305
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3476002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61833 0039d316-1c4b-4281-b951-d872f2087c98
* Reapply 61613 that was reverted for false valgrind alarm.kinuko@chromium.org2010-10-073-58/+212
| | | | | | | | | | | | | Original issue: http://codereview.chromium.org/3567012/show BUG=none TEST=FileSystemOperationTest.* TBR=ericu,thestig Review URL: http://codereview.chromium.org/3603014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61771 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 61613 - possible culprit for valgrind error. (Support ↵kinuko@chromium.org2010-10-063-212/+58
| | | | | | | | | | | removeRecursively and new copy/move) BUG=none TEST=none Review URL: http://codereview.chromium.org/3576017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61713 0039d316-1c4b-4281-b951-d872f2087c98
* Second try for support removeRecursively and new copy/move behaviors added ↵kinuko@chromium.org2010-10-063-58/+212
| | | | | | | | | | | | | | | | | | | | | | to the spec recently. http://lists.w3.org/Archives/Public/public-webapps/2010JulSep/1101.html > For a move/copy of a file on top of existing file, or a directory on > top of an existing empty directory, you get an automatic overwrite. > A move/copy of a file on top of an existing directory, or of a > directory on top of an existing file, will always fail. > A move/copy of a file or directory on top of an existing non-empty > directory will always fail. original issue: http://codereview.chromium.org/3567012 BUG=none TEST=FileSystemOperationTest.* TBR=ericu Review URL: http://codereview.chromium.org/3531012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61613 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 61480 - Support removeRecursively and new copy/move behaviors added ↵kinuko@chromium.org2010-10-053-208/+42
| | | | | | | | | | | | | | | | | | | | | to the spec recently. http://lists.w3.org/Archives/Public/public-webapps/2010JulSep/1101.html > For a move/copy of a file on top of existing file, or a directory on > top of an existing empty directory, you get an automatic overwrite. > A move/copy of a file on top of an existing directory, or of a > directory on top of an existing file, will always fail. > A move/copy of a file or directory on top of an existing non-empty > directory will always fail. BUG=32277 TEST=FileSystemOperationTest.* Review URL: http://codereview.chromium.org/3567012 TBR=kinuko@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61482 0039d316-1c4b-4281-b951-d872f2087c98
* Marks Test{Move,Copy}FailureContainsPath FAILS on Windows, also makes a fix ↵kinuko@chromium.org2010-10-051-5/+17
| | | | | | | | | | | | | | attempt for the failure. This patch also contains an attempt to fix the failure. TBR=ericu BUG=none TEST=green bot Review URL: http://codereview.chromium.org/3516014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61481 0039d316-1c4b-4281-b951-d872f2087c98
* Support removeRecursively and new copy/move behaviors added to the spec ↵kinuko@chromium.org2010-10-053-42/+208
| | | | | | | | | | | | | | | | | | | recently. http://lists.w3.org/Archives/Public/public-webapps/2010JulSep/1101.html > For a move/copy of a file on top of existing file, or a directory on > top of an existing empty directory, you get an automatic overwrite. > A move/copy of a file on top of an existing directory, or of a > directory on top of an existing file, will always fail. > A move/copy of a file or directory on top of an existing non-empty > directory will always fail. BUG=32277 TEST=FileSystemOperationTest.* Review URL: http://codereview.chromium.org/3567012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61480 0039d316-1c4b-4281-b951-d872f2087c98
* Second try at submitting 61462.ericu@google.com2010-10-054-52/+144
| | | | | | | | | BUG=none TEST=in file_system_operation_unittest.cc Review URL: http://codereview.chromium.org/3526018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61468 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-01-07 06:59:22 +0000