Bug 188123 – Add some crash info to Heap::checkConn() RELEASE_ASSERTs.

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Mark Lam]

This is needed to help debug a crash.

<rdar://problem/42672268>

[Mark Lam]

Created attachment 345951 [details]
proposed patch.

[EWS Watchlist]

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


ERROR: Source/WTF/wtf/Assertions.cpp:607:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:619:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:630:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:640:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:649:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:657:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:664:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:670:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:670:  Extra space before )  [whitespace/parens] [2]
Total errors found: 9 in 8 files


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

[Mark Lam]

Comment on attachment 345951 [details]
proposed patch.

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

> Source/JavaScriptCore/ChangeLog:22
> +           VM::isEntered() tells us is the current VM is currently executing JS code.

/is/if/.  Will fix before landing.

[Keith Miller]

Comment on attachment 345951 [details]
proposed patch.

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

r=me.

> Source/JavaScriptCore/ChangeLog:10
> +           padding space in VM and Heap, and should cost any measurable perf to initialize

should *only* cost?

> Source/WTF/wtf/Assertions.cpp:676
> +void WTFCrashWithInfo(int, const char*, const char*, int, uint64_t, uint64_t, uint64_t, uint64_t, uint64_t, uint64_t) { CRASH(); }

I think you need another line here.

[Mark Lam]

Comment on attachment 345951 [details]
proposed patch.

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

Thanks for the review.

>> Source/JavaScriptCore/ChangeLog:10
>> +           padding space in VM and Heap, and should cost any measurable perf to initialize
> 
> should *only* cost?

I meant to say "should not cost"

>> Source/WTF/wtf/Assertions.cpp:676
>> +void WTFCrashWithInfo(int, const char*, const char*, int, uint64_t, uint64_t, uint64_t, uint64_t, uint64_t, uint64_t) { CRASH(); }
> 
> I think you need another line here.

Fixed.

[Mark Lam]

Created attachment 345955 [details]
patch for landing.

[EWS Watchlist]

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


ERROR: Source/WTF/wtf/Assertions.cpp:607:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:619:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:630:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:640:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:649:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:657:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:664:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:670:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/Assertions.cpp:670:  Extra space before )  [whitespace/parens] [2]
Total errors found: 9 in 8 files


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

[Mark Lam]

Landed in r234335: <http://trac.webkit.org/r234335>.

[Mark Lam]

Build fix for internal build landed in r234344: <http://trac.webkit.org/r234344>.