Attachment 348271 Details for Bug 188946 – Patch

[patch] Patch

bug-188946-20180828003655.patch (text/plain), 2.11 KB, created by Devin Rousso on 2018-08-28 00:36:56 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Devin Rousso

Created: 2018-08-28 00:36:56 PDT

Size: 2.11 KB

patch

obsolete

>diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
>index aca81e50605fd17f184bba6c95518eab29c53ad0..cabaafdd61c4a446ec0f4e8471ef25a33b29634d 100644
>--- a/Source/WebInspectorUI/ChangeLog
>+++ b/Source/WebInspectorUI/ChangeLog
>@@ -1,3 +1,21 @@
>+2018-08-28  Devin Rousso  <drousso@apple.com>
>+
>+        console.log() shows (anonymous function) instead of the passed string when a certain format is used
>+        https://bugs.webkit.org/show_bug.cgi?id=188946
>+        <rdar://problem/43756428>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Stack trace URLs are much more likely to follow the following format:
>+
>+            <protocol>://<path>:<line>:<column>
>+
>+        Modify the test regexp to always require that a protocol exists and to check that there are
>+        no ":" inside the protocol or path.
>+
>+        * UserInterface/Models/StackTrace.js:
>+        (WI.StackTrace.isLikelyStackTrace):
>+
> 2018-08-27  Keith Rollin  <krollin@apple.com>
> 
>         Unreviewed build fix -- disable LTO for production builds
>diff --git a/Source/WebInspectorUI/UserInterface/Models/StackTrace.js b/Source/WebInspectorUI/UserInterface/Models/StackTrace.js
>index ca72995416cc7b4625c924f6f86dd7a9ea4b9e77..9eabc3f62145785b8a6dbabcf3a75f1fafc52f35 100644
>--- a/Source/WebInspectorUI/UserInterface/Models/StackTrace.js
>+++ b/Source/WebInspectorUI/UserInterface/Models/StackTrace.js
>@@ -80,11 +80,11 @@ WI.StackTrace = class StackTrace
>         if (/^[^a-z$_]/i.test(stack[0]))
>             return false;
> 
>+        const reasonablyLongProtocolLength = 10;
>         const reasonablyLongLineLength = 500;
>         const reasonablyLongNativeMethodLength = 120;
>-        const stackTraceLine = `(.{1,${reasonablyLongLineLength}}:\\d+:\\d+|eval code|.{1,${reasonablyLongNativeMethodLength}}@\\[native code\\])`;
>+        const stackTraceLine = `([^:]{1,${reasonablyLongProtocolLength}}://[^:]{1,${reasonablyLongLineLength}}:\\d+:\\d+|eval code|.{1,${reasonablyLongNativeMethodLength}}@\\[native code\\])`;
>         const stackTrace = new RegExp(`^${stackTraceLine}(\\n${stackTraceLine})*$`, "g");
>-
>         return stackTrace.test(stack);
>     }
>

Actions: View | Formatted Diff | Diff

Attachments on bug 188946: 348079 | 348271 | 348272 | 348278 | 348282 | 348283 | 348303 | 348316 | 348356