Commit ba40a39f authored by arthursonzogni's avatar arthursonzogni Committed by Commit Bot

NavigationSimulator: Simulate item and document sequence number.

Simulate Blink assigning an item and document sequence number to the
navigation. This is similar to what is done in
TestRenderFrameHost::SendNavigateWithParameters().

Bug: 728571
Change-Id: I6915d7e80154779c87294d71a1a395b862fc428b
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_browser_side_navigation_rel
Reviewed-on: https://chromium-review.googlesource.com/579380
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: default avatarCamille Lamy <clamy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488310}
parent 554a5a03
......@@ -328,8 +328,15 @@ void NavigationSimulator::Commit() {
params.history_list_was_cleared = false;
params.original_request_url = navigation_url_;
params.was_within_same_document = false;
params.page_state =
PageState::CreateForTesting(navigation_url_, false, nullptr, nullptr);
// Simulate Blink assigning an item and document sequence number to the
// navigation.
params.item_sequence_number = base::Time::Now().ToDoubleT() * 1000000;
params.document_sequence_number = params.item_sequence_number + 1;
params.page_state = PageState::CreateForTestingWithSequenceNumbers(
navigation_url_, params.item_sequence_number,
params.document_sequence_number);
render_frame_host_->SendNavigateWithParams(&params);
......@@ -420,8 +427,16 @@ void NavigationSimulator::CommitErrorPage() {
params.transition = transition_;
params.was_within_same_document = false;
params.url_is_unreachable = true;
params.page_state =
PageState::CreateForTesting(navigation_url_, false, nullptr, nullptr);
// Simulate Blink assigning an item and document sequence number to the
// navigation.
params.item_sequence_number = base::Time::Now().ToDoubleT() * 1000000;
params.document_sequence_number = params.item_sequence_number + 1;
params.page_state = PageState::CreateForTestingWithSequenceNumbers(
navigation_url_, params.item_sequence_number,
params.document_sequence_number);
render_frame_host_->SendNavigateWithParams(&params);
// Simulate the UnloadACK in the old RenderFrameHost if it was swapped out at
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment