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

Issue 130003003: QUIC - minor clean up changes found while merging changes into internal (Closed)

Created:
6 years, 11 months ago by ramant (doing other things)
Modified:
6 years, 11 months ago
Reviewers:
Ryan Hamilton
CC:
chromium-reviews, cbentzel+watch_chromium.org
Visibility:
Public.

Description

QUIC - minor clean up changes found while merging changes into internal code base. R=rch@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243880

Patch Set 1 #

Patch Set 2 : Merging with TOT #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -3 lines) Patch
M net/quic/quic_sent_packet_manager.cc View 1 chunk +1 line, -2 lines 0 comments Download
M net/quic/quic_stream_sequencer.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
ramant (doing other things)
6 years, 11 months ago (2014-01-09 01:42:39 UTC) #1
Ryan Hamilton
lgtm
6 years, 11 months ago (2014-01-09 01:45:18 UTC) #2
ramant (doing other things)
On 2014/01/09 01:45:18, Ryan Hamilton wrote: > lgtm You are amazing. Very fast. thanks for ...
6 years, 11 months ago (2014-01-09 02:22:03 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rtenneti@chromium.org/130003003/1
6 years, 11 months ago (2014-01-09 02:22:25 UTC) #4
commit-bot: I haz the power
Failed to apply patch for net/quic/congestion_control/inter_arrival_probe.cc: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
6 years, 11 months ago (2014-01-09 02:22:28 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rtenneti@chromium.org/130003003/60001
6 years, 11 months ago (2014-01-09 02:55:19 UTC) #6
commit-bot: I haz the power
6 years, 11 months ago (2014-01-09 13:04:48 UTC) #7
Message was sent while issue was closed.
Change committed as 243880

Powered by Google App Engine
This is Rietveld 408576698