In chrome, an entry is added/updated in the history database in
HistoryTabHelper::DidNavigateAnyFrame(), which is called by
content::WebContents::DidNavigateAnyFramePostCommit(), which itself is
called by content::NavigatorImpl::DidNavigate(). This corresponds to a
LoadCommitted event in oxide.

LoadCommitted events emitted by oxide originate from
content::WebContents::DidCommitProvisionalLoad(), which is called by
content::NavigatorImpl::DidNavigate().

This confirms that we should indeed store entries in the shared history
when getting LoadCommitted events, not LoadSucceeded.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1455858

Title:
  History AWOL with YouTube "get more videos" button.

To manage notifications about this bug go to:
https://bugs.launchpad.net/oxide/+bug/1455858/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to