Chromium Code Reviews| Index: content/browser/web_contents/web_contents_impl.cc |
| diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc |
| index 152eceb6e4b3ed7fc238f64250fb1bfd54c9d7e6..7611f57a00217e223bcceabf77830eecf082f38a 100644 |
| --- a/content/browser/web_contents/web_contents_impl.cc |
| +++ b/content/browser/web_contents/web_contents_impl.cc |
| @@ -2685,7 +2685,13 @@ void WebContentsImpl::DidNavigateMainFramePostCommit( |
| UpdateTargetURL(details.entry->GetPageID(), GURL()); |
| } |
| - if (!details.is_in_page) { |
| + // When navigating back from a ref URL to the top non ref one is_in_page |
| + // is false even the navigation is in fact in page |
|
abarth-chromium
2013/08/01 20:48:32
Should we fix that?
|
| + // (see AreURLsInPageNavigation()). In order to not clear |
| + // the displayed_insecure_content_ flag too eagerly check the navigation type |
| + // as well. |
| + if (!details.is_in_page && |
| + details.type != content::NAVIGATION_TYPE_IN_PAGE) { |
| // Once the main frame is navigated, we're no longer considered to have |
| // displayed insecure content. |
| displayed_insecure_content_ = false; |