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

Issue 2791883003: Remove Http2PriorityWriteScheduler. (Closed)

Created:
3 years, 8 months ago by Bence
Modified:
3 years, 8 months ago
Reviewers:
xunjieli
CC:
chromium-reviews, cbentzel+watch_chromium.org, bnc+watch_chromium.org, net-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove Http2PriorityWriteScheduler. This class has been checked in but not used for over a year. BUG=603810

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1553 lines) Patch
M net/BUILD.gn View 2 chunks +0 lines, -2 lines 0 comments Download
D net/spdy/http2_write_scheduler.h View 1 chunk +0 lines, -752 lines 0 comments Download
D net/spdy/http2_write_scheduler_test.cc View 1 chunk +0 lines, -799 lines 0 comments Download

Messages

Total messages: 14 (9 generated)
Bence
PTAL.
3 years, 8 months ago (2017-04-03 22:14:22 UTC) #6
xunjieli
On 2017/04/03 22:14:22, Bence wrote: > PTAL. LGTM! +1 to less code.
3 years, 8 months ago (2017-04-04 01:05:01 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2791883003/1
3 years, 8 months ago (2017-04-04 12:57:36 UTC) #9
commit-bot: I haz the power
Prior attempt to commit was detected, but we were not able to check whether the ...
3 years, 8 months ago (2017-04-04 13:07:45 UTC) #12
Bence
3 years, 8 months ago (2017-04-04 13:18:22 UTC) #13
This CL has successfully landed: closing manually.

Powered by Google App Engine
This is Rietveld 408576698