Fix for the chrome frame anchor url navigation test failures. Fix is to appen...

Fix for the chrome frame anchor url navigation test failures. Fix is to append the anchor
portion of the url retrieved from the navigation manager to the url retrieved from the
protocol sink wrapper. This is needed as the protocol sink wrapper sees the url without
the anchor portion.

BUG=71860
TEST=ChromeFrame anchor url tests.

TBR=amit

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=75704

[amit, 2011-02-23 19:32:17]

ok

On Tue, Feb 22, 2011 at 5:04 PM, <ananta@chromium.org> wrote:

> Reviewers: amit,
>
> Description:
> Fix for the chrome frame anchor url navigation test failures. Fix is to
> append
> the anchor
> portion of the url retrieved from the navigation manager to the url
> retrieved
> from the
> protocol sink wrapper. This is needed as the protocol sink wrapper sees the
> url
> without
> the anchor portion.
>
> BUG=71860
> TEST=ChromeFrame anchor url tests.
>
>
> Please review this at http://codereview.chromium.org/6546081/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src/
>
> Affected files:
>  M     chrome_frame/chrome_active_document.cc
>
>
> Index: chrome_frame/chrome_active_document.cc
> ===================================================================
> --- chrome_frame/chrome_active_document.cc      (revision 75656)
> +++ chrome_frame/chrome_active_document.cc      (working copy)
> @@ -250,6 +250,16 @@
>   DCHECK(info);
>   if (info && !info->GetUrl().empty()) {
>     url = info->GetUrl();
> +    if (mgr) {
> +      // If the original URL contains an anchor, then the URL queried
> +      // from the protocol sink wrapper does not contain the anchor. To
> +      // workaround this we retrieve the anchor from the navigation
> manager
> +      // and append it to the url retrieved from the protocol sink
> wrapper.
> +      GURL url_for_anchor(mgr->url());
> +      if (url_for_anchor.has_ref()) {
> +        url += UTF8ToWide(url_for_anchor.ref());
> +      }
> +    }
>   } else {
>     // If the original URL contains an anchor, then the URL queried
>     // from the moniker does not contain the anchor. To workaround
>
>
>