Bug 43475

Summary: Web Inspector: serialize CPU profiles to InspectorValues
Product: WebKit Reporter: Yury Semikhatsky <yurys>
Component: Web Inspector (Deprecated)Assignee: Yury Semikhatsky <yurys>
Status: RESOLVED FIXED    
Severity: Normal CC: bweinstein, joepeck, keishi, loislo, mnaganov, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch pfeldman: review+

Yury Semikhatsky
Reported 2010-08-04 02:41:34 PDT
Web Inspector: serialize CPU profiles to InspectorValues when passing them to the frontend.
Attachments
Patch (24.10 KB, patch)
2010-08-04 03:16 PDT, Yury Semikhatsky 		pfeldman: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yury Semikhatsky
Comment 1 2010-08-04 03:16:58 PDT
Created attachment 63435 [details] Patch
Pavel Feldman
Comment 2 2010-08-04 03:26:06 PDT
Comment on attachment 63435 [details] Patch WebCore/inspector/InspectorController.cpp:1499 + #if USE(JSC) Do you still need this?
Yury Semikhatsky
Comment 3 2010-08-04 03:30:07 PDT
(In reply to comment #2) > (From update of attachment 63435 [details]) > WebCore/inspector/InspectorController.cpp:1499 > + #if USE(JSC) > Do you still need this? No, removed.
Yury Semikhatsky
Comment 4 2010-08-04 05:23:05 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/Android.jscbindings.mk M WebCore/CMakeLists.txt M WebCore/ChangeLog M WebCore/GNUmakefile.am M WebCore/WebCore.pro M WebCore/WebCore.vcproj/WebCore.vcproj M WebCore/WebCore.xcodeproj/project.pbxproj A WebCore/bindings/js/ScriptProfile.cpp M WebCore/bindings/js/ScriptProfile.h M WebCore/bindings/js/ScriptProfiler.cpp M WebCore/bindings/scripts/CodeGeneratorJS.pm M WebCore/bindings/v8/ScriptProfile.cpp M WebCore/bindings/v8/ScriptProfile.h M WebCore/inspector/Inspector.idl M WebCore/inspector/InspectorController.cpp M WebCore/inspector/InspectorController.h M WebCore/inspector/InspectorFrontend.cpp M WebCore/inspector/InspectorFrontend.h Committed r64642
Note You need to log in before you can comment on or make changes to this bug.