Bug 40834 - Web Inspector: Implement 'heapSnapshot' in InspectorController.
Summary: Web Inspector: Implement 'heapSnapshot' in InspectorController.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Mikhail Naganov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-18 08:30 PDT by Mikhail Naganov
Modified: 2010-06-18 09:23 PDT (History)
9 users (show)

See Also:


Attachments
proposed patch (9.15 KB, patch)
2010-06-18 08:33 PDT, Mikhail Naganov
no flags Details | Formatted Diff | Diff
lint (9.15 KB, patch)
2010-06-18 08:36 PDT, Mikhail Naganov
yurys: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mikhail Naganov 2010-06-18 08:30:30 PDT
After switching to ScriptDebugServer, DebuggerAgent protocol can no more be used.
Comment 1 Mikhail Naganov 2010-06-18 08:33:09 PDT
Created attachment 59111 [details]
proposed patch
Comment 2 WebKit Review Bot 2010-06-18 08:33:55 PDT
Attachment 59111 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style', '--no-squash']" exit_code: 1
WebCore/bindings/v8/ScriptProfiler.cpp:55:  Place brace on its own line for function definitions.  [whitespace/braces] [4]
Total errors found: 1 in 13 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Mikhail Naganov 2010-06-18 08:36:30 PDT
Created attachment 59114 [details]
lint
Comment 4 Yury Semikhatsky 2010-06-18 08:42:44 PDT
Comment on attachment 59114 [details]
lint

WebCore/bindings/js/ScriptProfiler.h:42
 +      static void heapSnapshot() { }
Please rename it for consistency with other code to takeHeapSnapshot.

WebCore/bindings/v8/ScriptProfiler.h:46
 +      static void heapSnapshot();
Please make sure there is a bug about heap snapshotting in JSC.
Comment 5 Mikhail Naganov 2010-06-18 08:58:58 PDT
(In reply to comment #4)
> (From update of attachment 59114 [details])
> WebCore/bindings/js/ScriptProfiler.h:42
>  +      static void heapSnapshot() { }
> Please rename it for consistency with other code to takeHeapSnapshot.
>

Renamed.
 
> WebCore/bindings/v8/ScriptProfiler.h:46
>  +      static void heapSnapshot();
> Please make sure there is a bug about heap snapshotting in JSC.

There is https://bugs.webkit.org/show_bug.cgi?id=31134 requesting heap snapshotting.
Comment 6 Mikhail Naganov 2010-06-18 09:23:03 PDT
Manually committed http://trac.webkit.org/changeset/61411


    2010-06-18  Mikhail Naganov  <mnaganov@chromium.org>
    
            Reviewed by Yury Semikhatsky.
    
            WebInspector: Implement 'takeHeapSnapshot' in InspectorController.
    
            https://bugs.webkit.org/show_bug.cgi?id=40834
    
            * bindings/js/ScriptProfiler.h:
            (WebCore::ScriptProfiler::takeHeapSnapshot):
            * bindings/v8/ScriptProfiler.cpp:
            (WebCore::ScriptProfiler::takeHeapSnapshot):
            * bindings/v8/ScriptProfiler.h:
            * inspector/InspectorBackend.cpp:
            (WebCore::InspectorBackend::takeHeapSnapshot):
            * inspector/InspectorBackend.h:
            * inspector/InspectorBackend.idl:
            * inspector/InspectorController.cpp:
            (WebCore::InspectorController::takeHeapSnapshot):
            * inspector/InspectorController.h:
    
            * src/js/HeapProfilerPanel.js:
            (WebInspector.HeapSnapshotProfileType.prototype.buttonClicked):
            * src/js/InspectorControllerImpl.js:
            (devtools.InspectorBackendImpl):
            * src/js/ProfilerAgent.js:
            (devtools.ProfilerAgent.prototype.startProfiling):