Web Inspector: Introduce SecurityOriginAdded and SecurityOriginRemoved events into ResourceTreeModel

[Alexander Pavlov (apavlov)]

This is necessary to keep track of all potential storages in the inspected page.

[Alexander Pavlov (apavlov)]

Created attachment 189265 [details]
Patch

[Yury Semikhatsky]

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 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 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)]

Committed r143443: <http://trac.webkit.org/changeset/143443>