This icon should be right next to [T] icon, that toggles Type Profiler.
<rdar://problem/27855650>
Created attachment 286412 [details] Patch
Created attachment 286413 [details] [Image] C icon
To answer possible copyright questions: I drew this "C" in Photoshop. I didn't use any existing fonts.
Comment on attachment 286412 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=286412&action=review Looks good! I assume you took a look on non-Retain too and it looked fine? File name is a bit weird. "Code Coverage" seems better than "Control Flow". > Source/WebInspectorUI/UserInterface/Images/NavigationItemControlFlow.svg:5 > + <path fill="none" stroke="currentColor" stroke-width="1" d="M9,5.5s0-1.5-2.5-1.5C3,4,3,10,6.5,10c2.5,0,2.5-1.5,2.5-1.5"/> This style for path is to use spaces not commas.
Created attachment 286467 [details] [Image] C icon @1x Pixel hinting isn't great, but I'd rather land this first and polish it later if we decide to stick with this icon.
Created attachment 286468 [details] Patch
Comment on attachment 286468 [details] Patch Clearing flags on attachment: 286468 Committed r204646: <http://trac.webkit.org/changeset/204646>
All reviewed patches have been landed. Closing bug.
(In reply to comment #5) > File name is a bit weird. "Code Coverage" seems better than "Control Flow". We have RuntimeAgent.enableControlFlowProfiler. Are you okay with this name? Or should it be enableCodeCoverageProfiler?
(In reply to comment #10) > (In reply to comment #5) > > File name is a bit weird. "Code Coverage" seems better than "Control Flow". > > We have RuntimeAgent.enableControlFlowProfiler. Are you okay with this name? > Or should it be enableCodeCoverageProfiler? I think code coverage profiler is better.