New issue
Advanced search Search tips

Issue 778583 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac , Fuchsia
Pri: 1
Type: Bug



Sign in to add a comment

Merge to M63: Enable CPU adaptation for screenshare in experiment

Project Member Reported by ilnik@chromium.org, Oct 26 2017

Issue description

This is a trivial fix for a bug, which blocks rollout of important experiment.

Original WebRTC bug: https://bugs.chromium.org/p/webrtc/issues/detail?id=8433

Landed in chrome here: https://chromium.googlesource.com/chromium/src/+/d6800dad034f164a10ba94e18bd24fb524b01cec
which made it to 64.0.3250.0

Change doesn't affect users without the experiment, and users under experiment without the change may have bad screensharing experience.
 

Comment 1 by rbyers@chromium.org, Oct 26 2017

Components: -Blink
Project Member

Comment 2 by sheriffbot@chromium.org, Oct 26 2017

Labels: -Merge-Request-63 Merge-Review-63 Hotlist-Merge-Review
This bug requires manual review: M63 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), gkihumba@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by gov...@chromium.org, Oct 26 2017

How is the change looking in Canary? If the change looks good in Canary, I will approve the merge.

Comment 4 by ilnik@chromium.org, Oct 26 2017

It works, looks good. Also, you can't even see any changes without specific field trials enabled. 

Comment 5 by gov...@chromium.org, Oct 26 2017

Labels: -Merge-Review-63 Merge-Approved-63
Approving merge to M63 branch 3239 based on comment #0 & #4. Please merge ASAP. Thank you.

Comment 6 by ilnik@chromium.org, Oct 27 2017

Labels: -Hotlist-Merge-Review -Merge-Approved-63 merge-merged-3239 M-63
Status: Fixed (was: Assigned)

Comment 7 by gov...@chromium.org, Oct 27 2017

could you please provide link for M63 merge cl?

Comment 9 by gov...@chromium.org, Oct 27 2017

Labels: merge-merged-m63
Thank you. 

Comment 10 by ilnik@chromium.org, Oct 27 2017

No, thank you for approval. Sorry, I forgot to add it in the first place.
Blocking: 611909
Blocking: -611909

Sign in to add a comment