Bug 52196
| Summary: | Web Inspector: Some Localized Strings are Outdated | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> |
| Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
Joseph Pecoraro
No-one reviewed this and I had some spare time. I did a quick check to verify that
the strings listed in localizedStrings.js are still valid. Here is what I found:
No longer seem to be used:
"%s (%s transferred)"
"Enable resource tracking"
"Enabling resource tracking will reload the page and make page loading slower."
"Resource tracking disabled. Click to enable."
"Resource tracking enabled. Click to disable."
"You need to enable resource tracking to use this panel."
"GRAPHS"
"Prototype"
"RESOURCES"
"Recording…"
"Sort by Duration"
"Sort by End Time"
"Sort by Latency"
"Sort by Response Time"
"Sort by Size"
"Sort by Start Time"
"Sort by Transfer Size"
"Stop on Attributes Modifications"
"Stop on Node Removal"
"Stop on Subtree Modifications"
"Any Request"
"Set XHR Breakpoint"
"Timing."
Might be used? Its a key in WebInspector.Resource.Type
"Media"
Used, but is different in Source Code:
"Assertion failed: "
It seems unnecessarily difficult to keep localizedStrings.js up to date with the code.
Hopefully we can come up with a solution for improving this file's maintance, other
than relying on programmers / reviewers catching a slip (failing to delete, or add)
a string at review time.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Joseph Pecoraro
*** This bug has been marked as a duplicate of bug 52295 ***