@capacitor/browser add a new BROWSER_CLOSED event

[fikretkurtiiev]

Feature Request

Plugin

@capacitor/browser

Description

This is probably feature-request and bug issue as well :)
I faced with problem when browser window is minimized (not closed) and event browserFinished Is triggered.
I need to know when is was closed, not minimized. Take a look the attached screenshot, it has "minimize" control.

Platform(s)

Android only.

Preferred Solution

Add one more event here BROWSER_CLOSED or don't trigger BROWSER_FINISHED while inappbrowser is minimized.

Alternatives

Additional Context

[ionitron-bot]

This issue has been labeled as type: bug. This label is added to issues that that have been reproduced and are being tracked in our internal issue tracker.

[cornflakes]

The PR: #2125 would in turn fix this bug.

This bug is introduced in commit e2cdf35 by @ItsChaceD on January 22nd, 2024.

The issue arises because the URL is opened from an activity instead of the plugin itself. This causes the unbindService method (called from handleOnPause in the plugin) to be invoked before the open method (called from the activity callback). This results in group.enter() being called before group.reset(). Consequently, when group.leave() is triggered from the TAB_HIDDEN navigation event, the browserEventListener.onBrowserEvent(BROWSER_FINISHED) event fires incorrectly when the app goes into the background.

The pull request fixes the bug by changing the count threshold from count <= 0 to count < 0, preventing the unintended firing of the BROWSER_FINISHED event.

While the order of calls remains unchanged, modifying the count threshold achieves the same outcome of ensuring group.reset() happens before group.leave().

[eric-horodyski]

This thread has been inactive for some time, and we’re cleaning up older items to keep the backlog focused and manageable.

Details on the effort:
https://ionic.io/blog/keeping-capacitors-backlog-healthy

If this is still relevant today, feel free to open a new issue with current information and we’ll review it.

[cornflakes]

I've just updated the PR... Why would you think it's not still relevant?

[ionitron-bot]

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of the plugin, please create a new issue and ensure the template is fully filled out.