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

Issue 803913 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 817691
Owner:
Leaves the project on 2018/03/02
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Figure out how ResourceTimingInfo.FinalResponse().GetResourceLoadTiming() can be null

Project Member Reported by dcheng@chromium.org, Jan 19 2018

Issue description

I removed the if check when making this code work with OOPIFs and earned the #1 renderer crash: https://bugs.chromium.org/p/chromium/issues/detail?id=803811

There was originally a DCHECK in the code: https://chromium.googlesource.com/chromium/src/+/1593920eed56dee727e7f78ae5d206052e4ad7e0/third_party/WebKit/Source/core/timing/PerformanceBase.cpp#362

but it seems we don't actually have test coverage of this case. I'm not very familiar with the resource timing code, but it does seem like we should try to get coverage of this if possible.
 

Comment 1 by npm@chromium.org, Mar 1 2018

Mergedinto: 817691
Status: Duplicate (was: Assigned)

Sign in to add a comment