I'd like to reuse PopoverContentHelper in FlameChart.
Created attachment 191483 [details] Patch
Comment on attachment 191483 [details] Patch LGTM
Comment on attachment 191483 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=191483&action=review > Source/WebCore/inspector/front-end/Popover.js:374 > +WebInspector.PopupContentHelper = function(title) Consider renaming to PopoverTableHelper ? > Source/WebCore/inspector/front-end/Popover.js:430 > + appendStackTrace: function(title, stackTrace, callFrameLinkifier) Now Popover.js depends on the stack trace, can we avoid this?
Committed r144759: <http://trac.webkit.org/changeset/144759>