RESOLVED FIXED 56557
Web Inspector: Clean up Inspector strings 
https://bugs.webkit.org/show_bug.cgi?id=56557
Summary Web Inspector: Clean up Inspector strings
Mikhail Naganov
Reported 2011-03-17 06:05:39 PDT
The following unused strings have been removed: "Error in fetching root path for file system." "File System root path not available." "File System is disabled." "Persistent File System" "Reveal folder in OS" "Temporary File System" "File System" "Clear changes log." "Retaining paths of the selected object" "<source is not available>" "Show changes view." "%d style change" "Hide changes view." "Drag" "%d style changes" Also, I have modified the check-inspector-strings script to deal correctly with "\xNN" escapes, and to check strings in Chromium's DevTools.js.
Attachments
patch (4.80 KB, patch)
2011-03-17 06:09 PDT, Mikhail Naganov 		yurys: review+
mnaganov: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mikhail Naganov
Comment 1 2011-03-17 06:09:12 PDT
Created attachment 86051 [details] patch
WebKit Review Bot
Comment 2 2011-03-17 10:26:45 PDT
http://trac.webkit.org/changeset/81358 might have broken Leopard Intel Release (Tests) The following tests are not passing: http/tests/inspector/resource-parameters.html
Mikhail Naganov
Comment 3 2011-03-17 10:51:19 PDT
Manually committed: http://trac.webkit.org/changeset/81358 2011-03-17 Mikhail Naganov <mnaganov@chromium.org> Reviewed by Yury Semikhatsky. Web Inspector: Clean up Inspector strings. https://bugs.webkit.org/show_bug.cgi?id=56557 * English.lproj/localizedStrings.js: 2011-03-17 Mikhail Naganov <mnaganov@chromium.org> Reviewed by Yury Semikhatsky. Web Inspector: Clean up Inspector strings. https://bugs.webkit.org/show_bug.cgi?id=56557 Modify the script to deal correctly with "\xNN" escapes, and to check strings in Chromium's DevTools.js. * Scripts/check-inspector-strings:
Note You need to log in before you can comment on or make changes to this bug.