Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Bug] Crash while uploading Gif to google images #12494

Closed
s-ankur opened this issue Jul 12, 2020 · 4 comments
Closed

[Bug] Crash while uploading Gif to google images #12494

s-ankur opened this issue Jul 12, 2020 · 4 comments
Assignees
Labels
b:crash Crashes Fenix: should link to Sentry, Crash-Stats or GPlay info 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:RequestDesktopSite needs:ac Needs Android Component Work S3 Blocks non-critical functionality and a work around exists

Comments

@s-ankur
Copy link
Contributor

s-ankur commented Jul 12, 2020

Steps to reproduce

Turn on Desktop mode
Open google images and click ont he camera icon to upload image
Choose a gif

Expected behavior

Upload properly

Actual behavior

Crash

f7333b47-1545-4238-8cae-ef4d48e26525
java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=72649, result=-1, data=Intent { dat=content://media/external/file/109537 typ=image/gif flg=0x1 }} to activity {org.mozilla.fenix/org.mozilla.fenix.HomeActivity}: java.lang.IllegalStateException: cursor.getString(nameIndex) must not be null
----
java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=72649, result=-1, data=Intent { dat=content://media/external/file/109537 typ=image/gif flg=0x1 }} to activity {org.mozilla.fenix/org.mozilla.fenix.HomeActivity}: java.lang.IllegalStateException: cursor.getString(nameIndex) must not be null
	at android.app.ActivityThread.deliverResults(ActivityThread.java:4398)
	at android.app.ActivityThread.handleSendResult(ActivityThread.java:4440)
	at android.app.servertransaction.ActivityResultItem.execute(ActivityResultItem.java:49)
	at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108)
	at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1816)
	at android.os.Handler.dispatchMessage(Handler.java:106)
	at android.os.Looper.loop(Looper.java:193)
	at android.app.ActivityThread.main(ActivityThread.java:6718)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:491)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)
Caused by: java.lang.IllegalStateException: cursor.getString(nameIndex) must not be null
	at kotlin.jvm.internal.ArrayIteratorKt.checkExpressionValueIsNotNull(ArrayIterator.kt:1)
	at mozilla.components.browser.engine.gecko.prompt.GeckoPromptDelegate.toFileUri(GeckoPromptDelegate.kt:9)
	at mozilla.components.browser.engine.gecko.prompt.GeckoPromptDelegate.access$toFileUri(GeckoPromptDelegate.kt:1)
	at mozilla.components.browser.engine.gecko.prompt.GeckoPromptDelegate$onFilePrompt$onSelectSingle$1.invoke(GeckoPromptDelegate.kt:3)
	at mozilla.components.feature.prompts.file.FilePicker.handleFilePickerIntentResult$feature_prompts_release(FilePicker.kt:6)
	at mozilla.components.feature.prompts.file.FilePicker$onActivityResult$1.invoke(FilePicker.kt:5)
	at androidx.core.app.AppOpsManagerCompat.consumePromptFrom(AppOpsManagerCompat.java:5)
	at androidx.core.app.AppOpsManagerCompat.consumePromptFrom$default(AppOpsManagerCompat.java:1)
	at mozilla.components.feature.prompts.file.FilePicker.onActivityResult(FilePicker.kt:1)
	at mozilla.components.feature.prompts.PromptFeature.onActivityResult(PromptFeature.kt:1)
	at org.mozilla.fenix.browser.BaseBrowserFragment$onActivityResult$1.invoke(BaseBrowserFragment.kt:3)
	at mozilla.components.support.base.feature.ViewBoundFeatureWrapper.withFeature(ViewBoundFeatureWrapper.kt:1)
	at org.mozilla.fenix.browser.BaseBrowserFragment.onActivityResult(BaseBrowserFragment.kt:1)
	at androidx.fragment.app.FragmentActivity.onActivityResult(FragmentActivity.java:7)
	at android.app.Activity.dispatchActivityResult(Activity.java:7462)
	at android.app.ActivityThread.deliverResults(ActivityThread.java:4391)
	... 11 more


Device information

  • Android device: ?
  • Fenix version: ?

┆Issue is synchronized with this Jira Task

@s-ankur s-ankur added the 🐞 bug Crashes, Something isn't working, .. label Jul 12, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Jul 12, 2020
@AndiAJ
Copy link
Collaborator

AndiAJ commented Jul 13, 2020

Hi @s-ankur , I wasn't able to reproduce this issue on the latest Fenix Nightly Build 200713 from 7/13 using the following devices:
• Google Pixel 3a (Android 10)
• Huawei Mate 20 Lite (Android 9)
• OnePlus A3 (Android 6.0.1)

► Video
20200713-150427

@AndiAJ AndiAJ added b:crash Crashes Fenix: should link to Sentry, Crash-Stats or GPlay info S3 Blocks non-critical functionality and a work around exists Feature:RequestDesktopSite and removed needs:triage Issue needs triage labels Jul 13, 2020
@s-ankur
Copy link
Contributor Author

s-ankur commented Jul 13, 2020

After some testing I found that it only happens when using simplemobiletools gallery app and it is not limited to google images but happens on any site where you can upload images
2

@ekager ekager added the needs:ac Needs Android Component Work label Jul 13, 2020
@Amejia481 Amejia481 self-assigned this Jul 13, 2020
@data-sync-user data-sync-user changed the title [Bug] Crash while uploading Gif to google images FNX3-13562 ⁃ [Bug] Crash while uploading Gif to google images Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-13562 ⁃ [Bug] Crash while uploading Gif to google images FNX-13567 ⁃ [Bug] Crash while uploading Gif to google images Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-13567 ⁃ [Bug] Crash while uploading Gif to google images FNX2-15289 ⁃ [Bug] Crash while uploading Gif to google images Aug 11, 2020
@kbrosnan kbrosnan changed the title FNX2-15289 ⁃ [Bug] Crash while uploading Gif to google images [Bug] Crash while uploading Gif to google images Aug 26, 2020
@Amejia481
Copy link
Contributor

Amejia481 commented Aug 31, 2020

I think this was fixed by mozilla-mobile/android-components#8104 let's ask QA to verify

@Amejia481 Amejia481 added the eng:qa:needed QA Needed label Aug 31, 2020
@AndiAJ
Copy link
Collaborator

AndiAJ commented Aug 31, 2020

Hi, verified as fixed on the latest Nightly Build 200831 using the following devices:
• Google Pixel 3a (Android 10)
• Huawei Mate 20 Lite (Android 10)
• OnePlus A3 (Android 6.0.1)

► Video
20200831-135214

@AndiAJ AndiAJ closed this as completed Aug 31, 2020
@AndiAJ AndiAJ added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Aug 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
b:crash Crashes Fenix: should link to Sentry, Crash-Stats or GPlay info 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:RequestDesktopSite needs:ac Needs Android Component Work S3 Blocks non-critical functionality and a work around exists
Projects
None yet
Development

No branches or pull requests

4 participants