Bug 84681 - Web Inspector: native nodes may have snapshot id less than base snapshot max JS object id
Summary: Web Inspector: native nodes may have snapshot id less than base snapshot max ...
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: Yury Semikhatsky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-24 01:22 PDT by Yury Semikhatsky
Modified: 2012-04-25 00:01 PDT (History)
12 users (show)

See Also:


Attachments
Patch (10.09 KB, patch)
2012-04-24 01:25 PDT, Yury Semikhatsky
pfeldman: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yury Semikhatsky 2012-04-24 01:22:45 PDT
We should bare that in mind when calculating heap snapshot diff.
Comment 1 Yury Semikhatsky 2012-04-24 01:25:35 PDT
Created attachment 138515 [details]
Patch
Comment 2 Yury Semikhatsky 2012-04-24 01:26:17 PDT
(In reply to comment #1)
> Created an attachment (id=138515) [details]
> Patch

The patch restores part of the logic removed in http://trac.webkit.org/changeset/114894 and adds a test for it.
Comment 3 Yury Semikhatsky 2012-04-24 04:23:42 PDT
Committed r115031: <http://trac.webkit.org/changeset/115031>
Comment 4 Csaba Osztrogonác 2012-04-24 05:30:08 PDT
It fails on Qt:

--- /home/webkitbuildbot/slaves/release32bit-qt5/buildslave/qt-linux-32-release-qt5/build/layout-test-results/inspector/profiler/heap-snapshot-comparison-dom-groups-change-expected.txt 
+++ /home/webkitbuildbot/slaves/release32bit-qt5/buildslave/qt-linux-32-release-qt5/build/layout-test-results/inspector/profiler/heap-snapshot-comparison-dom-groups-change-actual.txt 
@@ -1,12 +1,4 @@
 Tests that Comparison view of heap snapshots will contain added nodes even if their ids are less than the maximumm JS object id in the base snapshot.
 
-Profiler was enabled.
-Detailed heap profiles were enabled.
+Heap profiler is disabled
 
-Running: testShowAll
-Delta: +4 -1
-Deleted node id(s): 40
-Added node id(s): 15,25,35,5
-
-Profiler was disabled.
-


Is heap profiler disabled intentionally? Should we skip this test or add a platform specific expected file?
Comment 5 Yury Semikhatsky 2012-04-24 23:54:24 PDT
(In reply to comment #4)
> It fails on Qt:
> Is heap profiler disabled intentionally? Should we skip this test or add a platform specific expected file?

Sorry about that, I should have skipped the test for all non-v8 platforms as JSC doesn't support heap profiling.
Comment 6 Csaba Osztrogonác 2012-04-24 23:57:44 PDT
(In reply to comment #5)
> (In reply to comment #4)
> > It fails on Qt:
> > Is heap profiler disabled intentionally? Should we skip this test or add a platform specific expected file?
> 
> Sorry about that, I should have skipped the test for all non-v8 platforms as JSC doesn't support heap profiling.

Not problem, we found this section in the skipped list, and added this test too - http://trac.webkit.org/changeset/115035
Comment 7 Yury Semikhatsky 2012-04-25 00:01:47 PDT
(In reply to comment #6)
> (In reply to comment #5)
> > (In reply to comment #4)
> > > It fails on Qt:
> > > Is heap profiler disabled intentionally? Should we skip this test or add a platform specific expected file?
> > 
> > Sorry about that, I should have skipped the test for all non-v8 platforms as JSC doesn't support heap profiling.
> 
> Not problem, we found this section in the skipped list, and added this test too - http://trac.webkit.org/changeset/115035

Thanks! I've just updated Skipped lists on other platforms as well.