1289255 - Implement DataTransferItem.webkitGetAsEntry

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P2)

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

Bug 1289254 adds IsDirectory() to BlobImpl which should make this rather trivial.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Attachment: wip

I took the patch 7 from bug 1284987 and added support for DOMFileSystem and
directories.

The only issue I know atm is that FileEntry doesn't have fullpath.
baku, how would one get fullpath information to the FileEntries?
Basically /<filename>

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Looks like my patch for bug Bug 1289254 has the same issue with FileEntries. They are missing the fullpath

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=8216d6a07c17

Comment 6

[Andrea Marchesini [:baku]]

Comment on attachment 8775320 [details] [diff] [review]
patch

Review of attachment 8775320 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/events/DataTransferItem.cpp
@@ +282,5 @@
> +  }
> +
> +  nsCOMPtr<nsIGlobalObject> global;
> +  RefPtr<DataTransfer> dataTransfer;
> +  RefPtr<DataTransferItemList> list = GetParentObject();

Nit, but just to make this code more readable, what about if we do:

if (!list) {
  return nullptr;
}

RefPtr<DataTransfer> dataTransfer = list->GetParentObject();
if (!dataTransfer) {
  return nullptr;
}

@@ +306,5 @@
> +    return nullptr;
> +  }
> +
> +  RefPtr<DOMFileSystem> fs = DOMFileSystem::Create(global);
> +  Sequence<RefPtr<Entry>> entries;

move this  to line  336.

@@ +309,5 @@
> +  RefPtr<DOMFileSystem> fs = DOMFileSystem::Create(global);
> +  Sequence<RefPtr<Entry>> entries;
> +  RefPtr<Entry> entry;
> +
> +  BlobImpl* impl = file->Impl();

MOZ_ASSERT(impl);

and remove if(impl) part.

@@ +320,5 @@
> +    }
> +
> +    nsCOMPtr<nsIFile> directoryFile;
> +    NS_ConvertUTF16toUTF8 path(fullpath);
> +    nsresult rv = NS_NewNativeLocalFile(path, true,

nsresult rv = NS_NewNativeLocalFile(NS_ConvertUTF16toUTF8(fullPath),
                                    true, getter_...);

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

Attachment: webkitGetAsEntry_3.diff

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5e254e995379
Implement DataTransferItem.webkitGetAsEntry, r=baku

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5e254e995379

Comment 10

[Eric Shepherd [:sheppy]]

Documented, with lovely live sample to boot (which is reused on other pages about file system, too):

https://developer.mozilla.org/en-US/docs/Web/API/DataTransferItem/webkitGetAsEntry

Updated https://developer.mozilla.org/en-US/docs/Web/API/DataTransferItem to include it.

Listed on Firefox 50 for developers.