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

"Open in..." option doesn't display the debug Focus builds installed #5632

Closed
lobontiumira opened this issue Oct 15, 2021 · 9 comments
Closed

Comments

@lobontiumira
Copy link

lobontiumira commented Oct 15, 2021

Prerequisites

Make sure you have at least two Firefox Focus builds installed, and at least one of them is a debug build.

Steps to reproduce

  1. Install Beta 94.0.0-beta.1.
  2. Open a page, and from the three-dot menu select the "Open in..." option.
  3. Observe the installed apps displayed in the "Open in..." dialogue.

Expected behavior

All the installed apps that are suitable for opening the page are displayed.

Actual behavior

The Focus debug builds installed are not displayed at all.
I had the latest debug build from main from 10/15, and Firefox Focus Nightly.

Device information

  • Android device: Lenovo Tab M10 (Android 10), Pixel 2 (Android 9)
  • Not reproducible on Google Pixel (Android 10)
  • Focus version: Beta 94.0.0-beta.1
  • Focus Nightly, and the debug build display Focus Beta, but not each other.

Screenshot_1

@mcarare
Copy link
Contributor

mcarare commented Oct 15, 2021

We should remove KnownBrowsers from Focus and use the one from AC. But I am not sure if that is updated either.
But I am not sure debug builds should show up either way.
cc @pocmo

@mcarare mcarare added the a-c Needs Android Components work label Oct 15, 2021
@pocmo
Copy link
Contributor

pocmo commented Oct 15, 2021

Agree, let's see if we can use the implementation from A-C. And yeah, we will have to add the new Focus Nightly/Beta builds to it too.

@lobontiumira
Copy link
Author

Reproducible on Beta 94.0.0-beta.2 (build 352922038 GV 94.0-20211017185800) with Lenovo Tab M10 (Android 10).

@ionutbedregeanu ionutbedregeanu self-assigned this Oct 29, 2021
@ionutbedregeanu ionutbedregeanu added this to Backlog in Android Focus via automation Oct 29, 2021
@ionutbedregeanu ionutbedregeanu added this to Ready for Engineering (min-5 ; max-22) in Android Engineering Team Kanban board via automation Oct 29, 2021
@ionutbedregeanu ionutbedregeanu moved this from Ready for Engineering (min-5 ; max-22) to In Development (WIP limit - 15) in Android Engineering Team Kanban board Oct 29, 2021
@ionutbedregeanu ionutbedregeanu moved this from Backlog to In Progress in Android Focus Oct 29, 2021
@ionutbedregeanu
Copy link
Contributor

@lobontiumira Can you please tell me if it's still reproducible? Because I didn't succeed to reproduce it

@ionutbedregeanu ionutbedregeanu moved this from In Development (WIP limit - 15) to Ready for Engineering (min-5 ; max-22) in Android Engineering Team Kanban board Oct 29, 2021
@ionutbedregeanu ionutbedregeanu moved this from In Progress to Backlog in Android Focus Oct 29, 2021
@lobontiumira
Copy link
Author

Yes, I was able to reproduce on RC 94.1.1, and on Beta 94.0.0-beta.2 with Lenovo tabley M10 (Android 10).

@ionutbedregeanu ionutbedregeanu moved this from Ready for Engineering (min-5 ; max-22) to In Development (WIP limit - 15) in Android Engineering Team Kanban board Nov 1, 2021
@ionutbedregeanu ionutbedregeanu moved this from Backlog to In Progress in Android Focus Nov 1, 2021
ionutbedregeanu pushed a commit to ionutbedregeanu/focus-android that referenced this issue Nov 1, 2021
Use the source of know browsers from android components
@ionutbedregeanu ionutbedregeanu moved this from In Development (WIP limit - 15) to Dev Complete (WIP limit - 5) in Android Engineering Team Kanban board Nov 1, 2021
ionutbedregeanu pushed a commit to ionutbedregeanu/focus-android that referenced this issue Nov 2, 2021
Remove number of tabs verification since is always one in a custom tab
and the tabs floating button is not visible
ionutbedregeanu pushed a commit to ionutbedregeanu/focus-android that referenced this issue Nov 2, 2021
Use the source of know browsers from android components
ionutbedregeanu pushed a commit to ionutbedregeanu/focus-android that referenced this issue Nov 2, 2021
Use the source of know browsers from android components
@ionutbedregeanu ionutbedregeanu added the QAReady flag for test read & needed label Nov 2, 2021
@ionutbedregeanu ionutbedregeanu moved this from Dev Complete (WIP limit - 5) to Reviewer approved, ready for test (WIP limit - ?) in Android Engineering Team Kanban board Nov 2, 2021
mergify bot pushed a commit that referenced this issue Nov 2, 2021
Use the source of know browsers from android components
@delia-pop
Copy link

Tested on Beta 95.0.0-beta.1 build and latest debug build from main 11/04 with Huawei P9 Lite (Android 8).

The issue is verified fixed on the debug build. With the debug build set as default browser, on both browser page and custom tab the suitable applications to open the page with are displayed: chrome, beta build and the debug build itself.

@ionutbedregeanu Please note that with the Beta 95.0.0-beta.1 build set as default browser, on both browser page and custom tab only the chrome and Debug build are displayed, without the Beta build itself.

Capture

@lobontiumira lobontiumira removed the QAReady flag for test read & needed label Nov 4, 2021
@ionutbedregeanu
Copy link
Contributor

ionutbedregeanu commented Nov 8, 2021

I don't think we should show the current browser in Open in list. We should log a bug for that.

@pocmo pocmo closed this as completed Nov 11, 2021
Android Engineering Team Kanban board automation moved this from Reviewer approved, ready for test (WIP limit - ?) to Done Nov 11, 2021
Android Focus automation moved this from In Progress to Done Nov 11, 2021
@lobontiumira
Copy link
Author

This issue still reproduces on RC 95.1.0 with Lenovo tab M10 (Android 10), and Sony Xperia Z5 Premium (Android 7.1.1).

@lobontiumira
Copy link
Author

This issue doesn't reproduce in Beta 96.0.0-beta.1, nor in Focus Nightly from 12/7.
It only reproduces on RC 95.1.0 with some devices, like Sony Xperia Z5 Premium (Android 7.1.1), and LEnovo tablet M10 (Android 10):

open in

Note: it does not reproduce on RC 95.1.0 on the following devices:

  • Samsung Galaxy Note10 (Android 11)
  • LG G7 fit (Android 8.1.0)
  • Xiaomi Redmi Note 8T (Android 9)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a-c Needs Android Components work bug QAVerified
Projects
Android Focus
  
Done
Development

No branches or pull requests

5 participants