summaryrefslogtreecommitdiffstats
path: root/webkit/fileapi
Commit message (Collapse)AuthorAgeFilesLines
* Merge 82274 - Clang made happy again.TEST=noneBUG=noneReview URL: ↵kerz@chromium.org2011-04-202-1/+5
| | | | | | | | | http://codereview.chromium.org/6882075 TBR=zelidrag@chromium.org Review URL: http://codereview.chromium.org/6879092 git-svn-id: svn://svn.chromium.org/chrome/branches/742/src@82376 0039d316-1c4b-4281-b951-d872f2087c98
* Merge 82269 - Build break fix for Clang builds.TEST=noneBUG=noneReview URL: ↵kerz@chromium.org2011-04-201-0/+1
| | | | | | | | | http://codereview.chromium.org/6877079 TBR=zelidrag@chromium.org Review URL: http://codereview.chromium.org/6879091 git-svn-id: svn://svn.chromium.org/chrome/branches/742/src@82373 0039d316-1c4b-4281-b951-d872f2087c98
* Merge 82266 - Fixed file/directory url resolution for external mount point ↵kerz@chromium.org2011-04-2030-190/+552
| | | | | | | | | provider.Per Eric's request, refactored FileSystemDirURLRequestJob and FileSystemURLRequestJob classes to resolve local file system through a new operation.BUG=chromium-os:14225TEST=added new test cases to FileSystemPathManagerTest.*, added FileSystemOperationTest.TestGetLocalFilePathSuccessReview URL: http://codereview.chromium.org/6864040 TBR=zelidrag@chromium.org Review URL: http://codereview.chromium.org/6882102 git-svn-id: svn://svn.chromium.org/chrome/branches/742/src@82372 0039d316-1c4b-4281-b951-d872f2087c98
* Add 'Cache-Control: no-cache' to filesystem: responses to avoidadamk@chromium.org2011-04-193-3/+39
| | | | | | | | | | | | | | | | | | | | having them improperly cached in WebKit's memory cache. Note that this change as currently written may be unnecessarily bad for performance, since the file must be read from disk on every access. For some use cases, this could be quite expensive. The right fix is probably to put a 'Last-Modified' header in the response along with 'Cache-Control: max-age: 0', but for now I'm favoring correctness over performance. R=michaeln@chromium.org BUG=79539 TEST=layout test at http://webkit.org/b/58854 Review URL: http://codereview.chromium.org/6879022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82156 0039d316-1c4b-4281-b951-d872f2087c98
* File API changes needed for safely passing user selected file entities from ↵zelidrag@chromium.org2011-04-1612-50/+111
| | | | | | | | | | the file browser component extension to a 3rd party extension. BUG=chromium-os:11996 TEST=FileAccessPermissionsTest.FileAccessChecks Review URL: http://codereview.chromium.org/6810037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81860 0039d316-1c4b-4281-b951-d872f2087c98
* Clang build fix, de-inline a virtual method and dtor.michaeln@google.com2011-04-163-5/+23
| | | | | | | TBR=ericu Review URL: http://codereview.chromium.org/6870022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81847 0039d316-1c4b-4281-b951-d872f2087c98
* Let Pepper open FileSystem files again.ericu@google.com2011-04-163-0/+82
| | | | | | | | TEST=none [existing tests, but they're not currently run automatically] BUG=none Review URL: http://codereview.chromium.org/6850027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81837 0039d316-1c4b-4281-b951-d872f2087c98
* Nit fix FileSystemOperationWriteTest about when exiting the thread and ↵dmikurube@chromium.org2011-04-141-6/+3
| | | | | | | | | | | | remove unnecessary lines. BUG=none TEST=none Review URL: http://codereview.chromium.org/6852021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81571 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leaks below file_util::FileEnumeratordmikurube@chromium.org2011-04-141-0/+2
| | | | | | | | | BUG=79329 TEST=./tools/valgrind/chrome_tests.sh -t test_shell --gtest_filter="FileSystemOperationTest.*" Review URL: http://codereview.chromium.org/6851010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81548 0039d316-1c4b-4281-b951-d872f2087c98
* More filesystem cleanup: convert URL-encoded-as-FilePath to actual URL, where ericu@google.com2011-04-1314-337/+242
| | | | | | | | | | | | | possible without WebKit API changes. The WebKit changes will happen in another CL. This is a resubmit of http://codereview.chromium.org/6767010/, which bounced recently. There are a few changes here that weren't there [most notably in file_system_operation_write_unittest.cc and file_system_operation.cc], but they're pretty trivial build/test fixes. Review URL: http://codereview.chromium.org/6833007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81463 0039d316-1c4b-4281-b951-d872f2087c98
* Extract a recursive copy function as a shared and non-virtual function.dmikurube@chromium.org2011-04-132-11/+152
| | | | | | | | | | | | | This extraction is required to stack FileUtil layers without duplicated code of recursive copy. No tests because it doesn't change any behavior. BUG=none TEST=none Review URL: http://codereview.chromium.org/6772005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81388 0039d316-1c4b-4281-b951-d872f2087c98
* Fix FileSystemOperationTest for writing operations to avoid conflicting ↵dmikurube@chromium.org2011-04-132-35/+41
| | | | | | | | | | | | class names. BUG=none TEST=FileSystemOperationWriteTest.* Review URL: http://codereview.chromium.org/6835015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81382 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 81319 - More filesystem cleanup: convert URL-encoded-as-FilePath to ↵ericu@google.com2011-04-1214-242/+337
| | | | | | | | | actual URL, where possible without WebKit API changes. The WebKit changes will happen in another CL. This is a resubmit of http://codereview.chromium.org/6767010/ [third try lucky,fourth try persistent?], which bounced most recently due to bugs manifestingonly on XP. There are a few changes here that weren't there [infile_system_operation_write_unittest.cc and file_system_operation.cc], butthey're pretty trivial build/test fixes.BUG=none TEST=noneReview URL: http://codereview.chromium.org/6821065 TBR=ericu@google.com Review URL: http://codereview.chromium.org/6836002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81320 0039d316-1c4b-4281-b951-d872f2087c98
* More filesystem cleanup: convert URL-encoded-as-FilePath to actual URL, where ericu@google.com2011-04-1214-337/+242
| | | | | | | | | | | | | | | | | possible without WebKit API changes. The WebKit changes will happen in another CL. This is a resubmit of http://codereview.chromium.org/6767010/ [third try lucky, fourth try persistent?], which bounced most recently due to bugs manifesting only on XP. There are a few changes here that weren't there [in file_system_operation_write_unittest.cc and file_system_operation.cc], but they're pretty trivial build/test fixes. BUG=none TEST=none Review URL: http://codereview.chromium.org/6821065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81319 0039d316-1c4b-4281-b951-d872f2087c98
* Modify tests for FileSystemOperation to test directory-in-directory.dmikurube@chromium.org2011-04-111-6/+20
| | | | | | | | | | BUG=none TEST=FileSystemOperationTest.Test*SuccessSrcDirRecursive Review URL: http://codereview.chromium.org/6816048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81071 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 81028 - It broke LayoutTests/ fast/filesystem/op-copy.html on Windows ↵tkent@chromium.org2011-04-1111-221/+322
| | | | | | | | | | | Release and Debug. More filesystem cleanup: convert URL-encoded-as-FilePath to actual URL, where possible without WebKit API changes. The WebKit changes will happen in another CL. This is a resubmit of http://codereview.chromium.org/6767010/, which bounced dueto a recent checkin that required a merge. There are a few changes here thatweren't there [in file_system_operation_write_unittest.cc andfile_system_operation.cc], but they're pretty trivial build/test fixes.BUG=none TEST=noneReview URL: http://codereview.chromium.org/6813066 TBR=ericu@google.com Review URL: http://codereview.chromium.org/6813072 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81065 0039d316-1c4b-4281-b951-d872f2087c98
* More filesystem cleanup: convert URL-encoded-as-FilePath to actual URL, where ericu@google.com2011-04-0911-322/+221
| | | | | | | | | | | | | | | | possible without WebKit API changes. The WebKit changes will happen in another CL. This is a resubmit of http://codereview.chromium.org/6767010/, which bounced due to a recent checkin that required a merge. There are a few changes here that weren't there [in file_system_operation_write_unittest.cc and file_system_operation.cc], but they're pretty trivial build/test fixes. BUG=none TEST=none Review URL: http://codereview.chromium.org/6813066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81028 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 80982, tests failing on Win XPdmichael@chromium.org2011-04-0811-222/+325
| | | | | | | | | TBR=ericu BUG=None TEST=None Review URL: http://codereview.chromium.org/6821020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80991 0039d316-1c4b-4281-b951-d872f2087c98
* More filesystem cleanup: convert URL-encoded-as-FilePath to actual URL, where ericu@google.com2011-04-0811-325/+222
| | | | | | | | | | | | | | | | possible without WebKit API changes. The WebKit changes will happen in another CL. This is a resubmit of http://codereview.chromium.org/6767010/, which bounced due to a recent checkin that required a merge. There are a few changes here that weren't there [in file_system_operation_write_unittest.cc and file_system_operation.cc], but they're pretty trivial build/test fixes. BUG=none TEST=none Review URL: http://codereview.chromium.org/6812040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80982 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 80833 - More filesystem cleanup: convert URL-encoded-as-FilePath to ↵ericu@google.com2011-04-0710-199/+319
| | | | | | | | | actual URL, wherepossible without WebKit API changes. The WebKit changes will happen in anotherCL.BUG=noneTEST=noneReview URL: http://codereview.chromium.org/6767010 TBR=ericu@google.com Review URL: http://codereview.chromium.org/6813025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80835 0039d316-1c4b-4281-b951-d872f2087c98
* More filesystem cleanup: convert URL-encoded-as-FilePath to actual URL, whereericu@google.com2011-04-0710-319/+199
| | | | | | | | | | | possible without WebKit API changes. The WebKit changes will happen in another CL. BUG=none TEST=none Review URL: http://codereview.chromium.org/6767010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80833 0039d316-1c4b-4281-b951-d872f2087c98
* Add a unittest for FileSystemOperation::Write().dmikurube@chromium.org2011-04-072-2/+253
| | | | | | | | | | BUG=none TEST=FileSystemOperationTest.TestWrite Review URL: http://codereview.chromium.org/6770004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80735 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the path from PlatformFileInfo; it's cleaner just to pass it along as aericu@google.com2011-03-2810-21/+49
| | | | | | | | | | separate parameter, and not clutter up a base type. BUG:none TEST:none Review URL: http://codereview.chromium.org/6731033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79624 0039d316-1c4b-4281-b951-d872f2087c98
* Move some files from base to base/memory.levin@chromium.org2011-03-2822-48/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | raw_scoped_refptr_mismatch_checker.h ref_counted.cc ref_counted.h ref_counted_memory.cc ref_counted_memory.h ref_counted_unittest.cc scoped_callback_factory.h scoped_comptr_win.h scoped_handle.h scoped_native_library.cc scoped_native_library.h scoped_native_library_unittest.cc scoped_nsobject.h scoped_open_process.h scoped_ptr.h scoped_ptr_unittest.cc scoped_temp_dir.cc scoped_temp_dir.h scoped_temp_dir_unittest.cc scoped_vector.h singleton.h singleton_objc.h singleton_unittest.cc linked_ptr.h linked_ptr_unittest.cc weak_ptr.cc weak_ptr.h weak_ptr_unittest.cc BUG=None TEST=Compile Review URL: http://codereview.chromium.org/6714032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79524 0039d316-1c4b-4281-b951-d872f2087c98
* Wired local file system support for File API. The local file system provider ↵zelidrag@chromium.org2011-03-2516-21/+115
| | | | | | | | | | | | | | | currently exists for ChromeOS builds only. This CL exposes new extension permission 'fileSystem' that controls access to individual local file system elements from 3rd party extensions. Another new permission 'fileBrowserPrivate' controls access to following API call that retrieves root DOMFileSystem instance for locally exposed folders:  chrome.fileBrowserPrivate.requestLocalFileSystem(callback) BUG=chromium-os:11983 TEST=ExtensionApiTest.LocalFileSystem Review URL: http://codereview.chromium.org/6519040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79451 0039d316-1c4b-4281-b951-d872f2087c98
* Proper build fix.ericu@google.com2011-03-244-2/+10
| | | | | | | | | BUG=none TEST=none TBR=cmp Review URL: http://codereview.chromium.org/6734001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79234 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 79230 - Fix the clang build.ericu@google.com2011-03-242-7/+2
| | | | | | | | | | | BUG=none TEST=none TBR=cmp TBR=ericu@google.com Review URL: http://codereview.chromium.org/6733014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79231 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the clang build.ericu@google.com2011-03-242-2/+7
| | | | | | | | BUG=none TEST=none TBR=cmp git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79230 0039d316-1c4b-4281-b951-d872f2087c98
* Stop returning the true root path of each filesystem from openFileSystem.ericu@google.com2011-03-2424-529/+1410
| | | | | | | | | | | Instead, return the FileSystem URI of the root. This will make it easier to swap in different filesystem implementations. BUG=71635 TEST=Just a couple in FileSystemUtilTests, but a bunch of existing ones [this doesn't add much new functionality]. Review URL: http://codereview.chromium.org/6603034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79228 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leak in FileWriterDelegate by passing FileStream::Write a ↵adamk@chromium.org2011-03-152-9/+8
| | | | | | | | | | | | | | callback owned by the delegate. Remove the suppression of this memory leak. R=ericu@chromium.org BUG=75741 TEST=none Review URL: http://codereview.chromium.org/6685056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78148 0039d316-1c4b-4281-b951-d872f2087c98
* Re-enable FileWriterSync ui_tests by adding a ScopedAllowIO to ↵adamk@chromium.org2011-03-101-1/+4
| | | | | | | | | | | BlobURLRequestJob. BUG=75548 TEST=ui_tests Review URL: http://codereview.chromium.org/6666001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77705 0039d316-1c4b-4281-b951-d872f2087c98
* Virtualize file_util calls from ↵dmikurube@google.com2011-03-082-63/+96
| | | | | | | | | | | | | FileSystemFileUtil::PerformCommonCheckAndPreparationForMoveAndCopy. It refactors the TODO issue at http://codereview.chromium.org/6604020/ . BUG=74841 TEST=none Review URL: http://codereview.chromium.org/6625031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77375 0039d316-1c4b-4281-b951-d872f2087c98
* Make FileSystemDirURLRequestJob test single-threaded in hopes of making it ↵adamk@chromium.org2011-03-081-14/+7
| | | | | | | | | | | less flaky under Valgrind. BUG=75251 TEST=none Review URL: http://codereview.chromium.org/6646001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77333 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed the bug 75018: incorrect conditional jump in ↵kinuko@chromium.org2011-03-081-1/+3
| | | | | | | | | | | fileapi::FileSystemUsageCache::Read. BUG=75018 TEST=none Review URL: http://codereview.chromium.org/6621061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77239 0039d316-1c4b-4281-b951-d872f2087c98
* Revive FileAPI's recursive CreateDirectory.kinuko@chromium.org2011-03-076-12/+21
| | | | | | | | | | | PPAPI's MakeDirectoryIncludingAncestors() (ppapi/cpp/dev/file_ref_dev.cc) uses recursive CreateDirectory. BUG=74841 TEST=none Review URL: http://codereview.chromium.org/6627028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77204 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce FileSystemFileUtil and -Proxy to decorate base::file_util in ↵kinuko@chromium.org2011-03-049-28/+1150
| | | | | | | | | | | webkit/fileapi. BUG=74841 TEST=none Review URL: http://codereview.chromium.org/6604020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76875 0039d316-1c4b-4281-b951-d872f2087c98
* Remove FileSystemQuotaManager classkinuko@chromium.org2011-03-049-224/+129
| | | | | | | | | | | Now that we have SpecialStoragePolicy class and are going to have QuotaFileSystemUtil, I think we're ready to get rid of this class. BUG=none TEST=none Review URL: http://codereview.chromium.org/6609009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76858 0039d316-1c4b-4281-b951-d872f2087c98
* Replace EXPECT_EQ(false, ...) with EXPECT_FALSE(...) to fix clang buildkinuko@chromium.org2011-03-031-2/+2
| | | | | | | | | BUG=none TEST=green tree Review URL: http://codereview.chromium.org/6614024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76807 0039d316-1c4b-4281-b951-d872f2087c98
* Add a function for usage tracking.kinuko@chromium.org2011-03-036-8/+537
| | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/6453001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76797 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 76578 (accidental commit by the commit bot) - Add a function for ↵thakis@chromium.org2011-03-026-551/+8
| | | | | | | | | | | | | | usage tracking. It's based on the bug 6426001. BUG= TEST= Review URL: http://codereview.chromium.org/6453001 TBR=dmikurube@google.com Review URL: http://codereview.chromium.org/6588129 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76594 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 76586 - Fix clang build.thakis@chromium.org2011-03-021-12/+22
| | | | | | | | | | | BUG=none TEST=none TBR=jcivelli,dmikurube TBR=thakis@chromium.org Review URL: http://codereview.chromium.org/6597116 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76592 0039d316-1c4b-4281-b951-d872f2087c98
* Fix clang build.thakis@chromium.org2011-03-021-22/+12
| | | | | | | | BUG=none TEST=none TBR=jcivelli,dmikurube git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76586 0039d316-1c4b-4281-b951-d872f2087c98
* Add a function for usage tracking. It's based on the bug 6426001.dmikurube@google.com2011-03-026-8/+551
| | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/6453001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76578 0039d316-1c4b-4281-b951-d872f2087c98
* Pure pedantry: Replace all ".size() == 0" with ".empty()".erg@google.com2011-03-021-1/+1
| | | | | | | | | BUG=carnitas TEST=compiles; existing unit tests. Review URL: http://codereview.chromium.org/6602049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76467 0039d316-1c4b-4281-b951-d872f2087c98
* Add an accessor for an ExtensionSpecialStoragePolicy to the Profile classmichaeln@google.com2011-02-265-142/+76
| | | | | | | | | | | and use it in the extension service, data remover, and storage subsystems. BUG=52357 TEST=extension_service_unittest.cc Review URL: http://codereview.chromium.org/6551028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76126 0039d316-1c4b-4281-b951-d872f2087c98
* Properly return TYPE_MISMATCH_ERR for DirectoryExists/FileExistsadamk@chromium.org2011-02-252-2/+16
| | | | | | | | | | | | | | operation when the target isn't a directory/file, respectively. This matches what the File API spec says about error codes: http://dev.w3.org/2009/dap/file-system/file-dir-sys.html#error-code-descriptions BUG=none TEST=none Review URL: http://codereview.chromium.org/6591013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76098 0039d316-1c4b-4281-b951-d872f2087c98
* Add a utility method that returns a list of origins that have FileSystem ↵kinuko@chromium.org2011-02-223-8/+131
| | | | | | | | | | | | | storage. The method is to be used to retrieve total amount of usage for a type, to collect all the origins at startup after initialization etc. BUG=none TEST=FileSystemUtilListOriginsTest Review URL: http://codereview.chromium.org/6524038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75582 0039d316-1c4b-4281-b951-d872f2087c98
* Display only the sandboxed portion of the URL when listingadamk@chromium.org2011-02-171-3/+3
| | | | | | | | | | a filesystem: URL that refers to a directory. This avoids leaking implementation details of the sandboxed filesystem. Review URL: http://codereview.chromium.org/6537011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75329 0039d316-1c4b-4281-b951-d872f2087c98
* Make more methods of FileSystemPathManager static and portable.kinuko@chromium.org2011-02-166-64/+66
| | | | | | | | | | | | | Changes: - Renamed StorageIdentifier to OriginIdentifier to better reflect its meaning. - Changed a few methods to take string origin identifier but not origin URL as its parameter, since in some cases it seems to be more convenient just to use the identifier string (which can be easily extracted from the path) than to call WebKit method to convert back the identifier to GURL. BUG=none TEST=FileSystemPathManagerTest.* Review URL: http://codereview.chromium.org/6483017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75077 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use a separate FILE thread in FileSystemURLRequestJobTest.adamk@chromium.org2011-02-141-10/+2
| | | | | | | | | | | | Instead, just pass FileUtilProxy a MessageLoopProxy pointing to the current thread. This gets rid of a memory leak under Valgrind, and should make the test less flaky in the long run. BUG=72541 Review URL: http://codereview.chromium.org/6474022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74854 0039d316-1c4b-4281-b951-d872f2087c98