New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 799032 link

Starred by 1 user
Status: Duplicate
Merged: issue 707224
Owner:
mfo...@chromium.org
Closed: Jan 2018
Cc:
█ imch...@chromium.org
mlamouri@chromium.org
mfo...@chromium.org
btolsch@chromium.org
powerb@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Availability change get triggered after tab change

Project Member Reported by fbeaufort@chromium.org, Jan 4 2018 
Chrome Version       : 65.0.3299.0
OS Version: 10272.0.0

What steps will reproduce the problem?
1. Make sure you're connected to a network with a Chromecast. 
2. Go to https://googlechrome.github.io/samples/presentation-api/
3. Click "new PresentationRequest()" button to start monitoring presentation displays
4. Switch between this tab and another random tab every 2 seconds.

What is the expected result?
It should not affect availability changes

What happens instead of that?
I get plenty of lines like below

Presentation displays are unavailable.
Presentation displays are available.
Presentation displays are unavailable.
Presentation displays are available.
Presentation displays are unavailable.
Presentation displays are available.
Presentation displays are unavailable.
Presentation displays are available.
Presentation displays are unavailable.
Presentation displays are available.

Comment 1 by mfo...@chromium.org, Jan 10 2018

Cc: imch...@chromium.org
Status: Assigned (was: Unconfirmed)
I can reproduce this, but I can't figure out why it's happening from a quick glance.  Moving a tab to the background does inactivate the presentation availability observer (and remove the associated listener from the presentation service) but I don't see a code path that fires an event.

+imcheng in case this is related to Onion Soup 2.0.

Comment 2 by mfo...@chromium.org, Jan 10 2018

Mergedinto: 707224
Status: Duplicate (was: Assigned)
Looks like this has been reported earlier.
►

Sign in to add a comment