Bug 100340

Summary:

Web Inspector: NMI instrument InspectorDebuggerAgent. It uses a lot of memory on heavy js sites because it keeps all the scripts in a map

Product:

WebKit

Reporter:

Ilya Tikhonovsky <loislo>

Component:

Web Inspector (Deprecated)

Assignee:

Ilya Tikhonovsky <loislo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

apavlov, dglazkov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, webkit.review.bot, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	yurys: review+, webkit-ews: commit-queue-
for try bots	none
for try bots	none

Ilya Tikhonovsky

Reported 2012-10-25 00:09:35 PDT

EOM

Attachments
Patch (5.19 KB, patch) 2012-10-25 00:36 PDT, Ilya Tikhonovsky	yurys: review+ webkit-ews: commit-queue-	Details Formatted Diff Diff
for try bots (5.22 KB, patch) 2012-10-25 00:56 PDT, Ilya Tikhonovsky	no flags	Details Formatted Diff Diff
for try bots (5.48 KB, patch) 2012-10-25 01:12 PDT, Ilya Tikhonovsky	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Ilya Tikhonovsky

Comment 1 2012-10-25 00:36:35 PDT

Created attachment 170571 [details] Patch

Early Warning System Bot

Comment 2 2012-10-25 00:44:53 PDT

Comment on attachment 170571 [details] Patch Attachment 170571 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/14552522

Early Warning System Bot

Comment 3 2012-10-25 00:45:12 PDT

Comment on attachment 170571 [details] Patch Attachment 170571 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/14566512

Yury Semikhatsky

Comment 4 2012-10-25 00:49:26 PDT

Comment on attachment 170571 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=170571&action=review > Source/WebCore/ChangeLog:3 > + Web Inspector: NMI instrument InspectorDebuggerAgent. It uses a lot of memory on heavy js sites because it keeps all the scripts in a map. I'm wondering if the scripts are the same as in VM or we store a copy of them in the debugger agent.

Ilya Tikhonovsky

Comment 5 2012-10-25 00:56:00 PDT

Created attachment 170576 [details] for try bots

Build Bot

Comment 6 2012-10-25 01:04:37 PDT

Comment on attachment 170571 [details] Patch Attachment 170571 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14562510

WebKit Review Bot

Comment 7 2012-10-25 01:05:29 PDT

Comment on attachment 170571 [details] Patch Attachment 170571 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14544534

Early Warning System Bot

Comment 8 2012-10-25 01:06:09 PDT

Comment on attachment 170576 [details] for try bots Attachment 170576 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/14552531