The notification will be used to know when history state is out of date.
Created attachment 42532 [details] Adds FrameLoaderClient::didChangeScrollOffset
Can you please explain what you mean by the "history state is out of date"?
The frame's current scroll offset is a property of HistoryItem. However that field is only updated when HistoryController::saveDocumentAndScrollState() is called, which does not ordinarily happen when a frame scrolls. Chromium needs a way to know when the scroll offset for any frame has changed, so that it can schedule a call to saveDocumentAndScrollState(). Our objective is to use this as a signal to save session information for Chromium's session restore feature.
Comment on attachment 42532 [details] Adds FrameLoaderClient::didChangeScrollOffset > Index: WebCore/ChangeLog .. > + Reviewed by NOBODY (OOPS!). > + > + Adds FrameLoaderClient::didChangeScrollOffset that is called when > + the frame scrolls. This will be used to know when history state > + needs to be updated. ^^^ fix the indentation please include a link to this bug. delete the line below: > + No new tests. LGTM otherwise
Created attachment 42571 [details] Revised patch
Sam, sorry for not being clear on why I want to add this. Darin, thanks for clarifying!
Comment on attachment 42571 [details] Revised patch r=me per Darin's comments.
Comment on attachment 42571 [details] Revised patch Clearing flags on attachment: 42571 Committed r50580: <http://trac.webkit.org/changeset/50580>
All reviewed patches have been landed. Closing bug.