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

Issue 724223 link

Starred by 1 user
Status: Fixed
Owner:
█ x...@chromium.org
Last visit > 30 days ago
Closed: May 2017
Cc:
█ avayvod@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Remove redundant call to hide the interstitial when media remoting is disabled.

Project Member Reported by x...@chromium.org, May 18 2017 
When media remoting is disabled by clicking the disable button, the remoting interstitial is hidden when remoting stops. There is no need to call MediaRemotingStopped() again to hide the interstitial.
Project Member

Comment 1 by bugdroid1@chromium.org, May 26 2017 

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

commit 108ef61710f08366884633492aa05a23b699bc44
Author: xjz <xjz@chromium.org>
Date: Fri May 26 20:59:41 2017

Remove redundant call and early check when media remoting is disabled.

Remove the redundant call to hide the interstitial when media remoting
is disabled. And also removed the early return check when media
remoting is stopped.

BUG= 724223 

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

[modify] https://crrev.com/108ef61710f08366884633492aa05a23b699bc44/third_party/WebKit/Source/core/html/HTMLVideoElement.cpp

Comment 2 by x...@chromium.org, May 26 2017

Status: Fixed (was: Started)
►

Sign in to add a comment