We never remove frames from ResourceTreeModel._frames. To test for that: - open inspector - navigate to a few sites - open inspector on inspector - check WebInspector.resourceTreeModel._frames -- it contains frames belonging to all sites visited.
Created attachment 122910 [details] Patch
Comment on attachment 122910 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=122910&action=review > Source/WebCore/inspector/front-end/ResourceTreeModel.js:199 > + _removeFrame: function(frame) This method should be defined in frame.
Created attachment 122917 [details] Patch
Comment on attachment 122917 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=122917&action=review > Source/WebCore/inspector/front-end/ResourceTreeModel.js:524 > + frame._remove(); I think this should be the last line in the method (after the childFrames.remove) > Source/WebCore/inspector/front-end/ResourceTreeModel.js:538 > + this._model.dispatchEventToListeners(WebInspector.ResourceTreeModel.EventTypes.FrameDetached, this); Also, dispatch as the last line.
Committed r105274: <http://trac.webkit.org/changeset/105274>