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

[Bug] javascript: scheme is not handled inside Fenix #4286

Closed
makotom opened this issue Jul 25, 2019 · 8 comments
Closed

[Bug] javascript: scheme is not handled inside Fenix #4286

makotom opened this issue Jul 25, 2019 · 8 comments
Labels
b:webcompat Web compatability - https://webcompat.com 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified

Comments

@makotom
Copy link

makotom commented Jul 25, 2019

Steps to reproduce

  1. Open https://makotom.net/tc-20190725.html .
  2. Tap the text javascript:console.log('fired').

Expected behavior

Nothing noticeable should happen in any tab. (Observed behaviour in the conventional Firefox Mobile for Android.)

Actual behavior

A: In the case the page was opened in a normal tab, Chrome was opened upon tapping the text.
B: In the case the page was opened in a private tab, a prompt saying "Open in app? Your activity may no longer be priate." was shown.

Device information

  • Android device: Pixel 3a, Build PQ3B.190705.003
  • Fenix version:
    Screenshot_20190725-133105

Footnotes

┆Issue is synchronized with this Jira Task

@makotom makotom added the 🐞 bug Crashes, Something isn't working, .. label Jul 25, 2019
@makotom
Copy link
Author

makotom commented Jul 25, 2019

I'm uploading the content of https://makotom.net/tc-20190725.html as tc-20190725.html.txt.

@bogas04
Copy link

bogas04 commented Jul 25, 2019

Can face the issue on Amazon website.

Screen cast of the issue on Amazon's reviews section

Device: RealmeX
Operating System: ColorOS 6 based on Android Pie.
Browser: Nightly 190725 06:23 (Build #12060631) 📦: 6.0.0 🦎: 68.0-20190711090008

@NotWoods NotWoods added the b:webcompat Web compatability - https://webcompat.com label Jul 28, 2019
@makotom
Copy link
Author

makotom commented Jul 30, 2019

I recognize some relation between this issue and #1570 , because I see this issue reappeared aftter #1570 was fixed.

Maybe, the issue in interest should be #4114.

@johnsto
Copy link

johnsto commented Aug 1, 2019

Same issue here on build 1.1.0#12042112:

edit_20190728_091533_1

@johnsto
Copy link

johnsto commented Aug 8, 2019

Seems fixed in 1.2.0 (Build #12181645)!

@makotom
Copy link
Author

makotom commented Aug 20, 2019

This should have been fixed in mozilla-mobile/android-components@688d1f2 for mozilla-mobile/android-components#3944.

Can we close this issue for now?

@NotWoods
Copy link
Contributor

Also could not reproduce on Nightly 191109.

@NotWoods NotWoods added the eng:qa:needed QA Needed label Nov 15, 2019
@AndiAJ AndiAJ added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Nov 15, 2019
@AndiAJ
Copy link
Collaborator

AndiAJ commented Nov 15, 2019

Hi, verified as fixed on the latest Nightly Build #13190606 from 11/15 using the following devices:
• Google Pixel 3a XL (Android 9)
• Huawei Mate 20 Lite (Android 8.1.0)
• Samsung Galaxy S7 (Android 7)
• OnePlus A3 (Android 6.0.1)
• LG Nexus 4 (Android 5.1.1)

► Video
20191115_154052

► Video
20191115_154248

@AndiAJ AndiAJ closed this as completed Nov 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
b:webcompat Web compatability - https://webcompat.com 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified
Projects
None yet
Development

No branches or pull requests

5 participants