Bug 176165 – Implement FileSystemEntry.getParent()

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Chris Dumez]

Implement FileSystemEntry.getParent():
- https://wicg.github.io/entries-api/#dom-filesystementry-getparent

[Radar WebKit Bug Importer]

<rdar://problem/34187743>

[Chris Dumez]

Created attachment 319564 [details]
Patch

[Andreas Kling]

Comment on attachment 319564 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=319564&action=review

> Source/WebCore/Modules/entriesapi/DOMFileSystem.cpp:92
> +    return !virtualPath.isEmpty() && virtualPath[0] == '/';

The isEmpty() check here is redundant but I hate that operator[] API returns a null character for invalid indices so let's not even care about that.

> Source/WebCore/Modules/entriesapi/FileSystemEntry.cpp:67
> +void FileSystemEntry::getParent(ScriptExecutionContext& context, RefPtr<FileSystemEntryCallback>&& successCallback, RefPtr<ErrorCallback>&& errorCallback)

If we have neither successCallback or errorCallback, we could just early return here, no need to do work. Silly that the API allows this at all.

> Source/WebCore/Modules/entriesapi/FileSystemEntry.cpp:74
> +        if (result.hasException()) {
> +            if (errorCallback)
> +                errorCallback->handleEvent(DOMException::create(result.releaseException()));
> +            return;
> +        }

This is not covered by the layout tests.

[Chris Dumez]

Created attachment 319607 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 319607 [details]
Patch

Rejecting attachment 319607 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'validate-changelog', '--check-oops', '--non-interactive', 319607, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Ansdreas Kling found in /Volumes/Data/EWS/WebKit/LayoutTests/ChangeLog does not appear to be a valid reviewer according to contributors.json.
/Volumes/Data/EWS/WebKit/LayoutTests/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).

Full output: http://webkit-queues.webkit.org/results/4427131

[Chris Dumez]

Created attachment 319609 [details]
Patch

[Andreas Kling]

r=hans

[WebKit Commit Bot]

Comment on attachment 319609 [details]
Patch

Clearing flags on attachment: 319609

Committed r221481: <http://trac.webkit.org/changeset/221481>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.