I'd like to add Entry, Directory and File object under FileSystemModel that represent browser side entries. It will provide handy way to manipulate entries. This is split out bug/patch from http://webkit.org/b/73301 (Web Inspector: Add FileSystemView).
Created attachment 148965 [details] Patch
Comment on attachment 148965 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=148965&action=review > Source/WebCore/inspector/front-end/FileSystemModel.js:287 > + get fileSystemModel() { return this._fileSystemModel; }, Here and below: get fileSystemModel() { return this._fileSystemModel; }, Also could you please add compiler annotations to getters (where possible).
Created attachment 148989 [details] Patch
Comment on attachment 148965 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=148965&action=review >> Source/WebCore/inspector/front-end/FileSystemModel.js:287 >> + get fileSystemModel() { return this._fileSystemModel; }, > > Here and below: > > get fileSystemModel() > { > return this._fileSystemModel; > }, > > Also could you please add compiler annotations to getters (where possible). Done.
Comment on attachment 148989 [details] Patch Clearing flags on attachment: 148989 Committed r121017: <http://trac.webkit.org/changeset/121017>
All reviewed patches have been landed. Closing bug.