Summary: | Web Inspector: Canvas Tab: reloading from canvas overview after making a recording will show the recording instead of overview | ||
---|---|---|---|
Product: | WebKit | Reporter: | BJ Burg <bburg> |
Component: | Web Inspector | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED WORKSFORME | ||
Severity: | Normal | CC: | bburg, hi, inspector-bugzilla-changes, webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | WebKit Nightly Build | ||
Hardware: | All | ||
OS: | All | ||
URL: | http://acko.net | ||
Bug Depends on: | |||
Bug Blocks: | 173807, 175485 |
Description
BJ Burg
2017-11-06 15:30:37 PST
I don't think that they are auto-selected. My guess as to what is happening is that we are never switching views away from the Recording, which is why it stays visible. From what I remember, this is controlled by CanvasTabContentView.prototype.restoreStateFromCookie (and saveStateToCookie), so if we want to do this then we probably just want to modify those functions. Personally, if I am viewing a recording (which is pretty independent of the page itself), I wouldn't want the recording to be deselected/hidden if I refresh the page. Due to the exportability of Recordings, I don't see them as being "restricted" to a particular instance of a given page. I see them more as a snapshot in time of something that happened, and WebInspector is just the means by which we can view that snapshot. I can see the argument against this train of thought, however, so I could go either way. (In reply to Devin Rousso from comment #2) > I don't think that they are auto-selected. My guess as to what is happening > is that we are never switching views away from the Recording, which is why > it stays visible. From what I remember, this is controlled by > CanvasTabContentView.prototype.restoreStateFromCookie (and > saveStateToCookie), so if we want to do this then we probably just want to > modify those functions. > > Personally, if I am viewing a recording (which is pretty independent of the > page itself), I wouldn't want the recording to be deselected/hidden if I > refresh the page. Due to the exportability of Recordings, I don't see them > as being "restricted" to a particular instance of a given page. I see them > more as a snapshot in time of something that happened, and WebInspector is > just the means by which we can view that snapshot. I can see the argument > against this train of thought, however, so I could go either way. I'm not proposing to hide the recording details and go back to overview when the page reloads. I'm proposing that we stay on the overview when the page reloads, but currently it jumps to a recording I wasn't looking at previously. (In reply to Brian Burg from comment #3) > I'm not proposing to hide the recording details and go back to overview when > the page reloads. I'm proposing that we stay on the overview when the page > reloads, but currently it jumps to a recording I wasn't looking at > previously. I'm not seeing this issue on ToT. I followed your steps to reproduce, and when I refreshed the page I stayed on the overview view. I can't find a reliable reproduction, so let's close this. |