GoogleChrome / samples

A repo containing samples tied to new functionality in each release of Google Chrome.
https://www.chromestatus.com/samples
Apache License 2.0
5.79k stars 2.38k forks source link

requestLEscan not working any more #776

Closed arigus closed 2 years ago

arigus commented 2 years ago

this demo not working on latest chrome. Looke like it closes automatically when press scan button, so no grant to scan. Please, check and share how to fix it

beaufortfrancois commented 2 years ago

This issue is currently tracked at https://bugs.chromium.org/p/chromium/issues/detail?id=1323134.

arigus commented 2 years ago

So when would be fixed?

Sent from Yahoo Mail for iPhone

On Wednesday, June 29, 2022, 2:41 PM, François Beaufort @.***> wrote:

This issue is currently tracked at https://bugs.chromium.org/p/chromium/issues/detail?id=1323134.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

beaufortfrancois commented 2 years ago

There is no ETA yet. Your best bet is to follow progress in this bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1323134

arigus commented 2 years ago

Hi,Thanks for the prompt reply. Any workaround while waiting for the final fix?We used in smart school application for 2 years and need this function to work with.Thanks and Best Regards!Batulziiwww.sanchirtech.com On Wednesday, 29 June 2022 at 08:00:25 pm GMT+8, François Beaufort @.***> wrote:

There is no ETA yet. Your best bet is to follow progress in this bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1323134

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

beaufortfrancois commented 2 years ago

Noted. I've sent a message to the Chrome team at https://bugs.chromium.org/p/chromium/issues/detail?id=1323134#c12.

beaufortfrancois commented 2 years ago

For the record:

The regression of this issue is caused by https://chromium-review.googlesource.com/c/chromium/src/+/2117052. This is confirmed by resetting the chromium repo back to that commit and the issue can be observed right after having that change.

According to comment 27 (which links to an internal doc), we might want to evaluate a proper solution to address this issue. Current WIP CL https://chromium-review.googlesource.com/c/chromium/src/+/3752062 doesn't fix the issue in an appropriate way (ex: other kinds of bubble popping up might stop operations and cause more confusion). Besides, we can't just workaround it by ignoring focus change without proper security related checks, so we might want to take more time to evaluate a proper solution.

But please rest assure this is in our priority list as this is a blocker for Issue 654897 and Issue 897312 which we are committed to work on in this Q3.

Source: https://bugs.chromium.org/p/chromium/issues/detail?id=1323134#c28

arigus commented 1 year ago

Hi, Any update?BR, Ulzii On Wednesday, 3 August 2022 at 02:58:07 pm GMT+8, François Beaufort @.***> wrote:

For the record:

The regression of this issue is caused by https://chromium-review.googlesource.com/c/chromium/src/+/2117052. This is confirmed by resetting the chromium repo back to that commit and the issue can be observed right after having that change.

According to comment 27 (which links to an internal doc), we might want to evaluate a proper solution to address this issue. Current WIP CL https://chromium-review.googlesource.com/c/chromium/src/+/3752062 doesn't fix the issue in an appropriate way (ex: other kinds of bubble popping up might stop operations and cause more confusion). Besides, we can't just workaround it by ignoring focus change without proper security related checks, so we might want to take more time to evaluate a proper solution.

But please rest assure this is in our priority list as this is a blocker for Issue 654897 and Issue 897312 which we are committed to work on in this Q3.

Source: https://bugs.chromium.org/p/chromium/issues/detail?id=1323134#c28

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>