[Webkit-unassigned] [Bug 47310] Implement FileEntrySync.file() in FileSystem API

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=47310


Jian Li <jianli at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #70787|review?                     |review-
               Flag|                            |




--- Comment #12 from Jian Li <jianli at chromium.org>  2010-10-15 13:49:42 PST ---
(From update of attachment 70787)
View in context: https://bugs.webkit.org/attachment.cgi?id=70787&action=review

> LayoutTests/fast/filesystem/resources/file-from-file-entry-sync.js:1
> +if (this.importScripts) {

Is this resource file only used in worker testing? If so, it would be better to put under filesystem/workers/resources. In addition, you can remove the check for the existence of importScripts.

> LayoutTests/fast/filesystem/resources/file-from-file-entry.js:2
> +    importScripts('../resources/fs-worker-common.js');

Do we need to use "../resources"?

> LayoutTests/fast/filesystem/resources/file-from-file-entry.js:3
> +    importScripts('../resources/fs-test-util.js');

ditto.

> LayoutTests/fast/filesystem/workers/file-from-file-entry-sync-expected.txt:1
> +

Why do we have blank expected result here?

> WebCore/fileapi/DOMFileSystem.cpp:89
> +void DOMFileSystem::createFile(const FileEntry* file, PassRefPtr<FileCallback> successCallback, PassRefPtr<ErrorCallback>)

Please consider renaming 'file' argument to 'fileEntry' to avoid the confusion with the File being created in this method.

> WebCore/fileapi/DOMFileSystem.h:63
> +    void createFile(const FileEntry* file, PassRefPtr<FileCallback>, PassRefPtr<ErrorCallback>);

Please omit 'file' argument.

> WebCore/fileapi/DOMFileSystemSync.cpp:62
> +PassRefPtr<File> DOMFileSystemSync::createFile(const FileEntrySync* file, ExceptionCode& ec)

ditto.

> WebCore/fileapi/DOMFileSystemSync.h:59
> +    PassRefPtr<File> createFile(const FileEntrySync* file, ExceptionCode&);

ditto.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.