Bug 148601

Summary: Web Inspector: Resource.prototype.associateWithScript assertions firing
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer
Priority: P2 Keywords: DoNotImportToRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix
none
[PATCH] Proposed Fix none

Description Joseph Pecoraro 2015-08-28 18:15:09 PDT 
* SUMMARY
Resource.prototype.associateWithScript assertions firing.

* STEPS TO REPRODUCE
1. Inspect <http://code.tutsplus.com/tutorials/say-hello-to-webkit-filters--net-23318>
2. Inspect inspector
3. Reload
  => lots of asserts

* NOTES
This assert is firing quite a bit:

  console.assert(this._type !=== Other);

I recently added it when dropping iOS 6 support since there was an iOS 6 compatibility comment. Seems it is not just for iOS 6 support.

Lets restore the old handling here.
Comment 1 Joseph Pecoraro 2015-08-28 18:16:18 PDT 
Created attachment 260199 [details]
[PATCH] Proposed Fix
Comment 2 Joseph Pecoraro 2015-08-28 19:15:09 PDT 
Created attachment 260202 [details]
[PATCH] Proposed Fix
Comment 3 WebKit Commit Bot 2015-08-28 20:30:50 PDT 
Comment on attachment 260202 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 260202

Committed r189141: <http://trac.webkit.org/changeset/189141>
Comment 4 WebKit Commit Bot 2015-08-28 20:30:54 PDT 
All reviewed patches have been landed.  Closing bug.