Web Inspector: saving JavaScript after live edit resets scroller position.

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

[Pavel Feldman]

1. Start editing
2. Hit Ctrl+Save (Cmd+Save)

scrol top is set to 0

[Pavel Podivilov]

Created attachment 109971 [details]
Patch

[Yury Semikhatsky]

Comment on attachment 109971 [details]
Patch

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

> Source/WebCore/inspector/front-end/UISourceCode.js:81
> +    contentChanged: function()

Looking at the current implementation, it seems that we can pass new content as a parameter here and avoid additional requestContent later.

> Source/WebCore/inspector/front-end/UISourceCode.js:84
> +        this._contentLoaded = false;

this._content should also be cleared

[Pavel Podivilov]

Committed r97056: <http://trac.webkit.org/changeset/97056>

[Csaba Osztrogonác]

It broke inspector/debugger/script-formatter.html.

On the SL bot: http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28Tests%29/r97056%20%2833778%29/inspector/debugger/script-formatter-pretty-diff.html

On the Qt bot: http://build.webkit.org/results/Qt%20Linux%20Release/r97056%20%2838357%29/inspector/debugger/script-formatter-pretty-diff.html

Reopen to fix it.

[Csaba Osztrogonác]

Comment on attachment 109971 [details]
Patch

remove r+ from landed patcg

[Csaba Osztrogonác]

Fix landed in http://trac.webkit.org/changeset/97060

[Pavel Feldman]

*** Bug 69842 has been marked as a duplicate of this bug. ***