Bug 47601 – [gdb] Add pretty-print supports for UString,Identifier and JSString

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Yi Shen]

To make gdb debugging easy, I add three pretty-print supports for UString,Identifier and JSString.
e.g.

Breakpoint 1, callDefaultValueFunction (exec=0xb68674f8, object=0xb6815400, 
    propertyName="toString")
    at ../../../JavaScriptCore/runtime/JSObject.cpp:249
249	    CallType callType = getCallData(function, callData);
(gdb) l
244	
245	static ALWAYS_INLINE JSValue callDefaultValueFunction(ExecState* exec, const JSObject* object, const Identifier& propertyName)
246	{
247	    JSValue function = object->get(exec, propertyName);
(gdb) d 1
(gdb) p function.toString(exec)
$1 = "function toString() {\n    [native code]\n}"
(gdb) p propertyName
$2 = "toString"

(gdb) l
295	EncodedJSValue JSC_HOST_CALL stringProtoFuncReplace(ExecState* exec)
296	{
297	    JSValue thisValue = exec->hostThisValue();
298	    JSString* sourceVal = thisValue.toThisJSString(exec);
299	    JSValue pattern = exec->argument(0);
300	    JSValue replacement = exec->argument(1);
(gdb) p *sourceVal
$2 = "abs round ceil floor"
(gdb) p pattern.toString(exec)
$3 = "/\\s+$/"
(gdb) p replacement.toString(exec)
[Thread 0xb74ffb70 (LWP 5092) exited]
$4 = "-1.9918709106886894"

[Yi Shen]

Created attachment 70626 [details]
first try

[WebKit Review Bot]

Attachment 70626 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
WebKitTools/gdb/webkit.py:102:  expected 2 blank lines, found 1  [pep8/E302] [5]
WebKitTools/gdb/webkit.py:116:  expected 2 blank lines, found 1  [pep8/E302] [5]
WebKitTools/gdb/webkit.py:121:  expected 2 blank lines, found 1  [pep8/E302] [5]
Total errors found: 3 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Yi Shen]

Created attachment 70631 [details]
fix style

[WebKit Commit Bot]

Comment on attachment 70631 [details]
fix style 

Clearing flags on attachment: 70631

Committed r69713: <http://trac.webkit.org/changeset/69713>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[WebKit Review Bot]

http://trac.webkit.org/changeset/69713 might have broken Chromium Mac Release

[Yi Shen]

(In reply to comment #6)
> http://trac.webkit.org/changeset/69713 might have broken Chromium Mac Release

My changes should NOT be involved in any building process.