[webkit-reviews] review denied: [Bug 111198] Web Inspector: feature request save/load CPU profile : [Attachment 195976] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 2 05:52:38 PDT 2013


Yury Semikhatsky <yurys at chromium.org> has denied Ilya Tikhonovsky
<loislo at chromium.org>'s request for review:
Bug 111198: Web Inspector: feature request save/load CPU profile
https://bugs.webkit.org/show_bug.cgi?id=111198

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

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


Can we test this code?

> Source/WebCore/ChangeLog:10
> +	   So I refactored saving mechanic in ProfilesPanel which now doesn't
depend on

mechanic -> mechanics

> Source/WebCore/inspector/front-end/CPUProfileView.js:96
> +	   this._processProfileData(this.profileHeader._profile);

Can you add a note that this can happen only when the profile is loaded from
file?

> Source/WebCore/inspector/front-end/CPUProfileView.js:779
> + * @implements {WebInspector.OutputStreamDelegate}

This is weird, the same class is stream and delegate?

> Source/WebCore/inspector/front-end/CPUProfileView.js:816
> +    onError: function(reader, event) { },

Can we print something to the console?

> Source/WebCore/inspector/front-end/CPUProfileView.js:826
> +    close: function() { },

{} on separate lines

> Source/WebCore/inspector/front-end/CPUProfileView.js:870
> +		   // Profiling was tentatively terminated with the "Clear all
profiles." button.

tentatively ?


More information about the webkit-reviews mailing list