| Index: content/test/test_render_frame_host.cc
|
| diff --git a/content/test/test_render_frame_host.cc b/content/test/test_render_frame_host.cc
|
| index 838c42350666a0e00ff24e3945e6bde5abb5b1ef..bc54c3c3824d907bdaa878b278c122557c87d637 100644
|
| --- a/content/test/test_render_frame_host.cc
|
| +++ b/content/test/test_render_frame_host.cc
|
| @@ -79,6 +79,13 @@
|
| child_creation_observer_.last_created_frame());
|
| }
|
|
|
| +void TestRenderFrameHost::SendNavigateWithTransition(
|
| + int page_id,
|
| + const GURL& url,
|
| + ui::PageTransition transition) {
|
| + SendNavigateWithTransitionAndResponseCode(page_id, url, transition, 200);
|
| +}
|
| +
|
| void TestRenderFrameHost::SetContentsMimeType(const std::string& mime_type) {
|
| contents_mime_type_ = mime_type;
|
| }
|
| @@ -92,39 +99,18 @@
|
| OnSwappedOut();
|
| }
|
|
|
| -void TestRenderFrameHost::SendNavigate(int page_id,
|
| - int nav_entry_id,
|
| - bool did_create_new_entry,
|
| - const GURL& url) {
|
| - SendNavigateWithTransition(page_id, nav_entry_id, did_create_new_entry, url,
|
| - ui::PAGE_TRANSITION_LINK);
|
| -}
|
| -
|
| -void TestRenderFrameHost::SendFailedNavigate(int page_id,
|
| - int nav_entry_id,
|
| - bool did_create_new_entry,
|
| - const GURL& url) {
|
| - SendNavigateWithTransitionAndResponseCode(page_id, nav_entry_id,
|
| - did_create_new_entry, url,
|
| - ui::PAGE_TRANSITION_RELOAD, 500);
|
| -}
|
| -
|
| -void TestRenderFrameHost::SendNavigateWithTransition(
|
| - int page_id,
|
| - int nav_entry_id,
|
| - bool did_create_new_entry,
|
| - const GURL& url,
|
| - ui::PageTransition transition) {
|
| +void TestRenderFrameHost::SendNavigate(int page_id, const GURL& url) {
|
| + SendNavigateWithTransition(page_id, url, ui::PAGE_TRANSITION_LINK);
|
| +}
|
| +
|
| +void TestRenderFrameHost::SendFailedNavigate(int page_id, const GURL& url) {
|
| SendNavigateWithTransitionAndResponseCode(
|
| - page_id, nav_entry_id, did_create_new_entry, url, transition, 200);
|
| + page_id, url, ui::PAGE_TRANSITION_RELOAD, 500);
|
| }
|
|
|
| void TestRenderFrameHost::SendNavigateWithTransitionAndResponseCode(
|
| int page_id,
|
| - int nav_entry_id,
|
| - bool did_create_new_entry,
|
| - const GURL& url,
|
| - ui::PageTransition transition,
|
| + const GURL& url, ui::PageTransition transition,
|
| int response_code) {
|
| // DidStartProvisionalLoad may delete the pending entry that holds |url|,
|
| // so we keep a copy of it to use in SendNavigateWithParameters.
|
| @@ -135,15 +121,12 @@
|
| SetRenderFrameCreated(true);
|
|
|
| OnDidStartProvisionalLoadForFrame(url_copy, false);
|
| - SendNavigateWithParameters(page_id, nav_entry_id, did_create_new_entry,
|
| - url_copy, transition, url_copy, response_code, 0,
|
| - std::vector<GURL>());
|
| + SendNavigateWithParameters(page_id, url_copy, transition, url_copy,
|
| + response_code, 0, std::vector<GURL>());
|
| }
|
|
|
| void TestRenderFrameHost::SendNavigateWithOriginalRequestURL(
|
| int page_id,
|
| - int nav_entry_id,
|
| - bool did_create_new_entry,
|
| const GURL& url,
|
| const GURL& original_request_url) {
|
| // Ensure that the RenderFrameCreated notification has been sent to observers
|
| @@ -151,20 +134,16 @@
|
| SetRenderFrameCreated(true);
|
|
|
| OnDidStartProvisionalLoadForFrame(url, false);
|
| - SendNavigateWithParameters(page_id, nav_entry_id, did_create_new_entry, url,
|
| - ui::PAGE_TRANSITION_LINK, original_request_url,
|
| - 200, 0, std::vector<GURL>());
|
| + SendNavigateWithParameters(page_id, url, ui::PAGE_TRANSITION_LINK,
|
| + original_request_url, 200, 0, std::vector<GURL>());
|
| }
|
|
|
| void TestRenderFrameHost::SendNavigateWithFile(
|
| int page_id,
|
| - int nav_entry_id,
|
| - bool did_create_new_entry,
|
| const GURL& url,
|
| const base::FilePath& file_path) {
|
| - SendNavigateWithParameters(page_id, nav_entry_id, did_create_new_entry, url,
|
| - ui::PAGE_TRANSITION_LINK, url, 200, &file_path,
|
| - std::vector<GURL>());
|
| + SendNavigateWithParameters(page_id, url, ui::PAGE_TRANSITION_LINK, url, 200,
|
| + &file_path, std::vector<GURL>());
|
| }
|
|
|
| void TestRenderFrameHost::SendNavigateWithParams(
|
| @@ -175,18 +154,14 @@
|
|
|
| void TestRenderFrameHost::SendNavigateWithRedirects(
|
| int page_id,
|
| - int nav_entry_id,
|
| - bool did_create_new_entry,
|
| const GURL& url,
|
| const std::vector<GURL>& redirects) {
|
| - SendNavigateWithParameters(page_id, nav_entry_id, did_create_new_entry, url,
|
| - ui::PAGE_TRANSITION_LINK, url, 200, 0, redirects);
|
| + SendNavigateWithParameters(
|
| + page_id, url, ui::PAGE_TRANSITION_LINK, url, 200, 0, redirects);
|
| }
|
|
|
| void TestRenderFrameHost::SendNavigateWithParameters(
|
| int page_id,
|
| - int nav_entry_id,
|
| - bool did_create_new_entry,
|
| const GURL& url,
|
| ui::PageTransition transition,
|
| const GURL& original_request_url,
|
| @@ -195,7 +170,6 @@
|
| const std::vector<GURL>& redirects) {
|
| FrameHostMsg_DidCommitProvisionalLoad_Params params;
|
| params.page_id = page_id;
|
| - params.nav_entry_id = nav_entry_id;
|
| params.url = url;
|
| params.referrer = Referrer();
|
| params.transition = transition;
|
| @@ -203,7 +177,6 @@
|
| params.should_update_history = true;
|
| params.searchable_form_url = GURL();
|
| params.searchable_form_encoding = std::string();
|
| - params.did_create_new_entry = did_create_new_entry;
|
| params.security_info = std::string();
|
| params.gesture = NavigationGestureUser;
|
| params.contents_mime_type = contents_mime_type_;
|
| @@ -231,13 +204,11 @@
|
| OnDidCommitProvisionalLoad(msg);
|
| }
|
|
|
| -void TestRenderFrameHost::NavigateAndCommitRendererInitiated(
|
| - int page_id,
|
| - bool did_create_new_entry,
|
| - const GURL& url) {
|
| +void TestRenderFrameHost::NavigateAndCommitRendererInitiated(int page_id,
|
| + const GURL& url) {
|
| SendRendererInitiatedNavigationRequest(url, false);
|
| PrepareForCommit();
|
| - SendNavigate(page_id, 0, did_create_new_entry, url);
|
| + SendNavigate(page_id, url);
|
| }
|
|
|
| void TestRenderFrameHost::SendRendererInitiatedNavigationRequest(
|
|
|