[Webkit-unassigned] [Bug 140962] Web Inspector: type profiler does not dim functions that were never called

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 3 11:37:19 PDT 2016


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

Brian Burg <bburg at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bburg at apple.com

--- Comment #4 from Brian Burg <bburg at apple.com> ---
(In reply to comment #3)
> The problem here is that when the inspector first opens with type profiling
> enabled (or the user turns type profiling on mid-way through inspection),
> JavaScriptCore has an incomplete view of those functions that have run and
> those functions that have not run. Because of this, JavaScriptCore won't get
> completely accurate information until the page is refreshed.
> 
> The approach to this problem is two fold:
> 
> 1. We're going to switch the default to be from assuming a function has run
> to assuming it has not run. This doesn't solve the problem, but it is less
> misleading and probably a better default.
> 
> 2. We're removing gray backgrounds of unexecuted basic blocks and instead
> are going to lessen the saturation of the syntax highlighting on these
> unexecuted blocks. Again, this doesn't solve the problem, but provides a
> more subtle UI that is less obviously misleading. It is also more pleasant
> to use.
> 
> In the future, we can solve this more directly by at least considering
> enabling JavaScriptCore's /runtime/FunctionHasExecutedCache class to be
> enabled at all time and not only when the type profiler or control flow
> profiler is enabled.

What's the state of this bug today, Saam?

-- 
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/20160803/f9fb4d01/attachment.html>


More information about the webkit-unassigned mailing list