1489532 - file upload on lots websites are not working with latest nighlty 64 on Android

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[alessandro]

User Agent: Mozilla/5.0 (Android 7.0; Mobile; rv:57.0) Gecko/61.0 Firefox/61
Build ID: 20180904100117

Steps to reproduce:

file upload on lots websites are not working with latest nighlty 64 on Android
to reproduce install today nighlty 64 on Android and try to upload some files to wetransfer.com after selecting the file nothing happen.
with 2 days ado build (still Firefox Nightly 63) everything's od


Actual results:

I cannot upload files to wetransfer.com
2 days ago build works just fine. 


Expected results:

I don't really know how such invalidating bugs happen even on nighlty channel very disappointed.

Comment 1

[alessandro]

also Don t mind about my user agent I had to change it in about:config in order to make captcha work, I m (unfortunately) on latest 64 nightly.

Comment 2

[Jan Henning [:JanH]]

Do you get a permission prompt for storage and/or camera permission when that happens?

Comment 3

[alessandro]

 nope I dont get any permission prompt but in my opinion it s not a permission prompt problem as I already gave all possible permissions in setting also using two day ago build everything is OK. also the bug is super easy to repuduce just try to upload a file to wetransfer.com and see it for yourself nothing happens 
also keep in mind that I M still on marshmallow so permission shouldn't be an issue.

Comment 4

[Jan Henning [:JanH]]

Runtime permissions were introduced in Marshmallow, so they would matter.
The behaviour I'm seeing only happens when you get a runtime permission prompt and regressed in Firefox 60, so I'm opening a separate bug in case your issue is something else.

Comment 5

[alessandro]

sorry for bothering but if it 's a permission problem why on other websites like https://www.how-old.net/  I can upload photos just fine even with today build?
also i installed today build on a lollipop device and file uploaded just won't work as well

Comment 6

[Jan Henning [:JanH]]

It might be some device-specific issue? Unfortunately as far as I can tell I can only reproduce the issue described in bug 1489594. Maybe somebody else has more luck.

Comment 7

[alessandro]

I made a screen cast of the issue just in case
Link URL: http://sendanywhe.re/53U77QDY
this stupid issue totally ruined my Friday I spent over two hours (getting a terrible headache) in clearing app data reboot phone reinstalling etc etc it s time to let Firefox go.
and if u asking yourself why I use nighlty? because it since version 47 that I hope Firefox get a little speed boost but it slow and buggy just like years ago good luck

Comment 8

[Jan Henning [:JanH]]

Oh, now I see. This isn't about the file browser not appearing, but about the selected file not being properly processed afterwards. Will check that as well.

Comment 9

[Jan Henning [:JanH]]

https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=ad32f6300ecc205d1e4ae48ef20ff67dded7ac80&tochange=11351ef0d0b0a92db8a26828412f3b6586cccb7a

Comment 10

[Kris Maglione [:kmag]]

Attachment: Bug 1489532: Check for nsISimpleEnumerator wrapper before searching existing tear-offs. r=mccr8

When an XPC wrapped JS object returns an object from a property or function
which is expected to return a given interface, we create a wrapper for that
interface regardless of whether the object has a QueryInterface method which
claims to support it. For the nsISimpleEnumerator enumerator case, the check
for that existing wrapper comes before the check for the nsISimpleEnumerator
special casing, which breaks automatic conversion in those cases.

This patch moves that check just before the check for the existing wrapper,
and just after the nsIPropertyBag special casing, which has similar semantics.

Comment 11

[Andrew McCreight [:mccr8]]

Comment on attachment 9007389 [details]
Bug 1489532: Check for nsISimpleEnumerator wrapper before searching existing tear-offs. r=mccr8

Andrew McCreight [:mccr8] has approved the revision.

Comment 12

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a5dd7c81da0ed4e9eaf3181b99298566dfa319f5
Bug 1489532: Check for nsISimpleEnumerator wrapper before searching existing tear-offs. r=mccr8

Comment 13

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/a5dd7c81da0e

Comment 14

[Jan Henning [:JanH]]

Wetransfer file upload working again.