diff options
| author | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-21 21:33:16 +0000 |
|---|---|---|
| committer | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-21 21:33:16 +0000 |
| commit | a5500a1cc6aafe73ff19e0daeafdb935a3a0a6bc (patch) | |
| tree | d73f8ea9aa66d82446f1629190e84c5013896507 /chrome/browser/tab_contents/tab_contents.h | |
| parent | 679082058ce725b3bc6024c2bc01a7b99b79b3c6 (diff) | |
| download | chromium_src-a5500a1cc6aafe73ff19e0daeafdb935a3a0a6bc.zip chromium_src-a5500a1cc6aafe73ff19e0daeafdb935a3a0a6bc.tar.gz chromium_src-a5500a1cc6aafe73ff19e0daeafdb935a3a0a6bc.tar.bz2 | |
Links opening up in the host browser in ChromeFrame would not show up as visited. This was because top level link navigations
are directly sent via the OpenURL mechanism to the host browser and as a result the visited link history is not updated.
Fix is to update the navigation history in the ExternalTabContainer::OpenURLFromTab function. We currently use the
UpdateHistoryForNavigation function in TabContents which is also used by regular navigations initiated within the renderer.
Fixes bug http://code.google.com/p/chromium/issues/detail?id=37262
Bug=37262
Review URL: http://codereview.chromium.org/3020019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53247 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/tab_contents/tab_contents.h')
| -rw-r--r-- | chrome/browser/tab_contents/tab_contents.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/chrome/browser/tab_contents/tab_contents.h b/chrome/browser/tab_contents/tab_contents.h index 8d14246..227dcea 100644 --- a/chrome/browser/tab_contents/tab_contents.h +++ b/chrome/browser/tab_contents/tab_contents.h @@ -19,6 +19,7 @@ #include "chrome/browser/cancelable_request.h" #include "chrome/browser/dom_ui/dom_ui_factory.h" #include "chrome/browser/download/save_package.h" +#include "chrome/browser/external_tab_container_win.h" #include "chrome/browser/extensions/image_loading_tracker.h" #include "chrome/browser/fav_icon_helper.h" #include "chrome/browser/find_bar_controller.h" @@ -738,6 +739,9 @@ class TabContents : public PageNavigator, // TODO(brettw) TestTabContents shouldn't exist! friend class TestTabContents; + // Used to access the UpdateHistoryForNavigation function. + friend class ExternalTabContainer; + // Changes the IsLoading state and notifies delegate as needed // |details| is used to provide details on the load that just finished // (but can be null if not applicable). Can be overridden. |