[webkit-reviews] review granted: [Bug 131791] Add mention of Safari Debug menu and finding out the WebProcess that corresponds to a tab : [Attachment 229545] Patch

[bugzilla-daemon at webkit.org]

Sam Weinig <sam at webkit.org> has granted Bem Jones-Bey <bjonesbe at adobe.com>'s
request for review:
Bug 131791: Add mention of Safari Debug menu and finding out the WebProcess
that corresponds to a tab
https://bugs.webkit.org/show_bug.cgi?id=131791

Attachment 229545: Patch
https://bugs.webkit.org/attachment.cgi?id=229545&action=review

------- Additional Comments from Sam Weinig <sam at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=229545&action=review


> Websites/webkit.org/building/debug.html:15
> +<p>Current versions of Safari are very multiprocess, so it is easy to end up
with a new WebProcess and have to reattach. If you enable the Internal Debug
Menu as outlined above, you can have Safari display the Process ID of the
WebProcess for each tab. Simply go to Debug &gt; Miscellaneous Flags and select
Show Web Process IDs In Page Titles. You should now see something like
<code>[WP 60737]</code> in the title of every web page you navigate to, and
60737 is the Process ID of the WebProcess backing that tab.</p>

I'm not a huge fan of "very multiprocess".  How about something like, "Current
versions of Safari us a process for each tab".