fix: suppress the error if unexpected extraInfo events are detected - #7808
Conversation
|
Hey @josepharhar! Could you please take a look at this PR and the report #7805? I was able to reproduce the issue using the scenario from the report on Linux. It seems that a responseReceived event comes with hasExtraInfo = false but there is an extraInfo event in the cache: |
josepharhar
left a comment
There was a problem hiding this comment.
Would it make sense to remove the thrown error instead? I'm having a hard time understanding exactly what the added if (!extraInfo) does...
Also, instead of throwing an error, is there any other way we can signal that something bad happened so we can fix it in chromium?
|
we can swallow it and log using |
|
I have updated the CL. So do you think that chromium is sending wrong events here? |
|
If |
|
@josepharhar see my comment above #7808 (comment) it's hasExtraInfo=false but there is an responseReceivedExtraInfo event. Also, it seems to timeout if I ignore the error... |
|
Ah, sorry, my bad, correction: it's not timing out. |
Ah I see. Yeah I think the right thing to do is ignore the ExtraInfo event when
I'm guessing this means we're good to merge this |
Issues: #7805