Bug 216361 – [GTK] REGRESSION(r266815): Several GTK API tests are timing out

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Diego Pino]

The following GTK-API tests are failing after r266815:

Unexpected timeouts (3)
    /WebKit2Gtk/TestInspector
        /webkit/WebKitWebInspector/custom-container-destroyed
        /webkit/WebKitWebInspector/manual-attach-detach
        /webkit/WebKitWebInspector/default

[Diego Pino]

See: https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/15753/steps/API%20tests/logs/stdio

TEST: /app/webkit/Tools/glib/../../WebKitBuild/Release/bin/TestWebKitAPI/WebKit2Gtk/TestInspector...
  /webkit/WebKitWebInspector/default:                                 TIMEOUT
GLib-GIO-DEBUG: _g_io_module_get_default: Found default implementation local (GLocalVfs) for ‘gio-vfs’
GLib-DEBUG: posix_spawn avoided (child_setup specified) 
GLib-DEBUG: posix_spawn avoided (child_setup specified) 
GLib-DEBUG: posix_spawn avoided (child_setup specified) 
GLib-DEBUG: posix_spawn avoided (child_setup specified) 
resource:///org/webkit/inspector/UserInterface/Main.js:16766: CONSOLE JS ERROR SyntaxError: Unexpected keyword 'let'. Expected ';' after variable declaration.
resource:///org/webkit/inspector/UserInterface/Main.html:38:11: CONSOLE JS ERROR ReferenceError: Can't find variable: WI
  /webkit/WebKitWebInspector/manual-attach-detach:                    TIMEOUT
GLib-GIO-DEBUG: _g_io_module_get_default: Found default implementation local (GLocalVfs) for ‘gio-vfs’
GLib-DEBUG: posix_spawn avoided (child_setup specified) 
GLib-DEBUG: posix_spawn avoided (child_setup specified) 
GLib-DEBUG: posix_spawn avoided (child_setup specified) 
GLib-DEBUG: posix_spawn avoided (child_setup specified) 
resource:///org/webkit/inspector/UserInterface/Main.js:16766: CONSOLE JS ERROR SyntaxError: Unexpected keyword 'let'. Expected ';' after variable declaration.
resource:///org/webkit/inspector/UserInterface/Main.html:38:11: CONSOLE JS ERROR ReferenceError: Can't find variable: WI

[Carlos Garcia Campos]

Created attachment 408526 [details]
Patch

[Adrian Perez]

Comment on attachment 408526 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=408526&action=review

> Source/WebInspectorUI/UserInterface/Views/DOMNodeDetailsSidebarPanel.js:357
> +            let identifier = `${options.identifier ?? title}-event-listener-section`;

I wonder how was the debugging session which arrived at this… just, wow!

[Carlos Garcia Campos]

It was easy because the error message was pretty clear and we already had the revision that introduced the bug :-)

[Blaze Burg]

Thanks for investigating, Carlos!

[Carlos Garcia Campos]

Committed r266975: <https://trac.webkit.org/changeset/266975>