Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(15)

Issue 2898553002: [Media,Remoting] Remove the disable remoting button on Android

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 days, 19 hours ago by whywhat
Modified:
1 hour, 2 minutes ago
CC:
darktears, apavlov+blink_chromium.org, blink-reviews, blink-reviews-css, blink-reviews-html_chromium.org, chromium-reviews, dglazkov+blink, eric.carlson_apple.com, feature-media-reviews_chromium.org, fs, gasubic, mlamouri+watch-blink_chromium.org, rwlbuis, nessy, Srirama
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

[Media,Remoting] Remove the disable remoting button on Android The --new-remote-playback-pipeline flag will stay disabled on desktop by default. BUG=517102 TEST=initiate remoting with --new-remote-playback-pipeline and check the button is not there

Patch Set 1 #

Patch Set 2 : Don't show the exit button on Android only for now #

Messages

Total messages: 17 (10 generated)
whywhat
PTaL
4 days, 19 hours ago (2017-05-19 22:44:24 UTC) #2
xjz
Can you please hold off this change? We are not quite ready yet. We need ...
1 day, 20 hours ago (2017-05-22 20:49:41 UTC) #7
whywhat
On 2017/05/22 at 20:49:41, xjz wrote: > Can you please hold off this change? We ...
1 day, 20 hours ago (2017-05-22 21:35:57 UTC) #8
xjz
On 2017/05/22 21:35:57, whywhat wrote: > On 2017/05/22 at 20:49:41, xjz wrote: > > Can ...
1 day, 19 hours ago (2017-05-22 21:58:47 UTC) #9
whywhat
Don't show the exit button on Android only for now
16 hours, 19 minutes ago (2017-05-24 01:25:21 UTC) #10
whywhat
PTAL I didn't want to introduce a separate runtime feature flag and it seems that ...
16 hours, 15 minutes ago (2017-05-24 01:29:15 UTC) #12
mlamouri (OOO up to 23-05)
1 hour, 2 minutes ago (2017-05-24 16:42:05 UTC) #17
avayvod@, I am not sure what this is trying to do: can you explain in the CL
description why this button is being removed and also how this is related to the
new pipeline?
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 650457f06