Make HttpStreamFactoryImpl::JobController::Start() return an net error code on sync error |
|||
Issue descriptionThis is to follow up on https://codereview.chromium.org/2814633003/. When JobController::Start() fails synchronously, we post task to notify the sync failure so that we can avoid re-entrancy. This is ugly. We should modify the method signature of JobController::Start() to reflect the fact that it can fail synchronously.
,
May 19 2017
Bence already has a CL in the works to make the signature more sane. He might be able to put in a fix for this one as well. (Thanks, Bence!)
,
Jul 6
,
Jul 6
|
|||
►
Sign in to add a comment |
|||
Comment 1 by b...@chromium.org
, May 19 2017