summaryrefslogtreecommitdiffstats
path: root/webkit/appcache/appcache_update_job.cc
Commit message (Collapse)AuthorAgeFilesLines
* Appcache cross-origin HTTPS resources.michaeln@google.com2011-02-221-2/+19
| | | | | | | | | BUG=69594 TEST=updated unit tests Review URL: http://codereview.chromium.org/6526037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75622 0039d316-1c4b-4281-b951-d872f2087c98
* AppCache URLFetcher refactoring. Just pulling the URLRequest handlingmichaeln@google.com2011-02-151-321/+247
| | | | | | | | | | | | | logic out of class AppCacheUpdateJob and into a seperate URLFetcher class. There should be no functional changes here, just refactoring for the sake of maintainability to (hopefully) make future work easier. BUG=none TEST=existing tests apply Review URL: http://codereview.chromium.org/6486010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74998 0039d316-1c4b-4281-b951-d872f2087c98
* net: Remove typedef net::URLRequest URLRequest;tfarina@chromium.org2010-11-301-22/+25
| | | | | | | | | 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 the error message more clear in the appcache console logging when an ↵michaeln@chromium.org2010-11-191-3/+5
| | | | | | | | | | | initial cache attempt fails due to a missing manifest file. BUG=none TEST=manual Review URL: http://codereview.chromium.org/5178003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66729 0039d316-1c4b-4281-b951-d872f2087c98
* Convert implicit scoped_refptr constructor calls to explicit ones, part 1thakis@chromium.org2010-11-011-6/+6
| | | | | | | | | | | 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
* Convert LOG(INFO) to VLOG(1) - webkit/.pkasting@chromium.org2010-10-191-7/+7
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3822006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63001 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Move ctors/dtors into implementation files. Adds ctors/dtors to ↵erg@google.com2010-09-301-0/+11
| | | | | | | | | | | | | | non-POD structs. Cuts ~2MB off our .a files (Debug, Linux). Also added the "virtual" keyword on a whole bunch of virtual dtors that were missing it. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3522004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61100 0039d316-1c4b-4281-b951-d872f2087c98
* webkit: Append base:: in the StringPrintf calls.tfarina@chromium.org2010-09-291-10/+11
| | | | | | | | | | | (Note: This is a TODO in string_util.h) BUG=None TEST=None Review URL: http://codereview.chromium.org/3404027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60885 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Start cleaning up net/erg@google.com2010-08-111-0/+1
| | | | | | | | | | | | url_request.h and http_response_info.h are some of the most included headers from net/ in chrome. Do what forward declaration we can in those headers. BUG=none TEST=none Review URL: http://codereview.chromium.org/3150003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55780 0039d316-1c4b-4281-b951-d872f2087c98
* Undo in memory changes made by an update job if the commit to disk fails. ↵michaeln@chromium.org2010-08-031-5/+21
| | | | | | | | | | Provide a more clear error message when the commit fails due to exceeding the quota. TEST=MasterEntryFailStoreNewestCacheTest BUG=50985 Review URL: http://codereview.chromium.org/2836075 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54667 0039d316-1c4b-4281-b951-d872f2087c98
* A better error message for the mime type of a manifest file being off.michaeln@chromium.org2010-07-281-4/+16
| | | | | | | | | BUG=none TEST=manual Review URL: http://codereview.chromium.org/3076005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53890 0039d316-1c4b-4281-b951-d872f2087c98
* Make conditional requests when updating the appcache.michaeln@chromium.org2010-07-211-78/+99
| | | | | | | | | TEST=altered existing tests to also cover this case BUG=49548 Review URL: http://codereview.chromium.org/3033003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53230 0039d316-1c4b-4281-b951-d872f2087c98
* New javascript console logging related to the appcache.michaeln@chromium.org2010-07-141-73/+77
| | | | | | | | | | | | | * Log an error message when an update job results in an error. * Log info messages for progress events and other update status changed events. * Consistently use "Application Cache" in the console messages. TEST=existing unit tests apply for functional correctness, manual testing to see the new console output BUG=none Review URL: http://codereview.chromium.org/2910007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52270 0039d316-1c4b-4281-b951-d872f2087c98
* Provide more info to the renderer process in appcache progress events.michaeln@chromium.org2010-05-251-12/+37
| | | | | | | | | BUG=39370 TEST= yes, updated unittests Review URL: http://codereview.chromium.org/2166006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48197 0039d316-1c4b-4281-b951-d872f2087c98
* Pass HttpRequestHeaders to URLRequest::SetExtraRequestHeaders().willchan@chromium.org2010-05-071-8/+7
| | | | | | | | BUG=22588 Review URL: http://codereview.chromium.org/1998008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46752 0039d316-1c4b-4281-b951-d872f2087c98
* Indicate in the tab UI if appcache creation was blocked by privacy settings.jochen@chromium.org2010-04-091-1/+3
| | | | | | | | | TEST=manual BUG=38362 Review URL: http://codereview.chromium.org/1600002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44079 0039d316-1c4b-4281-b951-d872f2087c98
* Keep track of appcache creation and last access times.michaeln@chromium.org2010-02-231-1/+1
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/630009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39765 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce an AppCachePolicy interface that allows the containing browser to ↵michaeln@chromium.org2010-02-171-2/+49
| | | | | | | | | | | determine appcache permissions. The policy can allow or deny loading existing manifests or the creation of new manifests. The policy check for creating new manifests can be async to allow for a user prompt. BUG=none TEST=new unit tests added Review URL: http://codereview.chromium.org/565042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39280 0039d316-1c4b-4281-b951-d872f2087c98
* Couple of small AppCache update job modsmichaeln@chromium.org2010-02-121-3/+9
| | | | | | | | | | | | | 1) Fix a bug with failing to deliver the ERROR event in some cases. 2) Use a larger IO buffer to improve download performance. BUG=none TEST=manual Review URL: http://codereview.chromium.org/596078 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38913 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Reduce header dependencies in webkitphajdan.jr@chromium.org2010-02-091-0/+1
| | | | | | | | | TEST=none BUG=none Review URL: http://codereview.chromium.org/582015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38461 0039d316-1c4b-4281-b951-d872f2087c98
* Fix for a big appcache storage bug. Now stores a record of the new master ↵michaeln@chromium.org2010-01-241-28/+41
| | | | | | | | | | | entry in the database even when there is no manifest update. BUG=none TEST=manual and unittest Review URL: http://codereview.chromium.org/549127 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36973 0039d316-1c4b-4281-b951-d872f2087c98
* Enforce a hard coded 5MB per origin quota for appcache storage.michaeln@chromium.org2010-01-211-5/+0
| | | | | | | | | TEST=yes BUG=none Review URL: http://codereview.chromium.org/554008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36723 0039d316-1c4b-4281-b951-d872f2087c98
* Trigger the deletion of unused responses when caches are updated and made ↵michaeln@chromium.org2010-01-131-6/+10
| | | | | | | | | | | obsolete (deleted). BUG=none TEST=not yet Review URL: http://codereview.chromium.org/542021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36166 0039d316-1c4b-4281-b951-d872f2087c98
* AppCache quota tracking groundwork, store response sizes in the SQL database. michaeln@chromium.org2010-01-061-5/+20
| | | | | | | | | TEST=updated existing unittests BUG=none Review URL: http://codereview.chromium.org/525060 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35654 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 35609 - AppCache quota tracking groundwork, store response sizes in ↵michaeln@google.com2010-01-061-20/+5
| | | | | | | | | | | | | | the SQL database. TEST=updated existing unittests BUG=none Review URL: http://codereview.chromium.org/523046 TBR=michaeln@chromium.org Review URL: http://codereview.chromium.org/525059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35610 0039d316-1c4b-4281-b951-d872f2087c98
* AppCache quota tracking groundwork, store response sizes in the SQL database.michaeln@chromium.org2010-01-061-5/+20
| | | | | | | | | TEST=updated existing unittests BUG=none Review URL: http://codereview.chromium.org/523046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35609 0039d316-1c4b-4281-b951-d872f2087c98
* When an update job doesn't complete successfully, doom all responses created ↵michaeln@chromium.org2010-01-041-3/+4
| | | | | | | | | | | by that job. TEST=existing tests apply BUG=none Review URL: http://codereview.chromium.org/523039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35489 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes to address the failing appcache/remove-cache.html layout tests.michaeln@chromium.org2010-01-041-0/+1
| | | | | | | | | BUG=31300 TEST=yes, layout test Review URL: http://codereview.chromium.org/524014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35465 0039d316-1c4b-4281-b951-d872f2087c98
* Make appcache update conditionally include If-Modified-Since and ↵jennb@chromium.org2009-12-221-5/+60
| | | | | | | | | | | If-None-Match headers when fetching a manifest. TEST=new tests added BUG=none Review URL: http://codereview.chromium.org/500152 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35153 0039d316-1c4b-4281-b951-d872f2087c98
* During appcache update, copy response from newest complete cache if http ↵jennb@chromium.org2009-12-161-27/+50
| | | | | | | | | | | | caching semantics allow rather than make a network request. TEST=new tests added BUG=none Review URL: http://codereview.chromium.org/491064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34727 0039d316-1c4b-4281-b951-d872f2087c98
* Queue appcache update if current update process is terminating.jennb@chromium.org2009-12-101-1/+1
| | | | | | | | | TEST=update and group unittests added BUG=none Review URL: http://codereview.chromium.org/465132 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34299 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 449036, which rolled back 402098, and fix the valgrind error that ↵jennb@chromium.org2009-12-011-50/+327
| | | | | | | | | | | | | | | | | | | | | | | | | caused the original rollback: Revert 33394 (due to valgrind errors) - Appcache update support for pending master entries: Update process issues a URL request to fetch pending master entries. Pending master entry fetch logic kept separate from regular url fetching as this will be the case in the longterm solution. No optimizations to avoid issuing URL request if pending master entry is also listed in the manifest. (simpler) Only optimized to prevent refetching something that has already been successfully fetched. Longterm optimized solution should be to siphon the responses as the master resource is downloaded instead of having the update job issue URL requests. TEST=new tests for update jobs with pending master entries BUG=none Review URL: http://codereview.chromium.org/449039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33410 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 33394 (due to valgrind errors) - Appcache update support for pending ↵willchan@chromium.org2009-12-011-327/+50
| | | | | | | | | | | | | | | | | | | | | | master entries: Update process issues a URL request to fetch pending master entries. Pending master entry fetch logic kept separate from regular url fetching as this will be the case in the longterm solution. No optimizations to avoid issuing URL request if pending master entry is also listed in the manifest. (simpler) Only optimized to prevent refetching something that has already been successfully fetched. Longterm optimized solution should be to siphon the responses as the master resource is downloaded instead of having the update job issue URL requests. TEST=new tests for update jobs with pending master entries BUG=none Review URL: http://codereview.chromium.org/402098 TBR=jennb@chromium.org Review URL: http://codereview.chromium.org/449036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33405 0039d316-1c4b-4281-b951-d872f2087c98
* Appcache update support for pending master entries: jennb@chromium.org2009-12-011-50/+327
| | | | | | | | | | | | | | | | | - Update process issues a URL request to fetch pending master entries. - Pending master entry fetch logic kept separate from regular url fetching as this will be the case in the long-term solution. - No optimizations to avoid issuing URL request if pending master entry is also listed in the manifest. (simpler) - Only optimized to prevent refetching something that has already been successfully fetched. Long-term optimized solution should be to siphon the responses as the master resource is downloaded instead of having the update job issue URL requests. TEST=new tests for update jobs with pending master entries BUG=none Review URL: http://codereview.chromium.org/402098 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33394 0039d316-1c4b-4281-b951-d872f2087c98
* Isolated bug fixes from CL 385104 to address the following:jennb@chromium.org2009-11-181-27/+28
| | | | | | | | | | | | | | - using references for collections - only getting request response code if request was successful - writing response info for URL fetches that do not contain any data - only bothering with reading/writing response if response code is 2xx TEST=added test and data files to verify empty file still gets response written to storage BUG=none Review URL: http://codereview.chromium.org/385141 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32337 0039d316-1c4b-4281-b951-d872f2087c98
* Load manifest data for newest cache from storage to compare against fetched ↵jennb@chromium.org2009-11-131-13/+23
| | | | | | | | | | | manifest. TEST=updated BUG=none Review URL: http://codereview.chromium.org/386013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31918 0039d316-1c4b-4281-b951-d872f2087c98
* Change store group and newest cache API so that storage failure does not ↵jennb@chromium.org2009-11-111-9/+3
| | | | | | | | require storage clients to revert changes to group and so it's not racey.TEST=existing tests updatedBUG=none Review URL: http://codereview.chromium.org/384029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31730 0039d316-1c4b-4281-b951-d872f2087c98
* Add storage code to appcache update process.jennb@chromium.org2009-10-301-63/+200
| | | | | | | | | | | Add storage API for simulating storage errors. TEST=verify appcache update wrote to storage correctly BUG=none Review URL: http://codereview.chromium.org/326002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30612 0039d316-1c4b-4281-b951-d872f2087c98
* Fix minor style violation.jennb@chromium.org2009-10-161-2/+2
| | | | | | | | | TBR=mark TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29348 0039d316-1c4b-4281-b951-d872f2087c98
* Fix to avoid a bad mem access after URL request has already been deleted.jennb@chromium.org2009-10-161-1/+2
| | | | | | | | | | | Re-enabled tests that may trigger this condition. TEST=no new tests BUG=none Review URL: http://codereview.chromium.org/285006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29340 0039d316-1c4b-4281-b951-d872f2087c98
* Implement cancelling an appcache update. An update is cancelled when its ↵jennb@chromium.org2009-10-161-46/+66
| | | | | | | | | | | | application cache group is no longer in use. Refcounting of caches and groups changed to make cancelling an update work. TEST=updated existing tests, verify deleting group cancels update, verify new refcounting behavior BUG=none Review URL: http://codereview.chromium.org/274013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29291 0039d316-1c4b-4281-b951-d872f2087c98
* AppCache StorageAPIsmichaeln@google.com2009-10-091-1/+2
| | | | | | | | | TEST=none yet, these are just API definitions stubbed out for now BUG=none Review URL: http://codereview.chromium.org/209071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28513 0039d316-1c4b-4281-b951-d872f2087c98
* Make appcache update job retry 503 requests with retry-after of 0.jennb@chromium.org2009-10-081-9/+58
| | | | | | | | | TEST=test retry logic BUG=none Review URL: http://codereview.chromium.org/264002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28406 0039d316-1c4b-4281-b951-d872f2087c98
* Re-enabling disabled appcache update unittests. Change comparison of cache ↵jennb@chromium.org2009-10-061-2/+2
| | | | | | | | | | | update time. Change manifest-fb-404 entry ordering so all 3 requests always happen regardless of which order the url requests complete in. TEST=none BUG=none Review URL: http://codereview.chromium.org/257065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28174 0039d316-1c4b-4281-b951-d872f2087c98
* Disable some appcache update tests that depend on testserver.jennb@chromium.org2009-10-061-3/+3
| | | | | | | | | | Probably failing due to network timeout. Upped log message to ERROR to get more info. TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28126 0039d316-1c4b-4281-b951-d872f2087c98
* Implementation of application cache update algorithm.jennb@chromium.org2009-10-061-0/+671
Does not include storage nor processing of pending master entries. TEST=verify update functionality BUG=none Review URL: http://codereview.chromium.org/201070 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28123 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-01-21 21:58:34 +0000