Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1102)

Issue 2757733005: Revert of Don't set the priority of the decoder to 'high' on Android. (Closed)

Created:
3 years, 9 months ago by tommi
Modified:
3 years, 9 months ago
Reviewers:
stefan-webrtc
CC:
webrtc-reviews_webrtc.org, video-team_agora.io, yujie_mao (webrtc), zhengzhonghou_agora.io, tterriberry_mozilla.com, mflodman
Target Ref:
refs/heads/master
Project:
webrtc
Visibility:
Public.

Description

Revert of Don't set the priority of the decoder to 'high' on Android. (patchset #1 id:1 of https://codereview.webrtc.org/2745813003/ ) Reason for revert: Reverting this as it had no effect. Original issue's description: > Don't set the priority of the decoder to 'high' on Android. > Doing so competes with the actual decoding that happens on a different thread. > > BUG=695438 > > Review-Url: https://codereview.webrtc.org/2745813003 > Cr-Commit-Position: refs/heads/master@{#17184} > Committed: https://chromium.googlesource.com/external/webrtc/+/ca37cf669174184b5139ffd0141bf7f6c0fc7894 TBR=stefan@webrtc.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=695438 Review-Url: https://codereview.webrtc.org/2757733005 Cr-Commit-Position: refs/heads/master@{#17297} Committed: https://chromium.googlesource.com/external/webrtc/+/5f54419ae516adcc935939079b89c5c9ede6adf0

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -7 lines) Patch
M webrtc/video/video_receive_stream.cc View 1 chunk +0 lines, -7 lines 0 comments Download

Messages

Total messages: 6 (3 generated)
tommi
Created Revert of Don't set the priority of the decoder to 'high' on Android.
3 years, 9 months ago (2017-03-17 18:20:58 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2757733005/1
3 years, 9 months ago (2017-03-17 18:21:09 UTC) #3
commit-bot: I haz the power
3 years, 9 months ago (2017-03-17 18:54:33 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/5f54419ae516adcc935939079...

Powered by Google App Engine
This is Rietveld 408576698