Bug 111165 – Web Inspector: Introduce opaque SecurityOrigin string identifiers in the frontend

• Component: Web Inspector (Deprecated)
• Status: RESOLVED WONTFIX

[Alexander Pavlov (apavlov)]

This task has been split out of bug 110960.

Patch to follow.

[Alexander Pavlov (apavlov)]

Created attachment 190963 [details]
Patch

[Vsevolod Vlasov]

Comment on attachment 190963 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=190963&action=review

> Source/WebCore/inspector/front-end/ExtensionServer.js:800
> +            var originObject = new WebInspector.SecurityOrigin(securityOrigin, securityOrigin);

Let's revert this change.

> Source/WebCore/inspector/front-end/ResourceTreeModel.js:771
> +    equal: function(other)

equals

> Source/WebCore/inspector/front-end/RuntimeModel.js:77
> +     * @param {WebInspector.SecurityOrigin} securityOrigin

RuntimeModel.js changes should be reverted.

[Alexander Pavlov (apavlov)]

Committed r144455: <http://trac.webkit.org/changeset/144455>

[Andrey Kosyakov]

Comment on attachment 190963 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=190963&action=review

> Source/WebCore/inspector/front-end/ExtensionAuditCategory.js:34
> + * @param {WebInspector.SecurityOrigin} extensionOrigin

This got landed. Please revert.

[Alexander Pavlov (apavlov)]

Reverted r144455 for reason:

A simpler solution to the SecurityOrigin -> localStorage lookup exists

Committed r144600: <http://trac.webkit.org/changeset/144600>

[Alexander Pavlov (apavlov)]

Closing, since bug 110960 is WONTFIX.