Bug 41706

Summary:

Web Inspector: Ability to asynchronously run profiler

Product:

WebKit

Reporter:

Alexei Masterov <masterov>

Component:

Web Inspector (Deprecated)

Assignee:

Mikhail Naganov <mnaganov>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bweinstein, dglazkov, joepeck, keishi, mnaganov, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

43900

Bug Blocks:

Attachments:

Description	Flags
file to help reproduce this bug	none
assuming that http://codereview.chromium.org/2962007 has landed, here is the patch	mnaganov: commit-queue-

Alexei Masterov

Reported 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.

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
View All Add attachment proposed patch, testcase, etc.

Mikhail Naganov

Comment 1 2010-07-07 02:32:51 PDT

Sorry for having you misguided, but this bug doesn't actually depend on async breakpoints.

Mikhail Naganov

Comment 2 2010-07-13 09:45:01 PDT

Created attachment 61388 [details] assuming that http://codereview.chromium.org/2962007 has landed, here is the patch

WebKit Review Bot

Comment 3 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

Yury Semikhatsky

Comment 4 2010-08-15 23:52:55 PDT

Was fixed in a more general way in bug 43900.

Note You need to log in before you can comment on or make changes to this bug.