Bug 91444 - dumpCallFrame is broken in ToT
Summary: dumpCallFrame is broken in ToT
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Oliver Hunt
URL:
Keywords:
Depends on: 91462
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-16 16:39 PDT by Oliver Hunt
Modified: 2012-07-18 16:26 PDT (History)
2 users (show)

See Also:


Attachments
Patch (16.63 KB, patch)
2012-07-16 17:14 PDT, Oliver Hunt
no flags Details | Formatted Diff | Diff
Patch (17.43 KB, patch)
2012-07-16 18:05 PDT, Oliver Hunt
barraclough: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Hunt 2012-07-16 16:39:44 PDT
dumpCallFrame is broken in ToT
Comment 1 Oliver Hunt 2012-07-16 17:14:38 PDT
Created attachment 152649 [details]
Patch
Comment 2 Build Bot 2012-07-16 17:40:33 PDT
Comment on attachment 152649 [details]
Patch

Attachment 152649 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/13260176
Comment 3 Oliver Hunt 2012-07-16 18:05:13 PDT
Created attachment 152664 [details]
Patch
Comment 4 Oliver Hunt 2012-07-16 18:14:26 PDT
Committed r122790: <http://trac.webkit.org/changeset/122790>
Comment 5 Jessie Berlin 2012-07-16 18:30:04 PDT
(In reply to comment #4)
> Committed r122790: <http://trac.webkit.org/changeset/122790>

This broke the Windows build:

http://build.webkit.org/builders/Apple%20Win%20Release%20%28Build%29/builds/33091/steps/compile-webkit/logs/stdio

3>Linking...
3>   Creating library C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\lib\JavaScriptCore.lib and object C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\lib\JavaScriptCore.exp
3>JavaScriptCore.exp : error LNK2001: unresolved external symbol "public: void __thiscall JSC::Interpreter::dumpCallFrame(class JSC::ExecState *)" (?dumpCallFrame@Interpreter@JSC@@QAEXPAVExecState@2@@Z)
3>C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\bin\JavaScriptCore.dll : fatal error LNK1120: 1 unresolved externals
3>Build log was saved at "file://C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\obj\JavaScriptCore\BuildLog.htm"
3>JavaScriptCore - 2 error(s), 0 warning(s)
Comment 6 WebKit Review Bot 2012-07-16 18:44:34 PDT
Re-opened since this is blocked by 91462
Comment 7 Oliver Hunt 2012-07-18 16:26:19 PDT
Committed r123042: <http://trac.webkit.org/changeset/123042>