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

Issue 641970 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Suggested refactoring of handling of Blob schema in ResourceDispatcherHostImpl

Project Member Reported by rdsmith@chromium.org, Aug 29 2016

Issue description

There are several places in ResourceDispatcherHostImpl which call the function BlobProtocolHandler::SetRequestedBlobDataHandle: ContinuePendingBeginRequest(), BeginNavigationRequest(), BeginURLRequest().  However, all of these functions call the function BeginRequestInternal, and naively it would seem possible to move that call down into BeginRequestInternal for a small code cleanup.

(Not sure what component this belongs in, so dumping it into Internals>Network.  I'd add a blob component, but I can't find one.)



 

Comment 1 by mmenke@chromium.org, Aug 29 2016

Components: Blink>FileAPI
I think blobs fall under the FileAPI label?

Comment 2 Deleted

Comment 3 Deleted

Project Member

Comment 4 by sheriffbot@chromium.org, Aug 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 5 by mmenke@chromium.org, Aug 30 2017

Components: -Internals>Network
This is probably just WonFix - RDH is going away, someday.  Probably don't want to spend more time cleaning it up.
Status: WontFix (was: Untriaged)
cleanup work will fall out as we switch over to the network service
Components: Blink>Storage>FileAPI
Components: -Blink>FileAPI

Sign in to add a comment