From 2e9c1482cafb068e4ffcd4cbf6e090013ccbfcad Mon Sep 17 00:00:00 2001 From: "adamk@chromium.org" Date: Wed, 16 Mar 2011 01:55:58 +0000 Subject: Remove TODO in AppCacheURLRequestJob about GetContentEncodings(). I plan to make some changes to the interactions between URLRequestJobs and Filters, and so as a convenience I'd like this file to not show up in code search/git grep. It seems safe to assume that the URLRequestJob is already correctly implemented, as this comment has been here for well over a year with no change. R=michaeln@chromium.org BUG=none TEST=none Review URL: http://codereview.chromium.org/6698050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78322 0039d316-1c4b-4281-b951-d872f2087c98 --- webkit/appcache/appcache_url_request_job.h | 7 ------- 1 file changed, 7 deletions(-) (limited to 'webkit/appcache') diff --git a/webkit/appcache/appcache_url_request_job.h b/webkit/appcache/appcache_url_request_job.h index f66ed54..1ec9f95 100644 --- a/webkit/appcache/appcache_url_request_job.h +++ b/webkit/appcache/appcache_url_request_job.h @@ -115,13 +115,6 @@ class AppCacheURLRequestJob : public net::URLRequestJob, // This is how we get informed of range-requests. virtual void SetExtraRequestHeaders(const net::HttpRequestHeaders& headers); - // TODO(michaeln): does this apply to our cached responses? - // The payload we store should have been fully decoded prior to - // us storing it. Maybe we should strip the content-encoding headers - // when writing the response to the cache. - // virtual bool GetContentEncodings( - // std::vector* encoding_types); - // FilterContext methods virtual bool GetMimeType(std::string* mime_type) const; virtual int GetResponseCode() const; -- cgit v1.1