[webkit-reviews] review granted: [Bug 52634] Web Inspector: Add layout test for the CPU profiler : [Attachment 79288] patch

[bugzilla-daemon at webkit.org]

Yury Semikhatsky <yurys at chromium.org> has granted Mikhail Naganov
<mnaganov at chromium.org>'s request for review:
Bug 52634: Web Inspector: Add layout test for the CPU profiler
https://bugs.webkit.org/show_bug.cgi?id=52634

Attachment 79288: patch
https://bugs.webkit.org/attachment.cgi?id=79288&action=review

------- Additional Comments from Yury Semikhatsky <yurys at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=79288&action=review

> LayoutTests/inspector/cpu-profiler-profiling.html:29
> +	       var tree = this.profileDataGridTree;

You could dump textContent of the tree instead of traversing it.

> LayoutTests/inspector/cpu-profiler-profiling.html:62
> +Tests that CPU profiling works.

Layout tests usually include link to the bug.