Focus-android: Focus app crashes when tapping "Back" button while "Report site issue" was in loading

Created on 21 Dec 2018  路  8Comments  路  Source: mozilla-mobile/focus-android

Steps to reproduce

  1. Navigate to http://searchguide.windstream.net/index.php?
  2. Tap "More" button.
  3. Tap "Report site Issue" button.
  4. Tap "Back" button fast (before "Webcompat.com>Report bug" site is displayed).

Expected behavior

The web page is displayed and "Webcompat.com>Report bug" not triggered.

Actual behavior

Application crashes.

Device information

  • Android device: Google Pixel (Android 9) - 1080 x 1920 pixels (~441 ppi pixel density)
  • Focus version: 7.0.13 (Build #323021954 馃 62.0.3.20180922023437)

Notes:

  1. Reproducibility rate: 7/10
  2. Also reproducible while being on other web pages.
  3. Not reproducible on other devices (Samsung Galaxy S7 Edge (Android 8.0.0) - Resolution 1440 x 2560 pixels (~534 ppi pixel density)).
  4. Not reproducible on Focus without GeckoView.
  5. Attached gif and crash log CrashLogs.txt.

crashbackbtn

Geckoview馃 QAApproved bug testing

Most helpful comment

@vesta0 Now I remember, yeah I think this is intentional, we want to make GV better from this information, and we can't do much with sites that break WV.

All 8 comments

@softvision-oana-arbuzov I wasn't able to reproduce it on Focus 8.0.4/GV 64 so I am hoping it was fixed. Would you please give it another try?

@boek trying to repro this, but the Report Site issue settings option is missing from 8.0.5. Did we remove it?

screen

I can see Report Site Issue in 8.0.5

screenshot_20190102-143355
screenshot_20190102-143252

@rpappalax we just did some digging around with @npark-mozilla and found out that Report Site Issue only shows up in GeckoView. If you're on WebView, you won't see it.

@Sdaswani do you know if this was intentional?

@vesta0 Now I remember, yeah I think this is intentional, we want to make GV better from this information, and we can't do much with sites that break WV.

@sv-ohorvath could you please re-test this?

@vesta0, I've retested the issue with the latest Focus build and was not able to reproduce it.

Device information

  • Android device: Google Pixel (Android 9) - 1080 x 1920 pixels (~441 ppi pixel density)
  • Focus version: 8.0.4 (Build #330081229 馃 64.0.20181214004633)

It seems to be fixed, so I'll close it.

Was this page helpful?
0 / 5 - 0 ratings