RESOLVED FIXED 60286
Web Inspector: [Chromium] Enable detailed heap snapshots by default 
https://bugs.webkit.org/show_bug.cgi?id=60286
Summary Web Inspector: [Chromium] Enable detailed heap snapshots by default
Mikhail Naganov
Reported 2011-05-05 12:09:30 PDT
I'm currently leaving out the old panel code to be able to quickly switch back in case if something goes wrong. I'll remove old code later.
Attachments
patch (4.25 KB, patch)
2011-05-05 12:18 PDT, Mikhail Naganov 		pfeldman: review-
mnaganov: commit-queue-
DetailsFormatted DiffDiff
comments addressed (3.70 KB, patch)
2011-05-06 11:15 PDT, Mikhail Naganov 		pfeldman: review+
mnaganov: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Mikhail Naganov
Comment 1 2011-05-05 12:18:10 PDT
Created attachment 92443 [details] patch
Pavel Feldman
Comment 2 2011-05-05 14:15:48 PDT
Comment on attachment 92443 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=92443&action=review > Source/WebCore/inspector/front-end/ProfilesPanel.js:741 > var combo = ["U+004C", "U+0045", "U+0041", "U+004B", "U+005A"]; // "LEAKZ" Is there a reason you would like to keep this code around?
Mikhail Naganov
Comment 3 2011-05-06 11:15:00 PDT
Created attachment 92610 [details] comments addressed
Mikhail Naganov
Comment 4 2011-05-18 02:48:00 PDT
Manually committed http://trac.webkit.org/changeset/86746 2011-05-17 Mikhail Naganov <mnaganov@chromium.org> Reviewed by Pavel Feldman. Web Inspector: [Chromium] Enable detailed heap snapshots by default. https://bugs.webkit.org/show_bug.cgi?id=60286 * inspector/front-end/ProfilesPanel.js: 2011-05-17 Mikhail Naganov <mnaganov@chromium.org> Reviewed by Pavel Feldman. Web Inspector: [Chromium] Enable detailed heap snapshots by default. https://bugs.webkit.org/show_bug.cgi?id=60286 * src/js/DevTools.js: ():
Note You need to log in before you can comment on or make changes to this bug.