RESOLVED FIXED 141875
Web Inspector: Generate Previews more often for RemoteObject interaction
https://bugs.webkit.org/show_bug.cgi?id=141875
Summary Web Inspector: Generate Previews more often for RemoteObject interaction
Joseph Pecoraro
Reported 2015-02-21 18:25:24 PST
* SUMMARY Generate Previews more often for RemoteObject interaction. Enable, fix, and use "generatePreview" in more places.
Attachments
[PATCH] Proposed Fix (18.63 KB, patch)
2015-02-21 18:30 PST, Joseph Pecoraro
no flags
[PATCH] Proposed Fix (18.63 KB, patch)
2015-02-21 18:31 PST, Joseph Pecoraro
timothy: review+
timothy: commit-queue-
[PATCH] For Landing (18.64 KB, patch)
2015-02-21 19:00 PST, Joseph Pecoraro
no flags
Radar WebKit Bug Importer
Comment 1 2015-02-21 18:25:39 PST
Joseph Pecoraro
Comment 2 2015-02-21 18:30:52 PST
Created attachment 247071 [details] [PATCH] Proposed Fix
Joseph Pecoraro
Comment 3 2015-02-21 18:31:28 PST
Created attachment 247072 [details] [PATCH] Proposed Fix
Timothy Hatcher
Comment 4 2015-02-21 18:43:32 PST
Comment on attachment 247072 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=247072&action=review > Source/WebInspectorUI/UserInterface/Protocol/RemoteObject.js:394 > + RuntimeAgent.callFunctionOn(this._objectId, functionDeclaration.toString(), args, true, undefined, true mycallback); Missing a comma.
Joseph Pecoraro
Comment 5 2015-02-21 18:56:11 PST
Comment on attachment 247072 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=247072&action=review >> Source/WebInspectorUI/UserInterface/Protocol/RemoteObject.js:394 >> + RuntimeAgent.callFunctionOn(this._objectId, functionDeclaration.toString(), args, true, undefined, true mycallback); > > Missing a comma. Arg, good catch!! This was me rebasing to get isolated patches and this was the only conflict. Fail!
Joseph Pecoraro
Comment 6 2015-02-21 19:00:46 PST
Created attachment 247074 [details] [PATCH] For Landing
WebKit Commit Bot
Comment 7 2015-02-21 19:50:49 PST
Comment on attachment 247074 [details] [PATCH] For Landing Clearing flags on attachment: 247074 Committed r180483: <http://trac.webkit.org/changeset/180483>
WebKit Commit Bot
Comment 8 2015-02-21 19:50:53 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.