commit | 9fb98d15b5984dadb15ffa2475cc536ab539f62e | [log] [tgz] |
---|---|---|
author | Eugene But <[email protected]> | Tue May 22 18:28:33 2018 |
committer | Commit Bot <[email protected]> | Tue May 22 18:28:33 2018 |
tree | 2515bed8591d47bfb4d700193bf531c766d96906 | |
parent | 23e411cc8666eb095201383bb1872349d72cff60 [diff] [blame] |
Renamed IsURLInPageNavigation to IsURLSameDocumentNavigation This fixes the comment. The function was renamed here: https://chromium-review.googlesource.com/675443 Bug: 695189 Change-Id: I58854785604b549d741c1a6287f66a767f749c12 Reviewed-on: https://chromium-review.googlesource.com/1067848 Reviewed-by: Charlie Reis <[email protected]> Commit-Queue: Eugene But <[email protected]> Cr-Commit-Position: refs/heads/master@{#560710}
diff --git a/content/browser/frame_host/navigation_controller_impl.cc b/content/browser/frame_host/navigation_controller_impl.cc index 3ce1e355..6020ab2 100644 --- a/content/browser/frame_host/navigation_controller_impl.cc +++ b/content/browser/frame_host/navigation_controller_impl.cc
@@ -1601,8 +1601,8 @@ // (https://crbug.com/373041). // TODO(creis): For now, restrict this check to HTTP(S) origins, because // about:blank, file, and unique origins are more subtle to get right. - // We'll abstract out the relevant checks from IsURLInPageNavigation and - // share them here. See https://crbug.com/618104. + // We'll abstract out the relevant checks from IsURLSameDocumentNavigation + // and share them here. See https://crbug.com/618104. const GURL& dest_top_url = GetEntryAtIndex(entry_index)->GetURL(); const GURL& current_top_url = GetLastCommittedEntry()->GetURL(); if (current_top_url.SchemeIsHTTPOrHTTPS() &&