In recent path CookiesTable added context menu listener on DataGrid element to override behavior. Integrating with DataGrid instead of overriding looks better. Actually that patch is reduced version of patch that was waiting for better back-end cookies management.
Created attachment 195517 [details] Patch
Comment on attachment 195517 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195517&action=review > Source/WebCore/inspector/front-end/CookiesTable.js:281 > + _refresh: function() { { on the next line. > Source/WebCore/inspector/front-end/CookiesTable.js:282 > + this._refreshCallback && this._refreshCallback(); We never do this, please use explicit if. > Source/WebCore/inspector/front-end/DataGrid.js:33 > + * @param {?function(!WebInspector.ContextMenu, WebInspector.DataGridNode)=} contextMenuCallback We don't use nullable optional fields. Please remove ? from here and above.
Comment on attachment 195517 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195517&action=review >> Source/WebCore/inspector/front-end/CookiesTable.js:281 >> + _refresh: function() { > > { on the next line. oops. fixed >> Source/WebCore/inspector/front-end/CookiesTable.js:282 >> + this._refreshCallback && this._refreshCallback(); > > We never do this, please use explicit if. Done. >> Source/WebCore/inspector/front-end/DataGrid.js:33 >> + * @param {?function(!WebInspector.ContextMenu, WebInspector.DataGridNode)=} contextMenuCallback > > We don't use nullable optional fields. Please remove ? from here and above. OK
Created attachment 195697 [details] Patch
Comment on attachment 195697 [details] Patch Clearing flags on attachment: 195697 Committed r147198: <http://trac.webkit.org/changeset/147198>
All reviewed patches have been landed. Closing bug.