[webkit-reviews] review requested: [Bug 45982] Web Inspector: FileSystem integration : [Attachment 71335] patch3
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Oct 20 14:18:33 PDT 2010
Kavita Kanetkar <kkanetkar at chromium.org> has asked for review:
Bug 45982: Web Inspector: FileSystem integration
https://bugs.webkit.org/show_bug.cgi?id=45982
Attachment 71335: patch3
https://bugs.webkit.org/attachment.cgi?id=71335&action=review
------- Additional Comments from Kavita Kanetkar <kkanetkar at chromium.org>
Changing UI needs a rewrite to FileSystem.js. Will be a fixme for this patch.
Also, I will need to change type of localizedStrings.js to plain text land it,
add strings from FileSystem.js and re-change the type.
I have a separate patch for that. Rest all nits addressed. Again, this is
disabled for non-chromium platforms.
Thanks for the review.
More information about the webkit-reviews
mailing list