@@ +584,5 @@
>> +
>> + // We only support dropping files onto a file upload control
>> + PRBool typeSupported;
>> + types->Contains(NS_LITERAL_STRING("Files"), &typeSupported);
>> + return typeSupported;
> You can also get the file list and check if it's empty.
No, you shouldn't do that during dragover, as it will unnecessarily cause the need to retrieve the data of the drag from the source application which can be slow.
Comment on attachment 534375
Patch 2
@@ +584,5 @@ Contains( NS_LITERAL_ STRING( "Files" ), &typeSupported);
>> +
>> + // We only support dropping files onto a file upload control
>> + PRBool typeSupported;
>> + types->
>> + return typeSupported;
> You can also get the file list and check if it's empty.
No, you shouldn't do that during dragover, as it will unnecessarily cause the need to retrieve the data of the drag from the source application which can be slow.