RESOLVED FIXED Bug 96913
Web Inspector: NMI: now when we can detect instrumented classes we can remove addInstrumentedMember and use addMember for everything 
https://bugs.webkit.org/show_bug.cgi?id=96913
Summary Web Inspector: NMI: now when we can detect instrumented classes we can remove...
Ilya Tikhonovsky
Reported 2012-09-17 06:19:59 PDT
EOM
Attachments
Patch (100.56 KB, patch)
2012-09-17 06:24 PDT, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
comment addressed - visited check returned (100.96 KB, patch)
2012-09-17 06:47 PDT, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
rebaselined (101.89 KB, patch)
2012-09-17 06:58 PDT, Ilya Tikhonovsky 		yurys: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Ilya Tikhonovsky
Comment 1 2012-09-17 06:24:25 PDT
Created attachment 164381 [details] Patch
Yury Semikhatsky
Comment 2 2012-09-17 06:37:24 PDT
Comment on attachment 164381 [details] Patch Before switching to the new instrumentaion we should make sure it works, in particular the problem with MemoryInstrumentationTest failure should be addressed first: http://trac.webkit.org/changeset/128744
Ilya Tikhonovsky
Comment 3 2012-09-17 06:47:07 PDT
Created attachment 164387 [details] comment addressed - visited check returned
Ilya Tikhonovsky
Comment 4 2012-09-17 06:58:36 PDT
Created attachment 164388 [details] rebaselined
Ilya Tikhonovsky
Comment 5 2012-09-17 08:12:15 PDT
Committed r128762: <http://trac.webkit.org/changeset/128762>
Note You need to log in before you can comment on or make changes to this bug.