InspectorFileSystemAgent.cpp has some callback dispatcher classes which are almost same each other, and similar class is going to increase in this way. I'd like to unify them to single template class to reduce duplication.
Created attachment 148246 [details] Patch
Comment on attachment 148246 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=148246&action=review > Source/WebCore/inspector/InspectorFileSystemAgent.cpp:181 > + RefPtr<EntryCallback> successCallback = createCallbackDispatcher(TypeParam<EntryCallback>(), this, &GetFileSystemRootTask::gotEntry); I think you can remove TypaParam argument and do createCallbackDispatcher<EntryCallback>(this, &GetFileSystemRootTask::gotEntry); instead.
Created attachment 148762 [details] Patch
Comment on attachment 148246 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=148246&action=review >> Source/WebCore/inspector/InspectorFileSystemAgent.cpp:181 >> + RefPtr<EntryCallback> successCallback = createCallbackDispatcher(TypeParam<EntryCallback>(), this, &GetFileSystemRootTask::gotEntry); > > I think you can remove TypaParam argument and do > createCallbackDispatcher<EntryCallback>(this, &GetFileSystemRootTask::gotEntry); > instead. We can't pass template parameter partially. Another option is like that: CallbackDispatcher<EntryCallback>::create(...). Is it acceptable?
Created attachment 148789 [details] Patch
Comment on attachment 148789 [details] Patch Clearing flags on attachment: 148789 Committed r120926: <http://trac.webkit.org/changeset/120926>
All reviewed patches have been landed. Closing bug.