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

Issue 647828 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Simplify QUIC server push stream canceling

Project Member Reported by rch@chromium.org, Sep 16 2016

Issue description

From comments on https://codereview.chromium.org/2345663002/

Of course, you don't have push_handle_ yet, but maybe something like this
refactor:

  o rename TryHandle to PushHandle
      o in addition to Cancel(), it also has Try()
Refactor the current QuicClientPushPromiseIndex::Try() so into two bits:
  o new method QuicClientPushPromiseIndex::Lookup() (returns PushHandle)
           o using this, found_promised_ could be consolidated into push_handle_
  o remainder of Try() logic goes into new PushHandle::Try() method

 

Sign in to add a comment