Unreviewed, rolling out r244952.
authorryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 May 2019 17:44:47 +0000 (17:44 +0000)
committerryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 May 2019 17:44:47 +0000 (17:44 +0000)
Caused inspector to appear blank.

Reverted changeset:

"Web Inspector: Provide UIString descriptions to improve
localizations"
https://bugs.webkit.org/show_bug.cgi?id=195132
https://trac.webkit.org/changeset/244952

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@245060 268f45cc-cd09-0410-ab3c-d52691b4dbfc

20 files changed:
Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js
Source/WebInspectorUI/UserInterface/Base/LoadLocalizedStrings.js
Source/WebInspectorUI/UserInterface/Models/LayoutTimelineRecord.js
Source/WebInspectorUI/UserInterface/Models/RenderingFrameTimelineRecord.js
Source/WebInspectorUI/UserInterface/Models/Resource.js
Source/WebInspectorUI/UserInterface/Views/AuditTestGroupContentView.js
Source/WebInspectorUI/UserInterface/Views/CPUTimelineView.js
Source/WebInspectorUI/UserInterface/Views/ContextMenuUtilities.js
Source/WebInspectorUI/UserInterface/Views/DOMBreakpointTreeElement.js
Source/WebInspectorUI/UserInterface/Views/DOMNodeTreeElement.js
Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js
Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js
Source/WebInspectorUI/UserInterface/Views/LayerTreeDetailsSidebarPanel.js
Source/WebInspectorUI/UserInterface/Views/NetworkTableContentView.js
Source/WebInspectorUI/UserInterface/Views/SourcesNavigationSidebarPanel.js
Source/WebInspectorUI/UserInterface/Views/SpreadsheetCSSStyleDeclarationSection.js
Source/WebInspectorUI/UserInterface/Views/SpreadsheetRulesStyleDetailsPanel.js
Tools/ChangeLog
Tools/Scripts/extract-localizable-js-strings

index 302d1be..8c3dafb 100644 (file)
@@ -1,3 +1,16 @@
+2019-05-08  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r244952.
+
+        Caused inspector to appear blank.
+
+        Reverted changeset:
+
+        "Web Inspector: Provide UIString descriptions to improve
+        localizations"
+        https://bugs.webkit.org/show_bug.cgi?id=195132
+        https://trac.webkit.org/changeset/244952
+
 2019-05-04  Nikita Vasilyev  <nvasilyev@apple.com>
 
         Web Inspector: Provide UIString descriptions to improve localizations
index f97c188..5046616 100644 (file)
@@ -97,10 +97,8 @@ localizedStrings["Additions"] = "Additions";
 localizedStrings["Address"] = "Address";
 localizedStrings["All"] = "All";
 localizedStrings["All Changes"] = "All Changes";
-/* Break (pause) on all exceptions */
 localizedStrings["All Exceptions"] = "All Exceptions";
 localizedStrings["All Layers"] = "All Layers";
-/* A submenu item of 'Break on' that breaks (pauses) before all network requests */
 localizedStrings["All Requests"] = "All Requests";
 localizedStrings["All Resources"] = "All Resources";
 localizedStrings["All Storage"] = "All Storage";
@@ -129,14 +127,12 @@ localizedStrings["Area"] = "Area";
 localizedStrings["Assertion"] = "Assertion";
 localizedStrings["Assertion Failed"] = "Assertion Failed";
 localizedStrings["Assertion Failed: %s"] = "Assertion Failed: %s";
-/* Break (pause) when console.assert() fails */
 localizedStrings["Assertion Failures"] = "Assertion Failures";
 localizedStrings["Assertion with message: %s"] = "Assertion with message: %s";
 localizedStrings["Assertive"] = "Assertive";
 localizedStrings["Associated Data"] = "Associated Data";
 localizedStrings["Async audits are not supported."] = "Async audits are not supported.";
 localizedStrings["Attribute"] = "Attribute";
-/* A submenu item of 'Break On' that breaks (pauses) before DOM attribute is modified */
 localizedStrings["Attribute Modified"] = "Attribute Modified";
 localizedStrings["Attributes"] = "Attributes";
 localizedStrings["Audit"] = "Audit";
@@ -200,7 +196,6 @@ localizedStrings["Canvas %d"] = "Canvas %d";
 localizedStrings["Canvas %s"] = "Canvas %s";
 localizedStrings["Canvas Element"] = "Canvas Element";
 localizedStrings["Canvases"] = "Canvases";
-/* Capture screenshot of the selected DOM node */
 localizedStrings["Capture Screenshot"] = "Capture Screenshot";
 localizedStrings["Capturing"] = "Capturing";
 /* Context menu label for whether searches should be case sensitive. */
@@ -214,7 +209,6 @@ localizedStrings["Changes"] = "Changes";
 localizedStrings["Character Data"] = "Character Data";
 localizedStrings["Charge \u201C%s\u201D to Callers"] = "Charge \u201C%s\u201D to Callers";
 localizedStrings["Checked"] = "Checked";
-/* A submenu item of 'Add' to append DOM nodes to the selected DOM node */
 localizedStrings["Child"] = "Child";
 localizedStrings["Child Layers"] = "Child Layers";
 localizedStrings["Child added to "] = "Child added to ";
@@ -249,8 +243,7 @@ localizedStrings["Collect garbage"] = "Collect garbage";
 localizedStrings["Comment"] = "Comment";
 localizedStrings["Compare snapshots"] = "Compare snapshots";
 localizedStrings["Comparison of total memory size at the end of the selected time range to the maximum memory size in this recording"] = "Comparison of total memory size at the end of the selected time range to the maximum memory size in this recording";
-/* Composite phase timeline records, where graphic layers are combined */
-localizedStrings["Composite @ Timeline record"] = "Composite";
+localizedStrings["Composite"] = "Composite";
 localizedStrings["Composited"] = "Composited";
 localizedStrings["Compressed"] = "Compressed";
 localizedStrings["Compression"] = "Compression";
@@ -470,9 +463,7 @@ localizedStrings["Extra Scripts"] = "Extra Scripts";
 localizedStrings["Fade unexecuted code"] = "Fade unexecuted code";
 localizedStrings["Failed to upgrade"] = "Failed to upgrade";
 localizedStrings["Failure status code"] = "Failure status code";
-/* Resource loaded via 'fetch' method */
 localizedStrings["Fetch"] = "Fetch";
-/* Resources loaded via 'fetch' method */
 localizedStrings["Fetches"] = "Fetches";
 localizedStrings["File or Resource"] = "File or Resource";
 localizedStrings["Filename"] = "Filename";
@@ -489,9 +480,7 @@ localizedStrings["Fonts"] = "Fonts";
 localizedStrings["Force Dark Appearance"] = "Force Dark Appearance";
 localizedStrings["Force Light Appearance"] = "Force Light Appearance";
 localizedStrings["Force Print Media Styles"] = "Force Print Media Styles";
-/* Layout phase records that were imperative (forced) */
 localizedStrings["Forced Layout"] = "Forced Layout";
-/* A context menu item to force (override) a DOM node's pseudo-classes */
 localizedStrings["Forced Pseudo-Classes"] = "Forced Pseudo-Classes";
 localizedStrings["Format: HSL"] = "Format: HSL";
 localizedStrings["Format: HSLA"] = "Format: HSLA";
@@ -581,7 +570,6 @@ localizedStrings["Index Key \u2014 %s"] = "Index Key \u2014 %s";
 localizedStrings["Indexed Databases"] = "Indexed Databases";
 localizedStrings["Info: "] = "Info: ";
 localizedStrings["Infos"] = "Infos";
-/* A section of CSS rules matching an ancestor DOM node */
 localizedStrings["Inherited From"] = "Inherited From";
 localizedStrings["Initial State"] = "Initial State";
 localizedStrings["Initial Velocity"] = "Initial Velocity";
@@ -610,9 +598,8 @@ localizedStrings["Layer Count: %d"] = "Layer Count: %d";
 localizedStrings["Layer Info"] = "Layer Info";
 localizedStrings["Layers"] = "Layers";
 localizedStrings["Layers:"] = "Layers:";
+localizedStrings["Layout"] = "Layout";
 localizedStrings["Layout & Rendering"] = "Layout & Rendering";
-/* Layout phase timeline records */
-localizedStrings["Layout @ Timeline record"] = "Layout";
 localizedStrings["Layout Invalidated"] = "Layout Invalidated";
 localizedStrings["Line %d"] = "Line %d";
 localizedStrings["Line %d:%d"] = "Line %d:%d";
@@ -630,12 +617,10 @@ localizedStrings["Local Variables"] = "Local Variables";
 localizedStrings["Located at %s"] = "Located at %s";
 localizedStrings["Location"] = "Location";
 localizedStrings["Log Canvas Context"] = "Log Canvas Context";
-/* Log (print) DOM element to Console */
 localizedStrings["Log Element"] = "Log Element";
 localizedStrings["Log Frame Text"] = "Log Frame Text";
 localizedStrings["Log Frame Value"] = "Log Frame Value";
 localizedStrings["Log Message"] = "Log Message";
-/* Log (print) DOM node to Console */
 localizedStrings["Log Node"] = "Log Node";
 localizedStrings["Log Symbol"] = "Log Symbol";
 localizedStrings["Log Value"] = "Log Value";
@@ -679,7 +664,6 @@ localizedStrings["Network Issue"] = "Network Issue";
 localizedStrings["Network Requests"] = "Network Requests";
 localizedStrings["Network Requests:"] = "Network Requests:";
 localizedStrings["New Tab"] = "New Tab";
-/* A submenu item of 'Add' to add DOM nodes after the selected DOM node */
 localizedStrings["Next Sibling"] = "Next Sibling";
 localizedStrings["No"] = "No";
 localizedStrings["No Accessibility Information"] = "No Accessibility Information";
@@ -720,7 +704,6 @@ localizedStrings["No request, served from the memory cache."] = "No request, ser
 localizedStrings["No response cookies."] = "No response cookies.";
 localizedStrings["No response headers"] = "No response headers";
 localizedStrings["Node"] = "Node";
-/* A submenu item of 'Break On' that breaks (pauses) before DOM node is removed */
 localizedStrings["Node Removed"] = "Node Removed";
 localizedStrings["Nodes"] = "Nodes";
 localizedStrings["None"] = "None";
@@ -760,11 +743,8 @@ localizedStrings["Page"] = "Page";
 localizedStrings["Page Issue"] = "Page Issue";
 localizedStrings["Page navigated at %s"] = "Page navigated at %s";
 localizedStrings["Page reloaded at %s"] = "Page reloaded at %s";
-/* Paint (render) phase timeline records */
-localizedStrings["Paint @ Timeline record"] = "Paint";
+localizedStrings["Paint"] = "Paint";
 localizedStrings["Paints"] = "Paints";
-/* A count of how many times an element was painted (rendered) */
-localizedStrings["Paints @ Column title"] = "Paints";
 localizedStrings["Parent"] = "Parent";
 localizedStrings["Partial Garbage Collection"] = "Partial Garbage Collection";
 localizedStrings["Passive"] = "Passive";
@@ -794,7 +774,6 @@ localizedStrings["Press %s to stop editing"] = "Press %s to stop editing";
 localizedStrings["Pressed"] = "Pressed";
 localizedStrings["Pretty print"] = "Pretty print";
 localizedStrings["Preview"] = "Preview";
-/* A submenu item of 'Add' to add DOM nodes before the selected DOM node */
 localizedStrings["Previous Sibling"] = "Previous Sibling";
 localizedStrings["Primary Key"] = "Primary Key";
 localizedStrings["Primary Key \u2014 %s"] = "Primary Key \u2014 %s";
@@ -882,11 +861,9 @@ localizedStrings["Return type for anonymous function"] = "Return type for anonym
 localizedStrings["Return type for function: %s"] = "Return type for function: %s";
 localizedStrings["Return value is not an object, string, or boolean"] = "Return value is not an object, string, or boolean";
 localizedStrings["Reveal Breakpoint"] = "Reveal Breakpoint";
-/* Open Elements tab and select this node in DOM tree */
 localizedStrings["Reveal in DOM Tree"] = "Reveal in DOM Tree";
 localizedStrings["Reveal in Debugger Tab"] = "Reveal in Debugger Tab";
 localizedStrings["Reveal in Elements Tab"] = "Reveal in Elements Tab";
-/* Open Layers tab and select the layer corresponding to this node */
 localizedStrings["Reveal in Layers Tab"] = "Reveal in Layers Tab";
 localizedStrings["Reveal in Network Tab"] = "Reveal in Network Tab";
 localizedStrings["Reveal in Original Resource"] = "Reveal in Original Resource";
@@ -915,8 +892,7 @@ localizedStrings["Script Element %d"] = "Script Element %d";
 localizedStrings["Script Entries:"] = "Script Entries:";
 localizedStrings["Script Evaluated"] = "Script Evaluated";
 localizedStrings["Scripts"] = "Scripts";
-/* Scroll selected DOM node into view on the inspected web page */
-localizedStrings["Scroll into View"] = "Scroll into View";
+localizedStrings["Scroll Into View"] = "Scroll Into View";
 localizedStrings["Search"] = "Search";
 localizedStrings["Search Again"] = "Search Again";
 localizedStrings["Search Resource Content"] = "Search Resource Content";
@@ -931,12 +907,10 @@ localizedStrings["Select baseline snapshot"] = "Select baseline snapshot";
 localizedStrings["Select comparison snapshot"] = "Select comparison snapshot";
 localizedStrings["Selected"] = "Selected";
 localizedStrings["Selected Canvas Context"] = "Selected Canvas Context";
-/* Selected DOM element */
 localizedStrings["Selected Element"] = "Selected Element";
 localizedStrings["Selected Frame"] = "Selected Frame";
 localizedStrings["Selected Item"] = "Selected Item";
 localizedStrings["Selected Items"] = "Selected Items";
-/* Selected DOM node */
 localizedStrings["Selected Node"] = "Selected Node";
 localizedStrings["Selected Symbol"] = "Selected Symbol";
 localizedStrings["Selected Value"] = "Selected Value";
@@ -1037,7 +1011,6 @@ localizedStrings["Stop recording canvas actions"] = "Stop recording canvas actio
 localizedStrings["Stop recording once page loads"] = "Stop recording once page loads";
 localizedStrings["Stopping the \u201C%s\u201D audit"] = "Stopping the \u201C%s\u201D audit";
 localizedStrings["Storage"] = "Storage";
-/* CSS properties defined via HTML style attribute */
 localizedStrings["Style Attribute"] = "Style Attribute";
 localizedStrings["Style Resolution"] = "Style Resolution";
 localizedStrings["Style rule"] = "Style rule";
@@ -1050,14 +1023,12 @@ localizedStrings["Styles:"] = "Styles:";
 localizedStrings["Stylesheet"] = "Stylesheet";
 localizedStrings["Stylesheets"] = "Stylesheets";
 localizedStrings["Subject"] = "Subject";
-/* A submenu item of 'Break On' that breaks (pauses) before child DOM node is modified */
 localizedStrings["Subtree Modified"] = "Subtree Modified";
 localizedStrings["Summary"] = "Summary";
 localizedStrings["TCP"] = "TCP";
 localizedStrings["TIFF"] = "TIFF";
 localizedStrings["Tab width:"] = "Tab width:";
 localizedStrings["Tabs"] = "Tabs";
-/* A submenu item of 'Edit' to change DOM element's tag name */
 localizedStrings["Tag"] = "Tag";
 localizedStrings["Take snapshot"] = "Take snapshot";
 localizedStrings["Target"] = "Target";
@@ -1120,7 +1091,6 @@ localizedStrings["Total: %s"] = "Total: %s";
 localizedStrings["Totals:"] = "Totals:";
 localizedStrings["Trace"] = "Trace";
 localizedStrings["Trace: %s"] = "Trace: %s";
-/* Amount of data sent over the network for a single resource */
 localizedStrings["Transfer Size"] = "Transfer Size";
 localizedStrings["Transferred"] = "Transferred";
 localizedStrings["Triggered Breakpoint"] = "Triggered Breakpoint";
@@ -1135,7 +1105,6 @@ localizedStrings["URL Breakpoint\u2026"] = "URL Breakpoint\u2026";
 localizedStrings["Unable to determine path to property from root"] = "Unable to determine path to property from root";
 localizedStrings["Unable to parse as JSON: %s"] = "Unable to parse as JSON: %s";
 localizedStrings["Unable to show certificate for \u201C%s\u201D"] = "Unable to show certificate for \u201C%s\u201D";
-/* Break (pause) on uncaught (unhandled) exceptions */
 localizedStrings["Uncaught Exceptions"] = "Uncaught Exceptions";
 localizedStrings["Undefined custom element"] = "Undefined custom element";
 localizedStrings["Unique"] = "Unique";
index e20d506..c0f5a37 100644 (file)
@@ -43,17 +43,9 @@ WI.unlocalizedString = function(string)
 
 WI.UIString = function(string, key, comment)
 {
-    "use strict";
-
     if (WI.dontLocalizeUserInterface)
         return string;
 
-    // UIString(string, comment)
-    if (arguments.length === 2) {
-        comment = key;
-        key = undefined;
-    }
-
     key = key || string;
 
     if (window.localizedStrings && key in window.localizedStrings)
@@ -72,41 +64,3 @@ WI.UIString = function(string, key, comment)
 
     return "LOCALIZED STRING NOT FOUND";
 };
-
-WI.repeatedUIString = {};
-
-WI.repeatedUIString.timelineRecordLayout = function() {
-    return WI.UIString("Layout", "Layout @ Timeline record", "Layout phase timeline records");
-};
-
-WI.repeatedUIString.timelineRecordPaint = function() {
-    return WI.UIString("Paint", "Paint @ Timeline record", "Paint (render) phase timeline records");
-};
-
-WI.repeatedUIString.timelineRecordComposite = function() {
-    return WI.UIString("Composite", "Composite @ Timeline record", "Composite phase timeline records, where graphic layers are combined");
-};
-
-WI.repeatedUIString.allExceptions = function() {
-    return WI.UIString("All Exceptions", "Break (pause) on all exceptions");
-};
-
-WI.repeatedUIString.uncaughtExceptions = function() {
-    return WI.UIString("Uncaught Exceptions", "Break (pause) on uncaught (unhandled) exceptions");
-};
-
-WI.repeatedUIString.assertionFailures = function() {
-    return WI.UIString("Assertion Failures", "Break (pause) when console.assert() fails");
-};
-
-WI.repeatedUIString.allRequests = function() {
-    return WI.UIString("All Requests", "A submenu item of 'Break on' that breaks (pauses) before all network requests");
-};
-
-WI.repeatedUIString.fetch = function() {
-    return WI.UIString("Fetch", "Resource loaded via 'fetch' method");
-};
-
-WI.repeatedUIString.revealInDOMTree = function() {
-    return WI.UIString("Reveal in DOM Tree", "Open Elements tab and select this node in DOM tree");
-};
index 4a831f1..faa4b6c 100644 (file)
@@ -51,13 +51,13 @@ WI.LayoutTimelineRecord = class LayoutTimelineRecord extends WI.TimelineRecord
         case WI.LayoutTimelineRecord.EventType.InvalidateLayout:
             return WI.UIString("Layout Invalidated");
         case WI.LayoutTimelineRecord.EventType.ForcedLayout:
-            return WI.UIString("Forced Layout", "Layout phase records that were imperative (forced)");
+            return WI.UIString("Forced Layout");
         case WI.LayoutTimelineRecord.EventType.Layout:
-            return WI.repeatedUIString.timelineRecordLayout();
+            return WI.UIString("Layout");
         case WI.LayoutTimelineRecord.EventType.Paint:
-            return WI.repeatedUIString.timelineRecordPaint();
+            return WI.UIString("Paint");
         case WI.LayoutTimelineRecord.EventType.Composite:
-            return WI.repeatedUIString.timelineRecordComposite();
+            return WI.UIString("Composite");
         }
     }
 
index cc2e1b2..5fd8108 100644 (file)
@@ -46,9 +46,9 @@ WI.RenderingFrameTimelineRecord = class RenderingFrameTimelineRecord extends WI.
         case WI.RenderingFrameTimelineRecord.TaskType.Script:
             return WI.UIString("Script");
         case WI.RenderingFrameTimelineRecord.TaskType.Layout:
-            return WI.repeatedUIString.timelineRecordLayout();
+            return WI.UIString("Layout");
         case WI.RenderingFrameTimelineRecord.TaskType.Paint:
-            return WI.repeatedUIString.timelineRecordPaint();
+            return WI.UIString("Paint");
         case WI.RenderingFrameTimelineRecord.TaskType.Other:
             return WI.UIString("Other");
         }
index e931c12..dc75208 100644 (file)
@@ -156,8 +156,8 @@ WI.Resource = class Resource extends WI.SourceCode
             return WI.UIString("XHR");
         case WI.Resource.Type.Fetch:
             if (plural)
-                return WI.UIString("Fetches", "Resources loaded via 'fetch' method");
-            return WI.repeatedUIString.fetch();
+                return WI.UIString("Fetches");
+            return WI.UIString("Fetch");
         case WI.Resource.Type.Ping:
             if (plural)
                 return WI.UIString("Pings");
index be2eeba..8e40dd6 100644 (file)
@@ -125,11 +125,11 @@ WI.AuditTestGroupContentView = class AuditTestGroupContentView extends WI.AuditT
                 scopeBarItems.push(scopeBarItem);
             };
 
-            addScopeBarItem(WI.AuditTestCaseResult.Level.Pass, WI.UIString("%d Passed", "%d Passed (singular)", ""), WI.UIString("%d Passed", "%d Passed (plural)", ""));
+            addScopeBarItem(WI.AuditTestCaseResult.Level.Pass, WI.UIString("%d Passed", "%d Passed (singular)"), WI.UIString("%d Passed", "%d Passed (plural)"));
             addScopeBarItem(WI.AuditTestCaseResult.Level.Warn, WI.UIString("%d Warning"), WI.UIString("%d Warnings"));
-            addScopeBarItem(WI.AuditTestCaseResult.Level.Fail, WI.UIString("%d Failed", "%d Failed (singular)", ""), WI.UIString("%d Failed", "%d Failed (plural)", ""));
+            addScopeBarItem(WI.AuditTestCaseResult.Level.Fail, WI.UIString("%d Failed", "%d Failed (singular)"), WI.UIString("%d Failed", "%d Failed (plural)"));
             addScopeBarItem(WI.AuditTestCaseResult.Level.Error, WI.UIString("%d Error"), WI.UIString("%d Errors"));
-            addScopeBarItem(WI.AuditTestCaseResult.Level.Unsupported, WI.UIString("%d Unsupported", "%d Unsupported (singular)", ""), WI.UIString("%d Unsupported", "%d Unsupported (plural)", ""));
+            addScopeBarItem(WI.AuditTestCaseResult.Level.Unsupported, WI.UIString("%d Unsupported", "%d Unsupported (singular)"), WI.UIString("%d Unsupported", "%d Unsupported (plural)"));
 
             this._levelScopeBar = new WI.ScopeBar(null, scopeBarItems);
             this._levelScopeBar.addEventListener(WI.ScopeBar.Event.SelectionChanged, this._handleLevelScopeBarSelectionChanged, this);
index 17ad5ad..1513d4e 100644 (file)
@@ -57,9 +57,9 @@ WI.CPUTimelineView = class CPUTimelineView extends WI.TimelineView
         case WI.CPUTimelineView.SampleType.Script:
             return WI.UIString("Script");
         case WI.CPUTimelineView.SampleType.Layout:
-            return WI.repeatedUIString.timelineRecordLayout();
+            return WI.UIString("Layout");
         case WI.CPUTimelineView.SampleType.Paint:
-            return WI.repeatedUIString.timelineRecordPaint();
+            return WI.UIString("Paint");
         case WI.CPUTimelineView.SampleType.Style:
             return WI.UIString("Style Resolution");
         }
index 3672d47..e1b126f 100644 (file)
@@ -216,10 +216,10 @@ WI.appendContextMenuItemsForDOMNode = function(contextMenu, domNode, options = {
     contextMenu.appendSeparator();
 
     if (!options.excludeLogElement && !domNode.isInUserAgentShadowTree() && !domNode.isPseudoElement()) {
-        let label = isElement ? WI.UIString("Log Element", "Log (print) DOM element to Console") : WI.UIString("Log Node", "Log (print) DOM node to Console");
+        let label = isElement ? WI.UIString("Log Element") : WI.UIString("Log Node");
         contextMenu.appendItem(label, () => {
             WI.RemoteObject.resolveNode(domNode, WI.RuntimeManager.ConsoleObjectGroup).then((remoteObject) => {
-                let text = isElement ? WI.UIString("Selected Element", "Selected DOM element") : WI.UIString("Selected Node", "Selected DOM node");
+                let text = isElement ? WI.UIString("Selected Element") : WI.UIString("Selected Node");
                 const addSpecialUserLogClass = true;
                 WI.consoleLogViewController.appendImmediateExecutionWithResult(text, remoteObject, addSpecialUserLogClass);
             });
@@ -227,19 +227,19 @@ WI.appendContextMenuItemsForDOMNode = function(contextMenu, domNode, options = {
     }
 
     if (!options.excludeRevealElement && window.DOMAgent && attached) {
-        contextMenu.appendItem(WI.repeatedUIString.revealInDOMTree(), () => {
+        contextMenu.appendItem(WI.UIString("Reveal in DOM Tree"), () => {
             WI.domManager.inspectElement(domNode.id);
         });
     }
 
     if (WI.settings.experimentalEnableLayersTab.value && window.LayerTreeAgent && attached) {
-        contextMenu.appendItem(WI.UIString("Reveal in Layers Tab", "Open Layers tab and select the layer corresponding to this node"), () => {
+        contextMenu.appendItem(WI.UIString("Reveal in Layers Tab"), () => {
             WI.showLayersTab({nodeToSelect: domNode});
         });
     }
 
     if (window.PageAgent && attached) {
-        contextMenu.appendItem(WI.UIString("Capture Screenshot", "Capture screenshot of the selected DOM node"), () => {
+        contextMenu.appendItem(WI.UIString("Capture Screenshot"), () => {
             PageAgent.snapshotNode(domNode.id, (error, dataURL) => {
                 if (error) {
                     const target = WI.mainTarget;
@@ -262,7 +262,7 @@ WI.appendContextMenuItemsForDOMNode = function(contextMenu, domNode, options = {
     }
 
     if (isElement && attached) {
-        contextMenu.appendItem(WI.UIString("Scroll into View", "Scroll selected DOM node into view on the inspected web page"), () => {
+        contextMenu.appendItem(WI.UIString("Scroll Into View"), () => {
             domNode.scrollIntoView();
         });
     }
index 991d716..667d557 100644 (file)
@@ -50,11 +50,11 @@ WI.DOMBreakpointTreeElement = class DOMBreakpointTreeElement extends WI.GeneralT
     {
         switch (type) {
         case WI.DOMBreakpoint.Type.SubtreeModified:
-            return WI.UIString("Subtree Modified", "A submenu item of 'Break On' that breaks (pauses) before child DOM node is modified");
+            return WI.UIString("Subtree Modified");
         case WI.DOMBreakpoint.Type.AttributeModified:
-            return WI.UIString("Attribute Modified", "A submenu item of 'Break On' that breaks (pauses) before DOM attribute is modified");
+            return WI.UIString("Attribute Modified");
         case WI.DOMBreakpoint.Type.NodeRemoved:
-            return WI.UIString("Node Removed", "A submenu item of 'Break On' that breaks (pauses) before DOM node is removed");
+            return WI.UIString("Node Removed");
         default:
             console.error("Unexpected DOM breakpoint type: " + type);
             return null;
index adaff3f..1f5d171 100644 (file)
@@ -61,7 +61,7 @@ WI.DOMNodeTreeElement = class DOMNodeTreeElement extends WI.GeneralTreeElement
 
         contextMenu.appendSeparator();
 
-        contextMenu.appendItem(WI.repeatedUIString.revealInDOMTree(), () => {
+        contextMenu.appendItem(WI.UIString("Reveal in DOM Tree"), () => {
             WI.domManager.inspectElement(this.representedObject.id);
         });
     }
index 2a68a0b..0be9b08 100644 (file)
@@ -777,14 +777,14 @@ WI.DOMTreeElement = class DOMTreeElement extends WI.TreeElement
             node.removeAttribute(attributeName);
         }, !attribute || !isNonShadowEditable);
 
-        subMenus.edit.appendItem(WI.UIString("Tag", "A submenu item of 'Edit' to change DOM element's tag name"), () => {
+        subMenus.edit.appendItem(WI.UIString("Tag"), () => {
             this._startEditingTagName();
         }, !isNonShadowEditable);
 
         contextMenu.appendSeparator();
 
         if (WI.cssManager.canForcePseudoClasses() && attached) {
-            let pseudoSubMenu = contextMenu.appendSubMenuItem(WI.UIString("Forced Pseudo-Classes", "A context menu item to force (override) a DOM node's pseudo-classes"));
+            let pseudoSubMenu = contextMenu.appendSubMenuItem(WI.UIString("Forced Pseudo-Classes"));
 
             let enabledPseudoClasses = node.enabledPseudoClasses;
             WI.CSSManager.ForceablePseudoClasses.forEach((pseudoClass) => {
@@ -815,9 +815,9 @@ WI.DOMTreeElement = class DOMTreeElement extends WI.TreeElement
 
         let isEditableNode = node.nodeType() === Node.ELEMENT_NODE && this.editable;
         let forbiddenClosingTag = WI.DOMTreeElement.ForbiddenClosingTagElements.has(node.nodeNameInCorrectCase());
-        subMenus.add.appendItem(WI.UIString("Child", "A submenu item of 'Add' to append DOM nodes to the selected DOM node"), this._addHTML.bind(this), forbiddenClosingTag || !isEditableNode);
-        subMenus.add.appendItem(WI.UIString("Previous Sibling", "A submenu item of 'Add' to add DOM nodes before the selected DOM node"), this._addPreviousSibling.bind(this), !isEditableNode);
-        subMenus.add.appendItem(WI.UIString("Next Sibling", "A submenu item of 'Add' to add DOM nodes after the selected DOM node"), this._addNextSibling.bind(this), !isEditableNode);
+        subMenus.add.appendItem(WI.UIString("Child"), this._addHTML.bind(this), forbiddenClosingTag || !isEditableNode);
+        subMenus.add.appendItem(WI.UIString("Previous Sibling"), this._addPreviousSibling.bind(this), !isEditableNode);
+        subMenus.add.appendItem(WI.UIString("Next Sibling"), this._addNextSibling.bind(this), !isEditableNode);
 
         subMenus.edit.appendItem(WI.UIString("HTML"), this._editAsHTML.bind(this), !this.editable);
         subMenus.copy.appendItem(WI.UIString("HTML"), this._copyHTML.bind(this), node.isPseudoElement());
index 696ffc0..4e7c40a 100644 (file)
@@ -474,13 +474,13 @@ WI.DebuggerSidebarPanel = class DebuggerSidebarPanel extends WI.NavigationSideba
 
         if (breakpoint === WI.debuggerManager.allExceptionsBreakpoint) {
             options.className = WI.DebuggerSidebarPanel.ExceptionIconStyleClassName;
-            options.title = WI.repeatedUIString.allExceptions();
+            options.title = WI.UIString("All Exceptions");
         } else if (breakpoint === WI.debuggerManager.uncaughtExceptionsBreakpoint) {
             options.className = WI.DebuggerSidebarPanel.ExceptionIconStyleClassName;
-            options.title = WI.repeatedUIString.uncaughtExceptions();
+            options.title = WI.UIString("Uncaught Exceptions");
         } else if (breakpoint === WI.debuggerManager.assertionFailuresBreakpoint) {
             options.className = WI.DebuggerSidebarPanel.AssertionIconStyleClassName;
-            options.title = WI.repeatedUIString.assertionFailures();
+            options.title = WI.UIString("Assertion Failures");
         } else if (breakpoint instanceof WI.DOMBreakpoint) {
             if (!breakpoint.domNodeIdentifier)
                 return null;
@@ -514,7 +514,7 @@ WI.DebuggerSidebarPanel = class DebuggerSidebarPanel extends WI.NavigationSideba
 
             if (breakpoint === WI.domDebuggerManager.allRequestsBreakpoint) {
                 options.className = WI.DebuggerSidebarPanel.AssertionIconStyleClassName;
-                options.title = WI.repeatedUIString.allRequests();
+                options.title = WI.UIString("All Requests");
             }
         } else {
             let sourceCode = breakpoint.sourceCodeLocation && breakpoint.sourceCodeLocation.displaySourceCode;
@@ -1457,7 +1457,7 @@ WI.DebuggerSidebarPanel = class DebuggerSidebarPanel extends WI.NavigationSideba
         if (InspectorBackend.domains.Debugger.setPauseOnAssertions) {
             let assertionFailuresBreakpointShown = WI.settings.showAssertionFailuresBreakpoint.value;
 
-            contextMenu.appendCheckboxItem(WI.repeatedUIString.assertionFailures(), () => {
+            contextMenu.appendCheckboxItem(WI.UIString("Assertion Failures"), () => {
                 if (assertionFailuresBreakpointShown)
                     WI.debuggerManager.removeBreakpoint(WI.debuggerManager.assertionFailuresBreakpoint);
                 else {
@@ -1479,7 +1479,7 @@ WI.DebuggerSidebarPanel = class DebuggerSidebarPanel extends WI.NavigationSideba
 
             let allRequestsBreakpointShown = WI.settings.showAllRequestsBreakpoint.value;
 
-            contextMenu.appendCheckboxItem(WI.repeatedUIString.allRequests(), () => {
+            contextMenu.appendCheckboxItem(WI.UIString("All Requests"), () => {
                 if (allRequestsBreakpointShown)
                     WI.domDebuggerManager.removeURLBreakpoint(WI.domDebuggerManager.allRequestsBreakpoint);
                 else {
index c1ad6fa..2b6513a 100644 (file)
@@ -136,7 +136,7 @@ WI.LayerTreeDetailsSidebarPanel = class LayerTreeDetailsSidebarPanel extends WI.
         columns.name.title = WI.UIString("Node");
         columns.name.sortable = false;
 
-        columns.paintCount.title = WI.UIString("Paints", "Paints @ Column title", "A count of how many times an element was painted (rendered)");
+        columns.paintCount.title = WI.UIString("Paints");
         columns.paintCount.sortable = true;
         columns.paintCount.aligned = "right";
         columns.paintCount.width = "70px";
index cffd1cf..33b9163 100644 (file)
@@ -211,7 +211,7 @@ WI.NetworkTableContentView = class NetworkTableContentView extends WI.ContentVie
         case WI.Resource.Type.XHR:
             return "XHR";
         case WI.Resource.Type.Fetch:
-            return WI.repeatedUIString.fetch();
+            return WI.UIString("Fetch");
         case WI.Resource.Type.Ping:
             return WI.UIString("Ping");
         case WI.Resource.Type.Beacon:
@@ -1172,7 +1172,7 @@ WI.NetworkTableContentView = class NetworkTableContentView extends WI.ContentVie
             align: "right",
         });
 
-        this._transferSizeColumn = new WI.TableColumn("transferSize", WI.UIString("Transfer Size", "Amount of data sent over the network for a single resource"), {
+        this._transferSizeColumn = new WI.TableColumn("transferSize", WI.UIString("Transfer Size"), {
             minWidth: 100,
             maxWidth: 150,
             initialWidth: 100,
index f393ecc..f0e94fb 100644 (file)
@@ -883,13 +883,13 @@ WI.SourcesNavigationSidebarPanel = class SourcesNavigationSidebarPanel extends W
 
         if (breakpoint === WI.debuggerManager.allExceptionsBreakpoint) {
             options.className = "breakpoint-exception-icon";
-            options.title = WI.repeatedUIString.allExceptions();
+            options.title = WI.UIString("All Exceptions");
         } else if (breakpoint === WI.debuggerManager.uncaughtExceptionsBreakpoint) {
             options.className = "breakpoint-exception-icon";
-            options.title = WI.repeatedUIString.uncaughtExceptions();
+            options.title = WI.UIString("Uncaught Exceptions");
         } else if (breakpoint === WI.debuggerManager.assertionFailuresBreakpoint) {
             options.className = "breakpoint-assertion-icon";
-            options.title = WI.repeatedUIString.assertionFailures();
+            options.title = WI.UIString("Assertion Failures");
         } else if (breakpoint instanceof WI.DOMBreakpoint) {
             if (!breakpoint.domNodeIdentifier)
                 return null;
@@ -923,7 +923,7 @@ WI.SourcesNavigationSidebarPanel = class SourcesNavigationSidebarPanel extends W
 
             if (breakpoint === WI.domDebuggerManager.allRequestsBreakpoint) {
                 options.className = "breakpoint-assertion-icon";
-                options.title = WI.repeatedUIString.allRequests();
+                options.title = WI.UIString("All Requests");
             }
         } else {
             let sourceCode = breakpoint.sourceCodeLocation && breakpoint.sourceCodeLocation.displaySourceCode;
@@ -1502,7 +1502,7 @@ WI.SourcesNavigationSidebarPanel = class SourcesNavigationSidebarPanel extends W
         if (InspectorBackend.domains.Debugger.setPauseOnAssertions) {
             let assertionFailuresBreakpointShown = WI.settings.showAssertionFailuresBreakpoint.value;
 
-            contextMenu.appendCheckboxItem(WI.repeatedUIString.assertionFailures(), () => {
+            contextMenu.appendCheckboxItem(WI.UIString("Assertion Failures"), () => {
                 if (assertionFailuresBreakpointShown)
                     WI.debuggerManager.removeBreakpoint(WI.debuggerManager.assertionFailuresBreakpoint);
                 else {
@@ -1524,7 +1524,7 @@ WI.SourcesNavigationSidebarPanel = class SourcesNavigationSidebarPanel extends W
 
             let allRequestsBreakpointShown = WI.settings.showAllRequestsBreakpoint.value;
 
-            contextMenu.appendCheckboxItem(WI.repeatedUIString.allRequests(), () => {
+            contextMenu.appendCheckboxItem(WI.UIString("All Requests"), () => {
                 if (allRequestsBreakpointShown)
                     WI.domDebuggerManager.removeURLBreakpoint(WI.domDebuggerManager.allRequestsBreakpoint);
                 else {
index fecbe60..bfce822 100644 (file)
@@ -315,7 +315,7 @@ WI.SpreadsheetCSSStyleDeclarationSection = class SpreadsheetCSSStyleDeclarationS
             break;
 
         case WI.CSSStyleDeclaration.Type.Inline:
-            this._selectorElement.textContent = WI.UIString("Style Attribute", "CSS properties defined via HTML style attribute");
+            this._selectorElement.textContent = WI.UIString("Style Attribute");
             this._selectorElement.classList.add("style-attribute");
             break;
 
index dc84c05..e1d4ebe 100644 (file)
@@ -256,7 +256,7 @@ WI.SpreadsheetRulesStyleDetailsPanel = class SpreadsheetRulesStyleDetailsPanel e
 
         let addSection = (section) => {
             if (section.style.inherited && (!previousStyle || previousStyle.node !== section.style.node))
-                addHeader(WI.UIString("Inherited From", "A section of CSS rules matching an ancestor DOM node"), section.style.node);
+                addHeader(WI.UIString("Inherited From"), section.style.node);
 
             if (!section.isDescendantOf(this)) {
                 let referenceView = this.subviews[this._sections.length];
index 517d0cc..b270126 100644 (file)
@@ -1,3 +1,16 @@
+2019-05-08  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r244952.
+
+        Caused inspector to appear blank.
+
+        Reverted changeset:
+
+        "Web Inspector: Provide UIString descriptions to improve
+        localizations"
+        https://bugs.webkit.org/show_bug.cgi?id=195132
+        https://trac.webkit.org/changeset/244952
+
 2019-05-08  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         TestRunnerWKWebView's menu callbacks should be cleared upon UI script completion
index 68c70bd..34b5ac7 100755 (executable)
@@ -74,15 +74,7 @@ for my $file (sort @files) {
         chomp;
 
         # Handle WebInspector strings. Prints a warning if a non-string literal is passed to WI.UIString().
-        # Allow: WI.UIString(string, comment)
-        #        WI.UIString(string, key, comment)
-        HandleUIString(
-            $1,
-            ($3 || (defined $3 and length $3 == 0)) ? $2 : $1,
-            ($3 || (defined $3 and length $3 == 0)) ? $3 : $2 || "",
-            $file,
-            $.
-        ) while s/WI\.UIString\("([^"]+)"(?:,\s*"([^"]*)"(?:,\s*"([^"]*)")?)?\)//;
+        HandleUIString($1, $2 || $1, $3 || "", $file, $.) while s/WI\.UIString\("([^"]+)"(?:,\s*"([^"]+)"(?:,\s*"([^"]+)")?)?\)//;
         print "$file:$.:WARNING: $&\n" while s/WI\.UIString\(.*?\)//;
 
         # Handle strings for other projects that also use this script.