summaryrefslogtreecommitdiffstats
path: root/webkit/blob
Commit message (Collapse)AuthorAgeFilesLines
* webkit: Merge blob and fileapi into one build target 'webkit_storage'hashimoto@chromium.org2012-10-1810-97/+36
| | | | | | | | | | | | | | Replace build targets 'blob' and 'fileapi' with newly introduced 'webkit_storage' Introduce a new directory webkit/storage to host webkit_storage.gypi and webkit_storage_export.h Replace BLOB_EXPORT and FILEAPI_EXPORT with WEBKIT_STORAGE_EXPORT BUG=155242 TEST=build TBR=darin@chromium.org, piman@chromium.org Review URL: https://chromiumcodereview.appspot.com/11103031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162679 0039d316-1c4b-4281-b951-d872f2087c98
* Implement CannedSyncableFileSystem.Write for write testingkinuko@chromium.org2012-10-162-0/+116
| | | | | | | | | | | | | Also factor out MockBlobURLRequestContext class so that we can easily test writing with blob urls. BUG=148897 TEST=LocalFileChangeTrackerTest.* TBR=jam@chromium.org Review URL: https://codereview.chromium.org/11146015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162078 0039d316-1c4b-4281-b951-d872f2087c98
* Add fileapi::FileSystemFileStreamReader::GetLength()hashimoto@chromium.org2012-10-102-8/+10
| | | | | | | | | | BUG=141835 TEST=content_unittests --gtest_filter="FileSystemURLRequestJobTest.*" Review URL: https://chromiumcodereview.appspot.com/11087048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@161114 0039d316-1c4b-4281-b951-d872f2087c98
* webkit_blob: Remove duplicated FileStreamReader creation code from ↵hashimoto@chromium.org2012-10-102-14/+21
| | | | | | | | | | | | BlobURLReqeustJob BUG=141835 TEST=content_unittests --gtest_filter="BlobURLRequestJobTest.*" Review URL: https://chromiumcodereview.appspot.com/11017053 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@161100 0039d316-1c4b-4281-b951-d872f2087c98
* Removed calls to URLRequest::URLRequest in favor of ↵erikwright@chromium.org2012-09-011-3/+2
| | | | | | | | | | | | | | | URLRequestContext::CreateRequest This patch is by departed intern shalev@chromium.org, but it failed to pass the CQ for reasons unrelated to the patch. Original review at http://codereview.chromium.org/10870060/ . TBR=mmenke@chromium.org,michaeln@chromium.org,brettw@chromium.org,ben@chromium.org,zelidrag@chromium.org,phajdan.jr@chromium.org,pfeldman@chromium.org BUG=142945 Review URL: https://chromiumcodereview.appspot.com/10910044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154550 0039d316-1c4b-4281-b951-d872f2087c98
* Support FileSystem URL in Filekinuko@chromium.org2012-08-286-0/+48
| | | | | | | | | | BUG=110119 TEST=existing tests (more will be added when we add more implementation) Review URL: https://chromiumcodereview.appspot.com/10828252 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153651 0039d316-1c4b-4281-b951-d872f2087c98
* Refactoring: ProtocolHandler::MaybeCreateJob and other functions take ↵shalev@chromium.org2012-08-247-6/+15
| | | | | | | | | | | | | | NetworkDelegate as argument This change goes a long way to prepare for removing NetworkDelegate from URLRequestContext. TBR=sky@chromium.org, michaeln@chromium.org, benjhayden@chromium.org, brettw@chromium.org, ben@chromium.org, davemoore@chromium.org, zelidrag@chromium.org, mnissler@chromium.org, thestig@chromium.org, asargent@chromium.org, jhawkins@chromium.org, bulach@chromium.org BUG=crbug.com/142945 Review URL: https://chromiumcodereview.appspot.com/10855209 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153133 0039d316-1c4b-4281-b951-d872f2087c98
* Factor out common Element struct from BlobData and ResourceRequestBodykinuko@chromium.org2012-08-236-164/+70
| | | | | | | | | | BUG=110119 TEST=none Review URL: https://chromiumcodereview.appspot.com/10827414 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152987 0039d316-1c4b-4281-b951-d872f2087c98
* Split net::UploadData into two: for resource request IPC and for upload handlingkinuko@chromium.org2012-08-213-235/+0
| | | | | | | | | | | | | | | | | Introducing webkit_glue::ResourceRequestBody as a content-level abstraction corresponding to WebHTTPBody and as an alternative of net::UploadData in ResourceRequest. This interface can contain content-level objects like Blob (or FileSystem URL in later patches) while net::UploadData should NOT. This patch also removes Blob support in net::UploadData. BUG=110119 TEST=existing tests Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=152528 Review URL: https://chromiumcodereview.appspot.com/10834289 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152553 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 152528 - Split net::UploadData into two: for resource request IPC and ↵kinuko@chromium.org2012-08-213-107/+98
| | | | | | | | | | | | | | | | | | | | for upload handling Introducing webkit_glue::ResourceRequestBody as a content-level abstraction corresponding to WebHTTPBody and as an alternative of net::UploadData in ResourceRequest. This interface can contain content-level objects like Blob (or FileSystem URL in later patches) while net::UploadData should NOT. This patch also removes Blob support in net::UploadData. BUG=110119 TEST=existing tests Review URL: https://chromiumcodereview.appspot.com/10834289 TBR=kinuko@chromium.org Review URL: https://chromiumcodereview.appspot.com/10831402 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152529 0039d316-1c4b-4281-b951-d872f2087c98
* Split net::UploadData into two: for resource request IPC and for upload handlingkinuko@chromium.org2012-08-213-98/+107
| | | | | | | | | | | | | | | Introducing webkit_glue::ResourceRequestBody as a content-level abstraction corresponding to WebHTTPBody and as an alternative of net::UploadData in ResourceRequest. This interface can contain content-level objects like Blob (or FileSystem URL in later patches) while net::UploadData should NOT. This patch also removes Blob support in net::UploadData. BUG=110119 TEST=existing tests Review URL: https://chromiumcodereview.appspot.com/10834289 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152528 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup Blob code dependencykinuko@chromium.org2012-08-212-63/+0
| | | | | | | | | | | Removing BlobData::BlobData(const WebBlobData&), and instead adding new NewBlobData(const WebBlobData&) method only for test_shell. BUG=none Review URL: https://chromiumcodereview.appspot.com/10834386 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152497 0039d316-1c4b-4281-b951-d872f2087c98
* net: Make UploadData::Element an independent classhashimoto@chromium.org2012-08-212-6/+6
| | | | | | | | | | | | | This is a part of the ongoing refactoring of net::UploadData. UploadData::Element is renamed to UploadElement and is moved from upload_data.cc/h to upload_element.cc/h. BUG=None TEST=net_unittests TBR=kinuko@chromium.org,darin@chromium.org Review URL: https://chromiumcodereview.appspot.com/10827425 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152481 0039d316-1c4b-4281-b951-d872f2087c98
* Add thread assertions in ShareableFileReferencekinuko@chromium.org2012-08-101-7/+53
| | | | | | | | | BUG=140508 Review URL: https://chromiumcodereview.appspot.com/10852017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151002 0039d316-1c4b-4281-b951-d872f2087c98
* Add const accessors to URLRequest and UploadData.vabr@chromium.org2012-07-281-1/+2
| | | | | | | | | | | | | | | These const accessors are needed in my other CL 10694055. The accessors are very simple, so I duplicated the code rather than const-casting the const accessor to implement the non-const. BUG=N/A TEST=N/A (no fix & no new feature) TBR=jianli@chromium.org,darin@chromium.org,joaodasilva@chromium.org Review URL: https://chromiumcodereview.appspot.com/10832003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148906 0039d316-1c4b-4281-b951-d872f2087c98
* Change ShareableFileReference to take TaskRunner instead of MessageLoopProxykinuko@chromium.org2012-07-282-13/+15
| | | | | | | | | | BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10825069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148893 0039d316-1c4b-4281-b951-d872f2087c98
* Factor out common error-code conversion code for PlatformFileErrorToNetError.nhiroki@google.com2012-07-252-19/+1
| | | | | | | | | BUG=136494 Review URL: https://chromiumcodereview.appspot.com/10808043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148255 0039d316-1c4b-4281-b951-d872f2087c98
* Offload disk accesses to WorkerPool in ExtensionProtocolHandlerpaivanof@gmail.com2012-07-232-8/+12
| | | | | | | | | | | | | | | | | and URLRequestResourceBundleJob. To achieve that URLRequestSimpleJob::GetData() signature changed to allow async operation finishing. All places implementing GetData() are updated. Patch from Pavel Ivanov <paivanof@gmail.com> BUG=59849,90207 Review URL: https://chromiumcodereview.appspot.com/10696135 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147869 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the rest of #pragma once in one big CL.ajwong@chromium.org2012-07-114-4/+0
| | | | | | | | | For context see this thread: https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II TBR=thakis,pkasting,jam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
* Decouple URLRequestJob from URLRequestContext; access NetworkDelegate as a ↵shalev@chromium.org2012-06-291-1/+2
| | | | | | | | | | | | | | contructor parameter. Note: this depends on http://codereview.chromium.org/10559036/ BUG=None TEST=None Review URL: https://chromiumcodereview.appspot.com/10534100 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144948 0039d316-1c4b-4281-b951-d872f2087c98
* Added URLRequestContext to constructor for URLRequest.erikwright@chromium.org2012-06-221-1/+4
| | | | | | | | | | | | | Previously, the context was set using the member function set_context. Note that the context for a URLRequest is not allowed to be null. BUG=81979 TEST=None TBR=ajwong,brettw,vitalybuka,sky Review URL: https://chromiumcodereview.appspot.com/10559036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143595 0039d316-1c4b-4281-b951-d872f2087c98
* Fix int overflow in blob_url_request_job when we read files > 2GBtbarzic@chromium.org2012-06-081-2/+4
| | | | | | | | | | BUG=131295 TEST=manual; successfully copied 3GB file in Chromeos filebrowser Review URL: https://chromiumcodereview.appspot.com/10536062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141181 0039d316-1c4b-4281-b951-d872f2087c98
* Move remaining webkit storage unit tests to content_unittestsadamk@chromium.org2012-06-081-1/+2
| | | | | | | | | | This includes appcache, blob, database, dom_storage, and quota. Many tests needed MessageLoops, which I've added as appropriate. Review URL: https://chromiumcodereview.appspot.com/10540042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141139 0039d316-1c4b-4281-b951-d872f2087c98
* Rename webkit_blob::FileReader to FileStreamReader.kinaba@chromium.org2012-05-317-81/+87
| | | | | | | | | | | | The name was confusion: it does not directly correspond to FileReader in File API, (which is implemented by WebCore::FileReader). BUG=128483 TEST=Existing file/blob tests. Review URL: https://chromiumcodereview.appspot.com/10447083 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139724 0039d316-1c4b-4281-b951-d872f2087c98
* Move fileapi into its own componentbenwells@chromium.org2012-05-301-7/+0
| | | | | | | | | | | | Before this change fileapi is a static library. As it has static data, this causes problems when used from multiple components like chrome and content. BUG=129885 TEST=Existing tests Review URL: https://chromiumcodereview.appspot.com/10447055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139498 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 138791 - Revert 138554 - Prevent zero-length items from being ↵danakj@chromium.org2012-05-243-26/+38
| | | | | | | | | | | | | | | | | | appended to a blob. BUG=128266 TEST=as in the bug Review URL: https://chromiumcodereview.appspot.com/10386183 TBR=ericu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10444010 TBR=danakj@chromium.org Review URL: https://chromiumcodereview.appspot.com/10442013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138802 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 138795 - Revert "nss: revert encrypted and origin bound certificates ↵agl@chromium.org2012-05-243-38/+26
| | | | | | | | | | | support." Cleaning up git-svn mess with drover. TBR=agl@chromium.org Review URL: https://chromiumcodereview.appspot.com/10451012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138796 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "nss: revert encrypted and origin bound certificates support."agl@chromium.org2012-05-243-26/+38
| | | | | | Screwed up git branches in that change. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138795 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 138554 - Prevent zero-length items from being appended to a blob.danakj@chromium.org2012-05-243-38/+26
| | | | | | | | | | | | | BUG=128266 TEST=as in the bug Review URL: https://chromiumcodereview.appspot.com/10386183 TBR=ericu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10444010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138791 0039d316-1c4b-4281-b951-d872f2087c98
* Prevent zero-length items from being appended to a blob.ericu@chromium.org2012-05-243-31/+45
| | | | | | | | | | | | BUG=128266 TEST=as in the bug Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=138554 Review URL: https://chromiumcodereview.appspot.com/10386183 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138702 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 138554 - Prevent zero-length items from being appended to a blob.ericu@google.com2012-05-233-38/+26
| | | | | | | | | | | | | BUG=128266 TEST=as in the bug Review URL: https://chromiumcodereview.appspot.com/10386183 TBR=ericu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10428003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138584 0039d316-1c4b-4281-b951-d872f2087c98
* Prevent zero-length items from being appended to a blob.ericu@chromium.org2012-05-233-26/+38
| | | | | | | | | | BUG=128266 TEST=as in the bug Review URL: https://chromiumcodereview.appspot.com/10386183 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138554 0039d316-1c4b-4281-b951-d872f2087c98
* Change the way _EXPORT macros look.thakis@chromium.org2012-05-151-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the current setup, if you have a header file my_class.h class BASE_EXPORT MyClass { public: void MyInlineMethod() { /* do stuff, inline */ } }; then every cc file that includes my_class.h will have a public symbol for MyInlineMethod (because inline methods need to be emitted to every translation unit, and the linker sorts them out). With the components build, the linker can't decide to drop these inline methods, so every .so that uses this header file will have the same public symbol. With this proposed change, the symbol will only be visible in the target the header file belongs to, and it will be hidden in all other components. That's cleaner, and it also prevents accident hidden dependencies (say target A depends on B, and B depends on C. A accidentally uses an inline function from a class in C. With this change, that would result in a linker error, and an explicit dependency from A on C would have to be added). Also add a missing CHROMEOS_IMPLEMENTATION define which went unnoticed until now. BUG=90078 TEST=Things still build. TBR=ben, tony, viettrungluu, thestig, agl, willchan Review URL: https://chromiumcodereview.appspot.com/10386108 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137130 0039d316-1c4b-4281-b951-d872f2087c98
* Change webkit/{fileapi,quota} code to use TaskRunner.kinuko@chromium.org2012-05-072-8/+7
| | | | | | | | | BUG=123559 TEST=existing tests Review URL: https://chromiumcodereview.appspot.com/10197007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135616 0039d316-1c4b-4281-b951-d872f2087c98
* LocalFileReaderTest leak fix.kinuko@chromium.org2012-05-061-7/+10
| | | | | | | | | BUG=123302 TEST=LocalFileReaderTest.* Review URL: https://chromiumcodereview.appspot.com/10260002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135594 0039d316-1c4b-4281-b951-d872f2087c98
* RefCounted types should not have public destructors, webkit/rsleevi@chromium.org2012-04-282-5/+7
| | | | | | | | | | BUG=123295 TEST=none Review URL: http://codereview.chromium.org/10066044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134449 0039d316-1c4b-4281-b951-d872f2087c98
* Relanding r134248 with leak fix in tests - Make LocalFileReader deletable at ↵kinuko@chromium.org2012-04-274-137/+369
| | | | | | | | | | | | | | | | any time and add unittests. Added one-line change (to force running pending cleanup tasks) for leak fix in the test. Original review URL: https://chromiumcodereview.appspot.com/10234006 BUG=123570 TEST=unit_tests:LocalFileReaderTest.* TBR=michaeln Review URL: https://chromiumcodereview.appspot.com/10170031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134276 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 134248 - Make LocalFileReader deletable at any time and add unittests.kinuko@chromium.org2012-04-274-364/+137
| | | | | | | | | | | | | | | | Changed the internal Open() logic not to deal with raw PlatformFile handle but instead to rely on net::FileStream's destructor to close the file. Also adding more specific comment to FileReader interface. BUG=123570 TEST=unit_tests:LocalFileReader.* Review URL: https://chromiumcodereview.appspot.com/10234006 TBR=kinuko@chromium.org Review URL: https://chromiumcodereview.appspot.com/10209025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134258 0039d316-1c4b-4281-b951-d872f2087c98
* Make LocalFileReader deletable at any time and add unittests.kinuko@chromium.org2012-04-274-137/+364
| | | | | | | | | | | | | Changed the internal Open() logic not to deal with raw PlatformFile handle but instead to rely on net::FileStream's destructor to close the file. Also adding more specific comment to FileReader interface. BUG=123570 TEST=unit_tests:LocalFileReader.* Review URL: https://chromiumcodereview.appspot.com/10234006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134248 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: IWYU for base/file_util_proxy.h.thestig@chromium.org2012-04-231-1/+2
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/10115017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133520 0039d316-1c4b-4281-b951-d872f2087c98
* Add FileReader interface and implement FileSystemFileReader.kinuko@chromium.org2012-04-184-35/+75
| | | | | | | | | | | | | | - Split LocalFileReader into FileReader interface and LocalFileReader implementation - Adding FileSystemFileReader as a default implementation for FileSystem files The FileSystemFileReader implementation uses CreateSnapshotFile which may force downloading the entire file (as the initial generic version). This would need to be changed in remote filesystem implementation. BUG=114999 TEST=FileSystemURLRequestJobTest, BlobURLRequestJobTest Review URL: https://chromiumcodereview.appspot.com/10038019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132774 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 132134 with memory fix - Use LocalFileReader in FileSystemURLRequestJobkinuko@chromium.org2012-04-173-11/+15
| | | | | | | | | | | | LocalFileReader change https://chromiumcodereview.appspot.com/10065011 with memory fix http://codereview.chromium.org/10067031/ BUG=113300, 114999, 123302 TEST=FileSystemURLRequestJobTest.* Review URL: https://chromiumcodereview.appspot.com/10080001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132559 0039d316-1c4b-4281-b951-d872f2087c98
* Accomodate files larger than maxint in size in blob/file reading.michaeln@google.com2012-04-134-5/+5
| | | | | | | BUG=114179 Review URL: https://chromiumcodereview.appspot.com/10067026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132244 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 132134 - Use LocalFileReader in FileSystemURLRequestJobglider@chromium.org2012-04-133-4/+6
| | | | | | | | | | | | | | | | | | Reason: multiple leak reports on the Memory FYI waterfall (Valgrind, Heapcheck). - to avoid file access on IO thread - to remove duplicated code - for further FileSystem stream related refactoring BUG=113300,114999,123302 TEST=FileSystemURLRequestJobTest.* Review URL: https://chromiumcodereview.appspot.com/10065011 TBR=kinuko@chromium.org Review URL: https://chromiumcodereview.appspot.com/10066043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132162 0039d316-1c4b-4281-b951-d872f2087c98
* Use LocalFileReader in FileSystemURLRequestJobkinuko@chromium.org2012-04-133-6/+4
| | | | | | | | | | | | | - to avoid file access on IO thread - to remove duplicated code - for further FileSystem stream related refactoring BUG=113300,114999 TEST=FileSystemURLRequestJobTest.* Review URL: https://chromiumcodereview.appspot.com/10065011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132134 0039d316-1c4b-4281-b951-d872f2087c98
* 2nd try: separate FileStream related code out of BlobURLRequestkinuko@chromium.org2012-04-116-123/+381
| | | | | | | | | | | | | | | | The original change I had tried to submit was: http://codereview.chromium.org/9651032 The change was reverted because FileStream cannot be created on other threads than on IO thread on Windows, and therefore broke the ExtensionPageCaptureApiTest.SaveAsMHTML browser test. BUG=114999 TEST=test_shell_tests:BlobURLRequestJobTest.*, browser_tests:ExtensionPageCaptureApiTest.SaveAsMHTML Review URL: http://codereview.chromium.org/10029008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131791 0039d316-1c4b-4281-b951-d872f2087c98
* Make FileStream::Seek async and add FileStream::SeekSync for sync operationkinuko@chromium.org2012-04-111-2/+2
| | | | | | | | | BUG=75548,113300 TEST=existing tests should pass Review URL: https://chromiumcodereview.appspot.com/9949011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131732 0039d316-1c4b-4281-b951-d872f2087c98
* Fix blob storage internals CSP typo.craig.schlenter@chromium.org2012-04-051-2/+2
| | | | | | | | | BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10009017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131004 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 126623 - Separate FileStream related code out of BlobURLRequestJobmarja@chromium.org2012-03-146-337/+122
| | | | | | | | | | | | | | | | | | | | | | | | | ExtensionPageCaptureApiTest.SaveAsMHTML times out consistently after this change. It also fails in the try job / commit queue runs of this CL. ------- This patch separates FileStream related code out of BlobURLRequestJob to: - make BlobURLRequesetJob's responsibility simpler and clearer by moving file-related details out of the class - remove blocking file operation on IO thread Long-term goal includes using the same reader class/interface to FileSystemURLRequestJob and other fileapi streaming related code. (Simplified the patch from the previous version: was: "Add FileStreamInterface for fileapi/blob stream related operations") BUG=114999,75548 TEST=blob tests Review URL: https://chromiumcodereview.appspot.com/9651032 TBR=kinuko@chromium.org Review URL: https://chromiumcodereview.appspot.com/9704010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126631 0039d316-1c4b-4281-b951-d872f2087c98
* Separate FileStream related code out of BlobURLRequestJobkinuko@chromium.org2012-03-146-122/+337
| | | | | | | | | | | | | | | | | This patch separates FileStream related code out of BlobURLRequestJob to: - make BlobURLRequesetJob's responsibility simpler and clearer by moving file-related details out of the class - remove blocking file operation on IO thread Long-term goal includes using the same reader class/interface to FileSystemURLRequestJob and other fileapi streaming related code. (Simplified the patch from the previous version: was: "Add FileStreamInterface for fileapi/blob stream related operations") BUG=114999,75548 TEST=blob tests Review URL: https://chromiumcodereview.appspot.com/9651032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126623 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-13 20:37:01 +0000