If we switch into ChromeFrame as a result of a redirect then we need to navig...

If we switch into ChromeFrame as a result of a redirect then we need to navigate chrome to
the correct URL to ensure that scripts on the page see the correct URL. This functionality
regressed when the protocol sink patch landed. 

Fix is to read the URL from the ProtData object which wraps the underlying protocol sink,
when our active document loads.

Removed stale code from the Chromeframe moniker patch which eventually needs to go.

Fixes bug http://code.google.com/p/chromium/issues/detail?can=2&q=71860

BUG=71860
TEST=As described in the bug.


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

[amit, 2011-02-22 23:12:05]

lg

http://codereview.chromium.org/6469103/diff/1/chrome_frame/bind_context_info.cc
File chrome_frame/bind_context_info.cc (right):

http://codereview.chromium.org/6469103/diff/1/chrome_frame/bind_context_info....
chrome_frame/bind_context_info.cc:84: scoped_refptr<ProtData> prot_data =
get_prot_data();
nit: this can simply be return prot_data_->url(), no need for extra refptr.

http://codereview.chromium.org/6469103/diff/1/chrome_frame/bind_context_info.h
File chrome_frame/bind_context_info.h (right):

http://codereview.chromium.org/6469103/diff/1/chrome_frame/bind_context_info....
chrome_frame/bind_context_info.h:107: std::wstring url_;
OK to remove this now?

[ananta, 2011-02-22 23:15:49]

http://codereview.chromium.org/6469103/diff/1/chrome_frame/bind_context_info.cc
File chrome_frame/bind_context_info.cc (right):

http://codereview.chromium.org/6469103/diff/1/chrome_frame/bind_context_info....
chrome_frame/bind_context_info.cc:84: scoped_refptr<ProtData> prot_data =
get_prot_data();
On 2011/02/22 23:12:05, amit wrote:
> nit: this can simply be return prot_data_->url(), no need for extra refptr.

Done.

http://codereview.chromium.org/6469103/diff/1/chrome_frame/bind_context_info.h
File chrome_frame/bind_context_info.h (right):

http://codereview.chromium.org/6469103/diff/1/chrome_frame/bind_context_info....
chrome_frame/bind_context_info.h:107: std::wstring url_;
On 2011/02/22 23:12:05, amit wrote:
> OK to remove this now?

Done.