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

Issue 672129 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

OnBeforeSendHeaders*() NetworkDelegate upcall should be moved to HttpNetworkTransaction

Project Member Reported by tbansal@chromium.org, Dec 7 2016

Issue description

(Based on discussion with rdsmith and mmenke)

This would require (not necessarily in order):
(1) Moving the DRP logic from
 DRPNetworkDelegate::OnBeforeSendHeaders*() to DRPProxyDelegate.
(2) Moving OnBeforeSendHeaders*() from NetworkDelegate to net::ProxyDelegate.
(3) Moving the NotifyBeforeSendHeadersCallback() from UrlRequestHttpJob
 to HttpNetworkTransaction.

This refactoring would reduce NetworkDelegate upcall by one.
 
Labels: CleanUp
Cc: tbansal@chromium.org
Owner: ----
Status: Untriaged (was: Assigned)
Returning this back to Untriaged pool if somebody else has time to take this.

Comment 3 by bengr@chromium.org, Jan 9 2017

Status: Available (was: Untriaged)

Comment 4 by bengr@chromium.org, Nov 8 2017

Refreshed during triage.
Status: WontFix (was: Available)
This is probably not needed due to architectural changes from network servicification. Please feel free to reopen, if somebody feels otherwise.

Sign in to add a comment