Bug 135570 - Web Inspector: timeline overview view state is not saved with BackForwardEntry navigations
Summary: Web Inspector: timeline overview view state is not saved with BackForwardEntr...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks: WebReplayUI
  Show dependency tree / graph
 
Reported: 2014-08-04 12:55 PDT by Brian Burg
Modified: 2016-12-13 15:36 PST (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brian Burg 2014-08-04 12:55:46 PDT 
Each back/forward entry currently represents a navigation + view state. Differing view states cannot create new back/forward entries. View states store the effects of user interaction within a content view, provided that the interaction doesn't create a new navigation.

Saved view state is currently limited to scroll positions of elements nominated by ContentView.prototype.get_scrollableElements. But, some view state that's important to the user (namely, selection ranges in the timeline) are not implemented with DOM scrolling, so that view state is not handled. ContentViews should be able to
Comment 1 Radar WebKit Bug Importer 2014-08-04 12:56:17 PDT 
<rdar://problem/17905858>
Comment 2 Brian Burg 2014-08-04 12:56:40 PDT 
.. provide custom save/restore functions. Importantly, these should not save into the BackForwardEntry's cookie, since that is for navigation state only.