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

Issue 2388453002: Report upload progress every 100ms (was 10ms) (Closed)

Created:
4 years, 2 months ago by Charlie Harrison
Modified:
4 years, 2 months ago
CC:
chromium-reviews, loading-reviews_chromium.org, jam, darin-cc_chromium.org, mmenke
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Report upload progress every 100ms (was 10ms) This was erroneously changed in a previous patch: https://codereview.chromium.org/1301103002 Let's revert it back to its original value. BUG=645704 Committed: https://crrev.com/90e3f912e7095ec556474e31771e6bafbe0c52f0 Cr-Commit-Position: refs/heads/master@{#422174}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M content/browser/loader/async_resource_handler.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 15 (9 generated)
Charlie Harrison
rdsmith@, PTAL at this small patch. I had bundled this change into another CL that ...
4 years, 2 months ago (2016-09-30 18:01:10 UTC) #4
Randy Smith (Not in Mondays)
On 2016/09/30 18:01:10, Charlie Harrison wrote: > rdsmith@, PTAL at this small patch. I had ...
4 years, 2 months ago (2016-09-30 19:06:04 UTC) #7
Charlie Harrison
On 2016/09/30 19:06:04, Randy Smith - Not in Mondays wrote: > On 2016/09/30 18:01:10, Charlie ...
4 years, 2 months ago (2016-09-30 19:08:15 UTC) #9
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/2388453002/1
4 years, 2 months ago (2016-09-30 19:08:42 UTC) #11
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 2 months ago (2016-09-30 19:16:04 UTC) #13
commit-bot: I haz the power
4 years, 2 months ago (2016-09-30 19:19:02 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/90e3f912e7095ec556474e31771e6bafbe0c52f0
Cr-Commit-Position: refs/heads/master@{#422174}

Powered by Google App Engine
This is Rietveld 408576698