diff options
| author | boliu@chromium.org <boliu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-30 00:29:43 +0000 |
|---|---|---|
| committer | boliu@chromium.org <boliu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-30 00:29:43 +0000 |
| commit | 59a733d9060c359966c39e50b78d219a0a84fddb (patch) | |
| tree | efb0498d1902f04f3bce33c7f6f150a6088939e0 /android_webview | |
| parent | eb9990e7aa5790de3f308de778838b802bee5850 (diff) | |
| download | chromium_src-59a733d9060c359966c39e50b78d219a0a84fddb.zip chromium_src-59a733d9060c359966c39e50b78d219a0a84fddb.tar.gz chromium_src-59a733d9060c359966c39e50b78d219a0a84fddb.tar.bz2 | |
Android webview fix visited link
FrameNavigateParams::base_url is empty most of the time. Plus also
adding all redirect URLs to the history to match chrome behavior.
BUG=
TBR=mkosiba@chromium.org
Android only change. Ran through android trybots
NOTRY=true
Review URL: https://chromiumcodereview.appspot.com/12050007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179481 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'android_webview')
| -rw-r--r-- | android_webview/browser/aw_browser_context.cc | 5 | ||||
| -rw-r--r-- | android_webview/browser/aw_browser_context.h | 1 | ||||
| -rw-r--r-- | android_webview/browser/renderer_host/aw_render_view_host_ext.cc | 2 |
3 files changed, 1 insertions, 7 deletions
diff --git a/android_webview/browser/aw_browser_context.cc b/android_webview/browser/aw_browser_context.cc index 010d692..d7f9789 100644 --- a/android_webview/browser/aw_browser_context.cc +++ b/android_webview/browser/aw_browser_context.cc @@ -38,11 +38,6 @@ void AwBrowserContext::PreMainMessageLoopRun() { visitedlink_master_->Init(); } -void AwBrowserContext::AddVisitedURL(const GURL& url) { - DCHECK(visitedlink_master_); - visitedlink_master_->AddURL(url); -} - void AwBrowserContext::AddVisitedURLs(const std::vector<GURL>& urls) { DCHECK(visitedlink_master_); visitedlink_master_->AddURLs(urls); diff --git a/android_webview/browser/aw_browser_context.h b/android_webview/browser/aw_browser_context.h index ae2b3958..198c970 100644 --- a/android_webview/browser/aw_browser_context.h +++ b/android_webview/browser/aw_browser_context.h @@ -51,7 +51,6 @@ class AwBrowserContext : public content::BrowserContext, void PreMainMessageLoopRun(); // These methods map to Add methods in components::VisitedLinkMaster. - void AddVisitedURL(const GURL& url); void AddVisitedURLs(const std::vector<GURL>& urls); // content::BrowserContext implementation. diff --git a/android_webview/browser/renderer_host/aw_render_view_host_ext.cc b/android_webview/browser/renderer_host/aw_render_view_host_ext.cc index 38b3a95..6eee62b 100644 --- a/android_webview/browser/renderer_host/aw_render_view_host_ext.cc +++ b/android_webview/browser/renderer_host/aw_render_view_host_ext.cc @@ -79,7 +79,7 @@ void AwRenderViewHostExt::DidNavigateAnyFrame( DCHECK(CalledOnValidThread()); AwBrowserContext::FromWebContents(web_contents()) - ->AddVisitedURL(params.base_url); + ->AddVisitedURLs(params.redirects); } bool AwRenderViewHostExt::OnMessageReceived(const IPC::Message& message) { |