[chromium] FileSystem mods: Changes to snapshot file creation to reduce dependencies...
authormichaeln@google.com <michaeln@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 3 Feb 2013 06:12:51 +0000 (06:12 +0000)
committermichaeln@google.com <michaeln@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 3 Feb 2013 06:12:51 +0000 (06:12 +0000)
This patch just alters the WebKitAPI in advance of coding to the new API in chromium and webkit
sources, defining two new virtual methods (unimplemented and uncalled). The existing API remains
in place and in use.
https://bugs.webkit.org/show_bug.cgi?id=108736

Reviewed by Darin Fisher.

Source/Platform:

* chromium/public/WebFileSystem.h:
(WebFileSystem):
(WebKit::WebFileSystem::createSnapshotFileAndReadMetadata):

Source/WebKit/chromium:

* public/WebFileSystemCallbacks.h:
(WebFileSystemCallbacks):
(WebKit::WebFileSystemCallbacks::didCreateSnapshotFile):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@141712 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/Platform/ChangeLog
Source/Platform/chromium/public/WebFileSystem.h
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/public/WebFileSystemCallbacks.h

index 7ecd9ac..48f8814 100644 (file)
@@ -1,3 +1,17 @@
+2013-02-02  Michael Nordman  <michaeln@google.com>
+
+        [chromium] FileSystem mods: Changes to snapshot file creation to reduce dependencies on blobs.
+        This patch just alters the WebKitAPI in advance of coding to the new API in chromium and webkit
+        sources, defining two new virtual methods (unimplemented and uncalled). The existing API remains
+        in place and in use.
+        https://bugs.webkit.org/show_bug.cgi?id=108736
+
+        Reviewed by Darin Fisher.
+
+        * chromium/public/WebFileSystem.h:
+        (WebFileSystem):
+        (WebKit::WebFileSystem::createSnapshotFileAndReadMetadata):
+
 2013-02-01  James Simonsen  <simonjam@chromium.org>
 
         Add didChangePriority() to ResourceHandle
index 25afb49..ec1ed1d 100644 (file)
@@ -120,13 +120,17 @@ public:
     // This is a fast, synchronous call, and should not stat the filesystem.
     virtual WebFileWriter* createFileWriter(const WebURL& path, WebFileWriterClient*) { WEBKIT_ASSERT_NOT_REACHED(); return 0; }
 
-    // Creates a snapshot file for a given file specified by |path| and registers the file with the |blobURL|. It returns the metadata of the created snapshot file.
+    // Creates a snapshot file for a given file specified by |path|. It returns the metadata of the created snapshot file.
     // The returned metadata should include a local platform path to the snapshot image.
-    // In local filesystem cases the backend may simply return the metadata of the file itself (as well as readMetadata does), while in remote filesystem case the backend may download the file into a temporary snapshot file and return the metadata of the temporary file.
+    // In local filesystem cases the backend may simply return the metadata of the file itself (as well as readMetadata does), while in
+    // remote filesystem case the backend may download the file into a temporary snapshot file and return the metadata of the temporary file.
     // The returned metadata is used to create a File object for the |path|.
-    // The snapshot file is supposed to be deleted when the last reference to the |blobURL| is dropped.
-    // WebFileSystemCallbacks::didReadMetadata() with the metadata of the snapshot file must be called when the operation is completed successfully.
+    // The snapshot file is supposed to be deleted when the last reference to a WebCore::File referring to it's path is dropped.
+    // WebFileSystemCallbacks::didCreateSnapshotFile() with the metadata of the snapshot file must be called when the operation is completed successfully.
     // WebFileSystemCallbacks::didFail() must be called otherwise.
+    virtual void createSnapshotFileAndReadMetadata(const WebURL& path, WebFileSystemCallbacks*) { WEBKIT_ASSERT_NOT_REACHED(); }
+
+    // DEPRECATED
     virtual void createSnapshotFileAndReadMetadata(const WebURL& blobURL, const WebURL& path, WebFileSystemCallbacks*) { WEBKIT_ASSERT_NOT_REACHED(); }
 
 protected:
index 30dc037..4f85e29 100644 (file)
@@ -1,3 +1,17 @@
+2013-02-02  Michael Nordman  <michaeln@google.com>
+
+        [chromium] FileSystem mods: Changes to snapshot file creation to reduce dependencies on blobs.
+        This patch just alters the WebKitAPI in advance of coding to the new API in chromium and webkit
+        sources, defining two new virtual methods (unimplemented and uncalled). The existing API remains
+        in place and in use.
+        https://bugs.webkit.org/show_bug.cgi?id=108736
+
+        Reviewed by Darin Fisher.
+
+        * public/WebFileSystemCallbacks.h:
+        (WebFileSystemCallbacks):
+        (WebKit::WebFileSystemCallbacks::didCreateSnapshotFile):
+
 2013-02-01  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed.  Rolled Chromium DEPS to r180221.  Requested by
index 9966b0f..b83958a 100644 (file)
@@ -51,6 +51,10 @@ public:
     // for the requested path.
     virtual void didReadMetadata(const WebFileInfo&) = 0;
 
+    // Callback for WebFileSystem::createSnapshot. The metadata also includes the
+    // platform file path.
+    virtual void didCreateSnapshotFile(const WebFileInfo&) { WEBKIT_ASSERT_NOT_REACHED(); }
+
     // Callback for WebFileSystem::readDirectory.  Called with a vector of
     // file entries in the requested directory. This callback might be called
     // multiple times if the directory has many entries. |hasMore| must be