var resourceForURL = this._resourcesByURL[resource.url];
this._resourcesByURL[resource.url] = resource;
else if (resourceForURL instanceof Array)
this._resourcesByURL[resource.url] = [resourceForURL]; <====
The last else clause converts a single element into an array, but it seems to have forgotten to put the resource into the newly formed array.
Created attachment 73105 [details]
Comment on attachment 73105 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=73105&action=review
Thanks for fixing this!
Fyi: there is a webkit.com/new-inspector-bug that has nice template for inspector bugs.
> + _bindResourceURL in ResourceManager.js may not add resource to map
You should use Web Inspector: prefix
> + var resourceForURLArray = [resourceForURL];
Nit: I would simply do
this._resourcesByURL[resource.url] = [resourceForURL, resource];
Created attachment 73246 [details]
I like your JS better. My JS-newbie-ness is showing...
Comment on attachment 73246 [details]
Clearing flags on attachment: 73246
Committed r71603: <http://trac.webkit.org/changeset/71603>
All reviewed patches have been landed. Closing bug.