Add 'Cache-Control: no-cache' to filesystem: responses to avoid

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

1 files changed, 3 insertions, 3 deletions

diff --git a/webkit/fileapi/file_system_url_request_job.h b/webkit/fileapi/file_system_url_request_job.h
index ffef027..6d0f6e3 100644
--- a/webkit/fileapi/file_system_url_request_job.h
+++ b/webkit/fileapi/file_system_url_request_job.h
@@ -40,13 +40,12 @@ class FileSystemURLRequestJob : public net::URLRequestJob {
   virtual bool ReadRawData(net::IOBuffer* buf, int buf_size, int* bytes_read);
   virtual bool IsRedirectResponse(GURL* location, int* http_status_code);
   virtual void SetExtraRequestHeaders(const net::HttpRequestHeaders& headers);
+  virtual void GetResponseInfo(net::HttpResponseInfo* info);
+  virtual int GetResponseCode() const;
 
   // FilterContext methods (via URLRequestJob):
   virtual bool GetMimeType(std::string* mime_type) const;
 
-  // TODO(adamk): Implement GetResponseInfo and GetResponseCode to simulate
-  // an HTTP response.
-
  private:
   virtual ~FileSystemURLRequestJob();
 
@@ -68,6 +67,7 @@ class FileSystemURLRequestJob : public net::URLRequestJob {
   net::CompletionCallbackImpl<FileSystemURLRequestJob> io_callback_;
   scoped_ptr<net::FileStream> stream_;
   bool is_directory_;
+  scoped_ptr<net::HttpResponseInfo> response_info_;
 
   net::HttpByteRange byte_range_;
   int64 remaining_bytes_;