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

Issue 659423 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[Presentation API] sets initial state of PresentationConnection to 'connecting'

Project Member Reported by zhaobin@chromium.org, Oct 26 2016

Issue description

Change initial state of PresentationConnection from 'connected' to 'connecting'.

MR extension side change CL/137086303 should land first.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/badd595e3a5263107e93fea0e1bd943186291153

commit badd595e3a5263107e93fea0e1bd943186291153
Author: zhaobin <zhaobin@chromium.org>
Date: Wed Nov 09 18:25:50 2016

[Presentation API] change initial state of PresentationConnection to 'connecting'

- change initial state to 'connecting'
- transition to 'connected' state in PresentationServiceImpl (browser side) as a work around
- verified Youtube and CastHelloVideo works fine for Cast; Youtube works fine for DIAL

BUG= 659423 

Review-Url: https://codereview.chromium.org/2470023002
Cr-Commit-Position: refs/heads/master@{#430983}

[modify] https://crrev.com/badd595e3a5263107e93fea0e1bd943186291153/chrome/test/media_router/resources/common.js
[modify] https://crrev.com/badd595e3a5263107e93fea0e1bd943186291153/content/browser/presentation/presentation_service_impl.cc
[modify] https://crrev.com/badd595e3a5263107e93fea0e1bd943186291153/content/browser/presentation/presentation_service_impl.h
[modify] https://crrev.com/badd595e3a5263107e93fea0e1bd943186291153/content/browser/presentation/presentation_service_impl_unittest.cc
[modify] https://crrev.com/badd595e3a5263107e93fea0e1bd943186291153/third_party/WebKit/Source/modules/presentation/PresentationConnection.cpp

Status: Fixed (was: Untriaged)

Comment 3 by mfo...@chromium.org, Mar 24 2017

Components: Blink>PresentationAPI

Comment 4 by mfo...@chromium.org, Mar 24 2017

Labels: -PresentationAPI

Sign in to add a comment