Add the Initiator field to the Headers tab |
||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.139 Safari/537.36 Steps to reproduce the problem: Currently, I must close the request details pane in order to see the explore initiator stack trace. I find myself doing it many times, but it should just show up on the Headers tab, or is its own tab, without closing the details pane. What is the expected behavior? What went wrong? :) Did this work before? No Chrome version: 66.0.3359.139 Channel: n/a OS Version: 10.0 Flash Version:
,
May 13 2018
Will the team accept this feature? If so, I might try to work on it.
,
May 13 2018
A separate tab sounds good! Thank you.
,
May 13 2018
Something like this? I think having it as another field in the Headers tab (right above "Response headers") will be much more convenient to the developer - less context switching - and a popover is enough, as well as consistent (is there any other place that shows a full stack trace, beside the console?). Since implementing it was so easy (half an hour, hehe), I will try and put it quickly in the Headers tab for experience comparison.
,
May 13 2018
This looks good. Requires a sentance of explanation though. It would be nice if method names were aligned to right though, with @ stacked vertically. I thought that was what standard stack renderer produced... Going forward, users would want to see more here. Try holding shift and hovering over network log. It would show (highlight with color) which requests it initiated and which requests it is initiated by. So there is a sizeable potential to accompany this boring stack with some more fun info in the new tab. Let's say, it has three sections: 1) Full async initiator stack (your patch) 2) List of requests that led to this request initiation 3) List of requests it caused. Each NetworkRequest sdk object has initiator chain, so should be super easy to implement. As for headers vs separate tab, hover is less accessible both from general UX and a11y perspectives. You could hover to see it in the grid, you can switch to the tab for exanded stack. If we rename Headers to Details, it would become appropriate and hover would work there, but it would be complementary to the separate tab. Happy hacking!
,
May 14 2018
Assigning
,
Sep 18
|
||||
►
Sign in to add a comment |
||||
Comment 1 by phistuck@chromium.org
, May 13 2018Status: Untriaged (was: Unconfirmed)