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

Issue 671775 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

InterceptingResourceHandler doesn't send EOF to swapped out ResourceHandler

Project Member Reported by mmenke@chromium.org, Dec 6 2016

Issue description

In some cases, when InterceptingResourceHandler swaps out the old ResourceHandler with a new one, it sends a payload to the old handler.  When it does this, it doesn't send a 0-byte EOF OnReadCompleted to the old handler, unlike what we do with real successful requests.

Since the ResourceHandlers being swapped out don't care about this message don't care about the final 0-byte read, nothing bad happens, but it still seems problematic.

It would be nice if we could remove the 0-byte read notification in favor of just a success notification, but the MimeSniffingResourceHandler defers the request on that read to pass information about the read downstream.  We can't just remove the completion messages, because RedirectToFileHandler defers that.  Unclear if we can move them both onto deferring the same event, and get rid of the other one / make the other one non-deferrable.
 
Project Member

Comment 1 by sheriffbot@chromium.org, Dec 11 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
Cc: xunji...@chromium.org
mmenke@: Is this still applicable with the move to mojo loading?

Comment 3 by mmenke@chromium.org, Dec 12 2017

Status: WontFix (was: Untriaged)
No, it's not.

Sign in to add a comment