86295 – [Qt] REGRESSION(r116828): inspector tests started to assert

Bug 86295 - [Qt] REGRESSION(r116828): inspector tests started to assert

Summary: [Qt] REGRESSION(r116828): inspector tests started to assert

Status:	RESOLVED FIXED

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	Tools / Tests (show other bugs)
Version:	528+ (Nightly build)
Hardware:	All All

Importance:	P1 Critical
Assignee:	Nobody

URL:
Keywords:	Qt, QtTriaged

Depends on:
Blocks:	79668 86241
	Show dependency tree / graph

Reported:	2012-05-12 07:36 PDT by Csaba Osztrogonác
Modified:	2012-05-14 01:56 PDT (History)
CC List:	5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Csaba Osztrogonác 2012-05-12 07:36:12 PDT

http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Debug/r116832%20%2822819%29/results.html

crash log for DumpRenderTree (pid 21220):
STDOUT: <empty>
STDERR: ASSERTION FAILED: !propertyName.isNull()
STDERR: ../../../Source/JavaScriptCore/runtime/PropertyName.h(83) : JSC::PropertyName::PropertyName(const JSC::Identifier&)
STDERR: 1   0x7fc56ffe959d /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/WebKitBuild/Debug/lib/libQtWebKit.so.4(JSC::PropertyName::PropertyName(JSC::Identifier const&)+0x57) [0x7fc56ffe959d]
STDERR: 2   0x7fc5710d371d /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::JSJavaScriptCallFrame::scopeType(JSC::ExecState*)+0x16f) [0x7fc5710d371d]
STDERR: 3   0x7fc5713d47cc /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::jsJavaScriptCallFramePrototypeFunctionScopeType(JSC::ExecState*)+0x12b) [0x7fc5713d47cc]
STDERR: 4   0x7fc5246d4265 [0x7fc5246d4265]

Now the Qt Debug bots exit early because of these new assertions.
Could you check what happened, please?

Comment 1 Csaba Osztrogonác 2012-05-13 23:45:37 PDT

I skipped inspector/debugger tests - https://trac.webkit.org/changeset/116923/trunk/LayoutTests/platform/qt/Skipped

Please unskip it with the proper fix.

Comment 2 Gavin Barraclough 2012-05-14 00:26:48 PDT

Apologies for the breakage, I think this should be fixed by r116926.

Comment 3 Csaba Osztrogonác 2012-05-14 01:56:55 PDT

Thanks for the fix. I unskipped the tests - https://trac.webkit.org/changeset/116929