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

Issue 686624 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug-Regression


Participants' hotlists:
media-router-PE


Sign in to add a comment

Regression: Unnecessary 'Cast tab' option is seen instead of 'Cast to' on opening cast for second time.

Reported by jshan...@etouch.net, Jan 30 2017

Issue description

Chrome Version: 57.0.2987.18 (Official Build) ad76d4a118911f50fc0aa39ff30b18faf05eff55-refs/branch-heads/2987@{#164} (32/64-bit)
OS: Windows(7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.11.6, 10.12.1)

Steps:
1. Launch Chrome and open Cast overlay on NTP via context menu
2. Click on 'Cast to', then click on 'Cast tab' or 'Cast desktop' and close the cast overlay
3. Again open Cast overlay via context menu and observe

Actual: Unnecessary 'Cast tab' option is seen instead of 'Cast to' on opening the cast overlay for second time.

Expected: 'Cast to' option should be seen on opening the cast overlay for every new instance.

This is regression issue broken in M-57 and will soon update bisect info :

Good build:57.0.2937.0
Bad build: 57.0.2938.0

 
Actual_video.mp4
597 KB View Download
Expected_video.mp4
495 KB View Download
Cc: jmukthavaram@chromium.org
Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: taku...@chromium.org
Status: Assigned (was: Unconfirmed)
Please find the below per-revision bisect information:
Good build:57.0.2937.0— Revision—435110
Bad build: 57.0.2938.0—Revision—435514

Bisect Tool Info:
----------------
You are probably looking for a change made after 435460 (known good), but no later than 435461 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspectas some perf builds might get missing due to failure.
  https://chromium.googlesource.com/chromium/src/+log/6130422db2a8ff2a08ab0cd2b19d861825c900ae..e5d669a3e9782b50b35a24d2e380b577c73b7a16

Review-Url: https://codereview.chromium.org/2487673003
takumif@ Kindly take a look and please help us to reassign this issue to a right owner if not with respect to this change.

Thank you!

Comment 2 by mfo...@chromium.org, Jan 30 2017

Cc: sko...@chromium.org
Components: -Blink>PresentationAPI Internals>Cast>UI
Labels: -Pri-1 Pri-2
+skonig 

We might want to handle the NTP specifically as I'm not sure the override makes sense there.  Thoughts Stephen?

Comment 3 by sko...@chromium.org, Jan 30 2017

Cc: taku...@chromium.org
Labels: -Pri-2 -ReleaseBlock-Stable -M-57 Pri-3
Owner: ----
Status: Available (was: Assigned)
We could blacklist the override on NTP, but that's hardly a high priority.

Comment 4 by mfo...@chromium.org, Feb 15 2017

Labels: Hotlist-Polish

Comment 5 by sko...@chromium.org, May 10 2017

Labels: Hotlist-Fixit-PE2017

Comment 6 by amp@chromium.org, Mar 30 2018

Owner: amyroberts@chromium.org
Status: Assigned (was: Available)
Assigning to Amy for Harmony consideration

Comment 7 by mfo...@chromium.org, Mar 31 2018

Labels: -Hotlist-Fixit-PE2017
noted, added scenario to spec list for Harmony documentation 

Sign in to add a comment