[Webkit-unassigned] [Bug 170568] Web Inspector: WebSockets: frame context menu items should content sniff to be more relevant

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=170568

--- Comment #6 from Nikita Vasilyev <nvasilyev at apple.com> ---
(In reply to Joseph Pecoraro from comment #4)
> (In reply to Nikita Vasilyev from comment #3)
> > (In reply to Brian Burg from comment #1)
> > > Since this uses RemoteObject to get the data, this change may require to
> > > sniff the content in the model side since we don't want to block showing the
> > > context menu on a round trip to get the remote object.
> > 
> > Are you suggesting to JSON.parse every message when it's added? If so, I'd
> > have to test how much performance penalty it adds for big JSON messages.
> 
> We already do this when the context menu is triggered.
> 
> See the discussion and patches on bug 169945:
> https://bugs.webkit.org/show_bug.cgi?id=169945

Correct. We do this when we display the context menu, but not when a message is added.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170406/ccd55800/attachment.html>