RESOLVED FIXED110232
Web Inspector: Introduce SecurityOriginAdded and SecurityOriginRemoved events into ResourceTreeModel 
https://bugs.webkit.org/show_bug.cgi?id=110232
Summary Web Inspector: Introduce SecurityOriginAdded and SecurityOriginRemoved events...
Alexander Pavlov (apavlov)
Reported 2013-02-19 09:06:36 PST
This is necessary to keep track of all potential storages in the inspected page.
Attachments
Patch (8.63 KB, patch)
2013-02-20 01:40 PST, Alexander Pavlov (apavlov) 		vsevik: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexander Pavlov (apavlov)
Comment 1 2013-02-20 01:40:25 PST
Created attachment 189265 [details] Patch
Yury Semikhatsky
Comment 2 2013-02-20 02:00:37 PST
Comment on attachment 189265 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=189265&action=review > Source/WebCore/inspector/front-end/ResourceTreeModel.js:139 > + if (!this._securityOriginFrameCount[securityOrigin]) When does this happen? > Source/WebCore/inspector/front-end/ResourceTreeModel.js:160 > + removeOriginForFrame.call(this, frame.childFrames[i]); this.removeOriginForFrame(frame.childFrames[i]); ?
Vsevolod Vlasov
Comment 3 2013-02-20 02:13:51 PST
Comment on attachment 189265 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=189265&action=review > LayoutTests/http/tests/inspector/resource-tree/resource-tree-events-expected.txt:10 > + SecurityOriginAdded : http://frame/child2.html Let's add a test for navigation of already existing frame.
Vsevolod Vlasov
Comment 4 2013-02-20 02:19:21 PST
Comment on attachment 189265 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=189265&action=review >> Source/WebCore/inspector/front-end/ResourceTreeModel.js:139 >> + if (!this._securityOriginFrameCount[securityOrigin]) > > When does this happen? I would put console.assert instead.
Alexander Pavlov (apavlov)
Comment 5 2013-02-20 03:03:14 PST
Committed r143443: <http://trac.webkit.org/changeset/143443>
Note You need to log in before you can comment on or make changes to this bug.