The existing implementation (InspectorBackend.Command.prototype.promise) assumes that there is only one return value (as a payload object, hence variable name 'payload') but InspectorBackend will actually unpack multiple return values as multiple callback arguments. The wrapper callback here should instead slice the arguments array (excluding the error message) and apply that to the resolve thunk.
<rdar://problem/17782742>
Created attachment 235367 [details] Patch
Created attachment 235371 [details] Patch
Comment on attachment 235371 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=235371&action=review Clever. Sounds good for now! r=me > Source/WebInspectorUI/UserInterface/Protocol/InspectorBackend.js:377 > + // than by setting a magical property on the callback. <webkit.org/b/132386> Nit: Include "https://" on that URL.
Comment on attachment 235371 [details] Patch Clearing flags on attachment: 235371 Committed r171489: <http://trac.webkit.org/changeset/171489>
All reviewed patches have been landed. Closing bug.