Bug 89939 – Web Inspector: WebInspector.TextViewer should be renamed WebInspector.TextEditor

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

[Jan Keromnes]

The class `WebInspector.TextViewer` is obviously not a "viewer" anymore but an "editor". We should rename it to `WebInspector.TextEditor`.

[Jan Keromnes]

Created attachment 149645 [details]
Patch

[Early Warning System Bot]

Comment on attachment 149645 [details]
Patch

Attachment 149645 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/13096616

[Jan Keromnes]

Ossy: Looks like the patch might break the qt build. The trybot says "dependencies are not automatically re-computed for local developer builds", how can I re-compute them manually?

[Andrey Adaikin]

Also modify the WebCore.xcodeproj/project.pbxproj file.

[Csaba Osztrogonác]

(In reply to comment #3)
> Ossy: Looks like the patch might break the qt build. The trybot says "dependencies are not automatically re-computed for local developer builds", how can I re-compute them manually?

Fortunately it isn't as big problem as you can see now. Build will 
work fine on the bots, because they recompute dependency always. 

But it is an annoying bug, which causes false positive alarms on the EWS
sometimes. I filed a new bug report and try to fix it soon - https://bugs.webkit.org/show_bug.cgi?id=90049

Thanks for noticing this problem.

[Csaba Osztrogonác]

(In reply to comment #5)
FYI: The incremental build problem on the Qt EWS bots is fixed - https://bugs.webkit.org/show_bug.cgi?id=90049

[Jan Keromnes]

(In reply to comment #4)
> Also modify the WebCore.xcodeproj/project.pbxproj file.

This file doesn't seem to have any occurrences of either `TextViewer.js` or `textViewer.css`, so I guess Xcode builds by copying the entire `front-end` folder -- no need to modify it.

(In reply to comment #6)
> (In reply to comment #5)
> FYI: The incremental build problem on the Qt EWS bots is fixed - https://bugs.webkit.org/show_bug.cgi?id=90049

Great news! Thanks for fixing that so quickly.

[Pavel Feldman]

Comment on attachment 149645 [details]
Patch

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

I'd land this manually.

> Source/WebCore/inspector/front-end/TextEditor.js:1
> +/*

Your git status detected that it is just a rename, right? We don't want to lose nice annotation history.

[Jan Keromnes]

(In reply to comment #8)
> Your git status detected that it is just a rename, right? We don't want to lose nice annotation history.

> 20 files changed, 205 insertions(+), 120 deletions(-)
> rename Source/WebCore/inspector/front-end/{TextViewer.js => TextEditor.js} (96%)
> rename Source/WebCore/inspector/front-end/{textViewer.css => textEditor.css} (100%)

Git detects that it's just a rename, but I'm not sure about webkit-patch. Landing it manually sounds like a good idea.

[Pavel Feldman]

(In reply to comment #9)
> (In reply to comment #8)
> > Your git status detected that it is just a rename, right? We don't want to lose nice annotation history.
> 
> > 20 files changed, 205 insertions(+), 120 deletions(-)
> > rename Source/WebCore/inspector/front-end/{TextViewer.js => TextEditor.js} (96%)
> > rename Source/WebCore/inspector/front-end/{textViewer.css => textEditor.css} (100%)
> 
> Git detects that it's just a rename, but I'm not sure about webkit-patch. Landing it manually sounds like a good idea.

Could you rebase and upload it again? Vsevolod has added quite a bit of annotations and the patch does not apply well.

[Jan Keromnes]

Created attachment 150486 [details]
Patch

[Jan Keromnes]

(In reply to comment #10)
> Could you rebase and upload it again? Vsevolod has added quite a bit of annotations and the patch does not apply well.

The new patch fast-forwards from http://trac.webkit.org/changeset/121688.

I also tried to get webkit-patch to understand the file renames but your `git config diff.renames copies` trick didn't work for me. I guess you're going to have to do a lean diff again, sorry about that.

[Vsevolod Vlasov]

Committed r121746: <http://trac.webkit.org/changeset/121746>