IPC messages and changes to ResourceLoaderBridge to support resource loading for media

net/disk_cache/disk_cache.h

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Defines the public interface of the disk cache. For more details see
 // http://dev.chromium.org/developers/design-documents/disk-cache
 
 #ifndef NET_DISK_CACHE_DISK_CACHE_H_
 #define NET_DISK_CACHE_DISK_CACHE_H_
 
@@ skipping 142 matching lines @@
   virtual int WriteData(int index, int offset, net::IOBuffer* buf, int buf_len,
                         net::CompletionCallback* completion_callback,
                         bool truncate) = 0;
 
   // Prepares a target stream as an external file, returns a corresponding
   // base::PlatformFile if successful, returns base::kInvalidPlatformFileValue
   // if fails. If this call returns a valid base::PlatformFile value (i.e.
   // not base::kInvalidPlatformFileValue), there is no guarantee that the file
   // is truncated. Implementor can always return base::kInvalidPlatformFileValue
   // if external file is not available in that particular implementation.
-  // Caller should never close the file handle returned by this method, since
-  // the handle should be managed by the implementor of this class. Caller
-  // should never save the handle for future use.
+  // The caller should close the file handle returned by this method or there
+  // will be a leak.
   // With a stream prepared as an external file, the stream would always be
   // kept in an external file since creation, even if the stream has 0 bytes.
   // So we need to be cautious about using this option for preparing a stream or
   // we will end up having a lot of empty cache files. Calling this method also
   // means that all data written to the stream will always be written to file
   // directly *without* buffering.
   virtual base::PlatformFile UseExternalFile(int index) = 0;
 
-  // Returns a read file handle for the cache stream referenced by |index|.
-  // Caller should never close the handle returned by this method and should
-  // not save it for future use. The lifetime of the base::PlatformFile handle
-  // is managed by the implementor of this class.
+  // Returns an asynchronous read file handle for the cache stream referenced by
+  // |index|. Values other than base::kInvalidPlatformFileValue are successful
+  // and the file handle should be managed by the caller, i.e. caller should
+  // close the handle after use or there will be a leak.
   virtual base::PlatformFile GetPlatformFile(int index) = 0;
 
  protected:
   virtual ~Entry() {}
 };
 
 }  // namespace disk_cache
 
 #endif  // NET_DISK_CACHE_DISK_CACHE_H_