Bug 41706 - Web Inspector: Ability to asynchronously run profiler
Summary: Web Inspector: Ability to asynchronously run profiler
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: 43900
Blocks:
  Show dependency treegraph
 
Reported: 2010-07-06 10:43 PDT by Alexei Masterov
Modified: 2010-08-15 23:52 PDT (History)
11 users (show)

See Also:


Attachments
file to help reproduce this bug (574 bytes, text/html)
2010-07-06 10:43 PDT, Alexei Masterov
no flags Details
assuming that http://codereview.chromium.org/2962007 has landed, here is the patch (17.61 KB, patch)
2010-07-13 09:45 PDT, Mikhail Naganov
mnaganov: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexei Masterov 2010-07-06 10:43:42 PDT
Created attachment 60639 [details]
file to help reproduce this bug

Users should be a able to start/stop profiling while the javascript code is running.

For example, in an attached page, once javascript enters infinite loop, if the user tries to start CPU profiling or take heap snapshot - nothing will happen.  Also if the CPU profiler is already running the user will not be able to stop it.

Desired behavior: CPU profiler / Heap snapshotting should be independent of the javascript, since it is especially useful in such cases where the code is already running.
Comment 1 Mikhail Naganov 2010-07-07 02:32:51 PDT
Sorry for having you misguided, but this bug doesn't actually depend on async breakpoints.
Comment 2 Mikhail Naganov 2010-07-13 09:45:01 PDT
Created attachment 61388 [details]
assuming that http://codereview.chromium.org/2962007 has landed, here is the patch
Comment 3 WebKit Review Bot 2010-07-13 10:09:44 PDT
Attachment 61388 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/3453263
Comment 4 Yury Semikhatsky 2010-08-15 23:52:55 PDT
Was fixed in a more general way in bug 43900.