New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 651957 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

GetTotalSentBytes() and GetTotalReceivedBytes() and in net::URLRequest do not track bytes from previous jobs

Project Member Reported by kundaji@chromium.org, Sep 30 2016

Issue description


The implementation of the above methods gets the bytes from the underlying |job_|. However, URLRequests can be associated with multiple jobs in its lifetime and data use by all previous jobs is being ignored.
 

Comment 1 by bengr@chromium.org, Oct 28 2016

Owner: sclit...@chromium.org
Status: Assigned (was: Available)
This sounds significant. sclittle, ptal!
Cc: bengr@chromium.org kundaji@chromium.org sclit...@chromium.org
Owner: ----
Status: Available (was: Assigned)
In practice, IIUC this doesn't lead to a significant number of missed bytes.

The missed bytes here mainly consist of stuff like redirects, e.g. if a request leads to a redirect which in turn leads to a resource, then GetTotalSent/ReceivedBytes() would only count the bytes from the resource load and wouldn't include the size of the headers and stuff from the redirect.

@kundaji: Are you still planning to fix this?
Project Member

Comment 3 by sheriffbot@chromium.org, Oct 30 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

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

Comment 4 by bengr@chromium.org, Dec 1 2017

Status: WontFix (was: Untriaged)

Sign in to add a comment