Bug 50029 – Web Inspector: place local modifications to under original resource nodes in the resource panel.

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

[Pavel Feldman]

Patch to follow.

[Pavel Feldman]

Created attachment 74769 [details]
[PATCH] Proposed change.

[Yury Semikhatsky]

Comment on attachment 74769 [details]
[PATCH] Proposed change.

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

> WebCore/inspector/front-end/Resource.js:639
> +    overrideContent: function(content)

This method should be renamed to setInitialContent

> WebCore/inspector/front-end/SourceView.js:151
> +        this.resource.content = newContent;

Move this line to _editLineComplete.

> WebCore/inspector/front-end/SourceView.js:155
> +    _editLineComplete: function(scrollTop)

scrollTop is not used.

[Pavel Feldman]

        M       WebCore/ChangeLog
        M       WebCore/English.lproj/localizedStrings.js
        M       WebCore/inspector/Inspector.idl
        M       WebCore/inspector/InspectorController.cpp
        M       WebCore/inspector/InspectorResourceAgent.cpp
        M       WebCore/inspector/InspectorResourceAgent.h
        M       WebCore/inspector/front-end/CSSStyleModel.js
        M       WebCore/inspector/front-end/NetworkPanel.js
        M       WebCore/inspector/front-end/Resource.js
        M       WebCore/inspector/front-end/ResourceManager.js
        M       WebCore/inspector/front-end/ResourcesPanel.js
        M       WebCore/inspector/front-end/SourceFrame.js
        M       WebCore/inspector/front-end/SourceView.js
        M       WebCore/inspector/front-end/StylesSidebarPane.js
Committed r72684

[WebKit Review Bot]

http://trac.webkit.org/changeset/72684 might have broken Leopard Intel Release (Tests)