Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (225633 => 225634)
--- trunk/Source/_javascript_Core/ChangeLog 2017-12-07 18:54:42 UTC (rev 225633)
+++ trunk/Source/_javascript_Core/ChangeLog 2017-12-07 19:06:17 UTC (rev 225634)
@@ -1,3 +1,13 @@
+2017-12-07 Eric Carlson <[email protected]>
+
+ Simplify log channel configuration UI
+ https://bugs.webkit.org/show_bug.cgi?id=180527
+ <rdar://problem/35908382>
+
+ Reviewed by Joseph Pecoraro.
+
+ * inspector/protocol/Console.json:
+
2017-12-07 Mark Lam <[email protected]>
[Re-landing r225620] Refactoring: Rename ScrambledPtr to Poisoned.
Modified: trunk/Source/_javascript_Core/inspector/protocol/Console.json (225633 => 225634)
--- trunk/Source/_javascript_Core/inspector/protocol/Console.json 2017-12-07 18:54:42 UTC (rev 225633)
+++ trunk/Source/_javascript_Core/inspector/protocol/Console.json 2017-12-07 19:06:17 UTC (rev 225634)
@@ -12,7 +12,7 @@
{
"id": "ChannelLevel",
"type": "string",
- "enum": ["off", "log", "error", "warning", "info", "debug"],
+ "enum": ["off", "basic", "verbose"],
"description": "Level of logging."
},
{
Modified: trunk/Source/WebCore/ChangeLog (225633 => 225634)
--- trunk/Source/WebCore/ChangeLog 2017-12-07 18:54:42 UTC (rev 225633)
+++ trunk/Source/WebCore/ChangeLog 2017-12-07 19:06:17 UTC (rev 225634)
@@ -1,3 +1,17 @@
+2017-12-07 Eric Carlson <[email protected]>
+
+ Simplify log channel configuration UI
+ https://bugs.webkit.org/show_bug.cgi?id=180527
+ <rdar://problem/35908382>
+
+ Reviewed by Joseph Pecoraro.
+
+ No new tests -- no change in functionality.
+
+ * inspector/agents/WebConsoleAgent.cpp:
+ (WebCore::WebConsoleAgent::getLoggingChannels):
+ (WebCore::channelConfigurationForString):
+
2017-12-07 Joseph Pecoraro <[email protected]>
ServiceWorker Inspector: Use the SW's scope url for the remote inspection target metadata
Modified: trunk/Source/WebCore/inspector/agents/WebConsoleAgent.cpp (225633 => 225634)
--- trunk/Source/WebCore/inspector/agents/WebConsoleAgent.cpp 2017-12-07 18:54:42 UTC (rev 225633)
+++ trunk/Source/WebCore/inspector/agents/WebConsoleAgent.cpp 2017-12-07 19:06:17 UTC (rev 225634)
@@ -66,23 +66,17 @@
if (!logChannel)
return;
- Inspector::Protocol::Console::ChannelLevel level = Inspector::Protocol::Console::ChannelLevel::Off;
+ auto level = Inspector::Protocol::Console::ChannelLevel::Off;
if (logChannel->state != WTFLogChannelOff) {
switch (logChannel->level) {
case WTFLogLevelAlways:
- level = Inspector::Protocol::Console::ChannelLevel::Log;
- break;
case WTFLogLevelError:
- level = Inspector::Protocol::Console::ChannelLevel::Error;
- break;
case WTFLogLevelWarning:
- level = Inspector::Protocol::Console::ChannelLevel::Warning;
+ level = Inspector::Protocol::Console::ChannelLevel::Basic;
break;
case WTFLogLevelInfo:
- level = Inspector::Protocol::Console::ChannelLevel::Info;
- break;
case WTFLogLevelDebug:
- level = Inspector::Protocol::Console::ChannelLevel::Debug;
+ level = Inspector::Protocol::Console::ChannelLevel::Verbose;
break;
}
}
@@ -105,15 +99,9 @@
level = WTFLogLevelError;
} else {
state = WTFLogChannelOn;
- if (equalIgnoringASCIICase(levelString, "log"))
- level = WTFLogLevelAlways;
- else if (equalIgnoringASCIICase(levelString, "error"))
- level = WTFLogLevelError;
- else if (equalIgnoringASCIICase(levelString, "warning"))
+ if (equalIgnoringASCIICase(levelString, "basic"))
level = WTFLogLevelWarning;
- else if (equalIgnoringASCIICase(levelString, "info"))
- level = WTFLogLevelInfo;
- else if (equalIgnoringASCIICase(levelString, "debug"))
+ else if (equalIgnoringASCIICase(levelString, "verbose"))
level = WTFLogLevelDebug;
else
return std::nullopt;
Modified: trunk/Source/WebInspectorUI/ChangeLog (225633 => 225634)
--- trunk/Source/WebInspectorUI/ChangeLog 2017-12-07 18:54:42 UTC (rev 225633)
+++ trunk/Source/WebInspectorUI/ChangeLog 2017-12-07 19:06:17 UTC (rev 225634)
@@ -1,3 +1,16 @@
+2017-12-07 Eric Carlson <[email protected]>
+
+ Simplify log channel configuration UI
+ https://bugs.webkit.org/show_bug.cgi?id=180527
+ <rdar://problem/35908382>
+
+ Reviewed by Joseph Pecoraro.
+
+ * Localizations/en.lproj/localizedStrings.js:
+ * UserInterface/Models/LoggingChannel.js:
+ * UserInterface/Views/SettingsTabContentView.js:
+ (WI.SettingsTabContentView.prototype._createGeneralSettingsView):
+
2017-12-06 Devin Rousso <[email protected]>
Web Inspector: Canvas: spurious warnings shown for commands that changed pixel contents
Modified: trunk/Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js (225633 => 225634)
--- trunk/Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js 2017-12-07 18:54:42 UTC (rev 225633)
+++ trunk/Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js 2017-12-07 19:06:17 UTC (rev 225634)
@@ -128,6 +128,7 @@
localizedStrings["Back (%s)"] = "Back (%s)";
localizedStrings["Background"] = "Background";
localizedStrings["Backtrace"] = "Backtrace";
+localizedStrings["Basic"] = "Basic";
localizedStrings["Basis"] = "Basis";
localizedStrings["Beacon"] = "Beacon";
localizedStrings["Beacons"] = "Beacons";
@@ -274,7 +275,6 @@
localizedStrings["Database no longer has expected version."] = "Database no longer has expected version.";
localizedStrings["Databases"] = "Databases";
localizedStrings["Date"] = "Date";
-localizedStrings["Debug"] = "Debug";
localizedStrings["Debug: "] = "Debug: ";
localizedStrings["Debugger"] = "Debugger";
localizedStrings["Debugger Paused"] = "Debugger Paused";
@@ -498,7 +498,6 @@
localizedStrings["Index"] = "Index";
localizedStrings["Index Key \u2014 %s"] = "Index Key \u2014 %s";
localizedStrings["Indexed Databases"] = "Indexed Databases";
-localizedStrings["Info"] = "Info";
localizedStrings["Info: "] = "Info: ";
localizedStrings["Infos"] = "Infos";
localizedStrings["Inherited From"] = "Inherited From";
@@ -556,7 +555,6 @@
localizedStrings["Local Variables"] = "Local Variables";
localizedStrings["Located at %s"] = "Located at %s";
localizedStrings["Location"] = "Location";
-localizedStrings["Log"] = "Log";
localizedStrings["Log Canvas Context"] = "Log Canvas Context";
localizedStrings["Log Element"] = "Log Element";
localizedStrings["Log Frame Text"] = "Log Frame Text";
@@ -1007,6 +1005,7 @@
localizedStrings["Value"] = "Value";
localizedStrings["Variables"] = "Variables";
localizedStrings["Variants"] = "Variants";
+localizedStrings["Verbose"] = "Verbose";
localizedStrings["Version"] = "Version";
localizedStrings["Vertex"] = "Vertex";
localizedStrings["Vertex Shader"] = "Vertex Shader";
@@ -1019,7 +1018,6 @@
localizedStrings["Waiting"] = "Waiting";
localizedStrings["Waiting for canvas contexts created by script or CSS."] = "Waiting for canvas contexts created by script or CSS.";
localizedStrings["Waiting for frames…"] = "Waiting for frames…";
-localizedStrings["Warning"] = "Warning";
localizedStrings["Warning: "] = "Warning: ";
localizedStrings["Warnings"] = "Warnings";
localizedStrings["Watch Expressions"] = "Watch Expressions";
Modified: trunk/Source/WebInspectorUI/UserInterface/Models/LoggingChannel.js (225633 => 225634)
--- trunk/Source/WebInspectorUI/UserInterface/Models/LoggingChannel.js 2017-12-07 18:54:42 UTC (rev 225633)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/LoggingChannel.js 2017-12-07 19:06:17 UTC (rev 225634)
@@ -52,9 +52,6 @@
WI.LoggingChannel.Level = {
Off: "off",
- Log: "log",
- Error: "error",
- Warning: "warning",
- Info: "info",
- Debug: "debug",
+ Basic: "basic",
+ Verbose: "verbose",
};
Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js (225633 => 225634)
--- trunk/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js 2017-12-07 18:54:42 UTC (rev 225633)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js 2017-12-07 19:06:17 UTC (rev 225634)
@@ -217,11 +217,8 @@
if (WI.LogManager.supportsLogChannels()) {
const logLevels = [
[WI.LoggingChannel.Level.Off, WI.UIString("Off")],
- [WI.LoggingChannel.Level.Log, WI.UIString("Log")],
- [WI.LoggingChannel.Level.Error, WI.UIString("Error")],
- [WI.LoggingChannel.Level.Warning, WI.UIString("Warning")],
- [WI.LoggingChannel.Level.Info, WI.UIString("Info")],
- [WI.LoggingChannel.Level.Debug, WI.UIString("Debug")],
+ [WI.LoggingChannel.Level.Basic, WI.UIString("Basic")],
+ [WI.LoggingChannel.Level.Verbose, WI.UIString("Verbose")],
];
const editorLabels = {
media: WI.UIString("Media Logging:"),
