Summary: | Web Inspector: allow recordings to be saved via ⌘-S in the Recording tab | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bburg, commit-queue, inspector-bugzilla-changes, joepeck, mattbaker, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | 174484 | ||||||||
Bug Blocks: | 173807 | ||||||||
Attachments: |
|
Description
Devin Rousso
2017-08-07 13:23:04 PDT
Created attachment 317461 [details]
Patch
Comment on attachment 317461 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=317461&action=review r-, since this duplicates code in RecordingNavigationSidebarPanel. The sidebar should probably be using the new `saveData()` getter. > Source/WebInspectorUI/UserInterface/Views/RecordingContentView.js:99 > + forceSaveAs: true, What are your thoughts on versioning? (In reply to Matt Baker from comment #2) > Comment on attachment 317461 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=317461&action=review > > r-, since this duplicates code in RecordingNavigationSidebarPanel. The > sidebar should probably be using the new `saveData()` getter. > > > Source/WebInspectorUI/UserInterface/Views/RecordingContentView.js:99 > > + forceSaveAs: true, > > What are your thoughts on versioning? We already have a version in the Recording protocol. I'm not sure what you mean otherwise. Created attachment 318736 [details]
Patch
Comment on attachment 318736 [details]
Patch
r=me
Comment on attachment 318736 [details] Patch Clearing flags on attachment: 318736 Committed r222003: <http://trac.webkit.org/changeset/222003> All reviewed patches have been landed. Closing bug. Comment on attachment 318736 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=318736&action=review > Source/WebInspectorUI/UserInterface/Views/RecordingNavigationSidebarPanel.js:171 > + WI.saveDataToFile(this.contentBrowser.currentContentView.saveData, forceSaveAs); The saveData already has `forceSaveAs: true`. I suppose its fine to be explicit here, but this means that no matter what ContentView eventually shows up in the Recording tab will always be a forceSaveAs even if the ContentView logically can re-save. I don't think that ever be likely to happen though. |