[Webkit-unassigned] [Bug 95985] Web Inspector: Migrate FileSystem to async protocol commands.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=95985





--- Comment #2 from Alexander Pavlov (apavlov) <apavlov at chromium.org>  2012-09-07 04:50:25 PST ---
(From update of attachment 162530)
View in context: https://bugs.webkit.org/attachment.cgi?id=162530&action=review

> Source/WebCore/inspector/front-end/FileSystemModel.js:169
> +                callback(FileError.SECURITY_ERR);

optional callback used unconditionally?

> Source/WebCore/inspector/front-end/FileSystemModel.js:255
> +                callback(FileError.SECURITY_ERR);

ditto

> Source/WebCore/inspector/front-end/FileSystemModel.js:332
> +     * @param {function(number, string=, string=)=} callback

param declared as optional but used without any checks in the code

> Source/WebCore/inspector/front-end/FileSystemModel.js:387
> +                callback(FileError.SECURITY_ERR);

ditto

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.